summaryrefslogtreecommitdiffstats
path: root/hw/fsp/fsp-mem-err.c
Commit message (Collapse)AuthorAgeFilesLines
* sparse: fix incorrect type in assignment in fsp-mem-err.cStewart Smith2015-11-101-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | Since skiboot is all BE, no built code changes hw/fsp/fsp-mem-err.c:215:35: warning: invalid assignment: |= hw/fsp/fsp-mem-err.c:215:35: left side has type restricted beint16_t hw/fsp/fsp-mem-err.c:215:35: right side has type int hw/fsp/fsp-mem-err.c:228:57: warning: incorrect type in assignment (different base types) hw/fsp/fsp-mem-err.c:228:57: expected restricted beint64_t [addressable] [assigned] [usertype] physical_address_start hw/fsp/fsp-mem-err.c:228:57: got unsigned long long [unsigned] [usertype] paddr hw/fsp/fsp-mem-err.c:229:55: warning: incorrect type in assignment (different base types) hw/fsp/fsp-mem-err.c:229:55: expected restricted beint64_t [addressable] [assigned] [usertype] physical_address_end hw/fsp/fsp-mem-err.c:229:55: got unsigned long long hw/fsp/fsp-mem-err.c:269:65: warning: incorrect type in assignment (different base types) hw/fsp/fsp-mem-err.c:269:65: expected restricted beint64_t [usertype] physical_address_end hw/fsp/fsp-mem-err.c:269:65: got unsigned long long [unsigned] [usertype] paddr_end hw/fsp/fsp-mem-err.c:318:58: warning: incorrect type in assignment (different base types) hw/fsp/fsp-mem-err.c:318:58: expected restricted beint64_t [addressable] [assigned] [usertype] physical_address_start hw/fsp/fsp-mem-err.c:318:58: got unsigned long long [unsigned] [usertype] paddr_start hw/fsp/fsp-mem-err.c:319:56: warning: incorrect type in assignment (different base types) hw/fsp/fsp-mem-err.c:319:56: expected restricted beint64_t [addressable] [assigned] [usertype] physical_address_end hw/fsp/fsp-mem-err.c:319:56: got unsigned long long [unsigned] [usertype] paddr_end Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* sparse: fix be64 degrades to integer warnings in fsp-mem-err.cStewart Smith2015-11-101-6/+6
| | | | | | | | | | No code changes in skiboot due to being BE hw/fsp/fsp-mem-err.c:162:44: warning: restricted beint64_t degrades to integer hw/fsp/fsp-mem-err.c:263:45: warning: restricted beint64_t degrades to integer hw/fsp/fsp-mem-err.c:266:52: warning: restricted beint64_t degrades to integer Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* Fix spelling mistakesStewart Smith2015-08-261-1/+1
| | | | | | See https://github.com/lucasdemarchi/codespel Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* errorlog: Deprecate elog callback parameterSamuel Mendoza-Jonas2015-07-311-2/+2
| | | | | | | | | | There are now no users of the call_out parameter and future users should use the log_append_msg() and log_append_data() functions, so remove all references to call_out. Signed-off-by: Samuel Mendoza-Jonas <sam.mj@au1.ibm.com> Reviewed-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* errorlog: Simplify log calling conventionSamuel Mendoza-Jonas2015-07-311-13/+16
| | | | | | | | | | | | | | | | | | | Remove the callback functionality from log_error() and replace it with the ability to append to a user data section, or add addtional user data sections to an error log. For multiline or otherwise complex logging the convention is now to call opal_elog_create() to obtain a errorlog buffer and use log_append_msg() and log_append_data() to append to the user data section. Additional user data sections can be added to the error log via log_add_section(). The caller is then responsible for calling log_commit(). For simple logs log_simple_error() takes care of creating and committing the error log as before. Signed-off-by: Samuel Mendoza-Jonas <sam.mj@au1.ibm.com> Reviewed-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* fsp/mem-err: Fix leak in one error path and minor clean upsNeelesh Gupta2015-07-101-10/+4
| | | | | Signed-off-by: Neelesh Gupta <neelegup@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* Add pr_fmt to do printf/prlog prefixing automagicallyStewart Smith2015-05-121-12/+9
| | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* elog: Clean up error logging headersAlistair Popple2014-12-021-1/+1
| | | | | | | | | | | | | | | Commit cf6f4e8912d29fb89ce85c84834607065ad595a5 introduced a platform independent frontend for error logging. However it failed to move the generic parts of the fsp-elog.h header into the platform independent one, instead relying on the fact that up until now fsp-elog.h was included whenever a function needed to log errors. This patch moves the platform independent defines into the frontend header file (errorlog.h) and removes the include of the platform specific header in generic code paths. Signed-off-by: Alistair Popple <alistair@popple.id.au> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* elog: Remove opal prefix from data structures in errorlog.hDeepthi Dharwar2014-07-251-2/+2
| | | | | | | Remove opal prefix from opal_* structures in errorlog.h Signed-off-by: Deepthi Dharwar <deepthi@linux.vnet.ibm.com> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
* Initial commit of Open Source releaseBenjamin Herrenschmidt2014-07-021-0/+415
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
OpenPOWER on IntegriCloud