aboutsummaryrefslogtreecommitdiff
path: root/hdata
AgeCommit message (Collapse)AuthorFilesLines
2014-12-10Fix bug/warning in hdata/memory.c I introduced in f1a3a19Stewart Smith1-2/+3
Passing the wrong parameter to snprintf (sizeof(char*)) rather than actual length of the string. GCC 4.9 told us about it (and broke make check when I enabled -Werror) hdata/test/../memory.c: In function ‘add_address_range’: hdata/test/../memory.c:144:23: error: argument to ‘sizeof’ in ‘snprintf’ call is the same expression as the destination; did you mean to provide an explicit length? [-Werror=sizeof-pointer-memaccess] Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-12-05Fix warning of potential unbound stack usage in hdata/memory.cStewart Smith1-1/+5
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-12-05Silence warning for hdata/tests/stubs.c prlog missing prototypeStewart Smith1-1/+3
We get the real prototype in real code from skiboot.h Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-11-28Remove sprintf: there's no good reason to have this in firmwareStewart Smith1-1/+1
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-11-18Remove useless global include memory.hBenjamin Herrenschmidt4-3/+1
It only exposed one function that is local to the hdat stuff Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
2014-11-05hdat: Add Venice hub ID to HDAT conversionBenjamin Herrenschmidt1-5/+9
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
2014-10-15Reduce severity of hdata/fsp.c log messages (mostly to PR_DEBUG)Stewart Smith1-7/+8
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-10-15Rework hdata/iohub.c CEC log messages to appropriate log levelsStewart Smith1-54/+69
We're mostly PR_DEBUG here, except for the basics, which are PR_INFO. Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-10-15Increase verbosity of CHIPTOD to PR_DEBUGStewart Smith1-3/+3
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-10-15Reduce severity of XSCOM detection log message to PR_INFOStewart Smith1-1/+1
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-10-15bump last two hdata/memory.c MS VPD printouts to PR_DEBUGStewart Smith1-4/+5
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-10-15hdata/memory.c: only prlog with PR_DEBUG most thingsStewart Smith1-8/+8
We will PR_INFO the total amount of memory, as this is just informative. Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-10-15Only print SPIRA discovered serial port info at PR_INFO priorityStewart Smith1-3/+3
It's certainly not an error that we have a serial port, and not essential to display during boot. Odds are, you already know about the serial port. Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-10-15decrease verbosity of logging CPU information.Stewart Smith2-17/+18
If we have an UNAVAILABLE core (e.g. garded by FSP), we *WILL* print that out with PR_WARNING, as that's probably something that people want to pay attention to. Otherwise we stick with PR_INFO, which should be enough for these kind of log messages. Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-10-15Reduce the amount of text 'parsing hdat' uses.Stewart Smith1-9/+3
We can, by default, deal with just two lines rather than 5 of ASCII art Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
2014-08-15hdata/test: Fix broken testsJoel Stanley2-1/+5
Commit e810dcbc (ATTN: Set up attention area to handle attention) broke tests, as the familiy of CPU_TO_BEXX macros are not compile time constant. hdata/test/../spira.c:60:4: error: initializer element is not constant .addr = CPU_TO_BE64((unsigned long)&(cpu_ctl_spat_area) + SKIBOOT_BASE) There is no test coverage of this code, so for now we can comment out these areas in order to allow the tests to pass. Signed-off-by: Joel Stanley <joel@jms.id.au> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
2014-08-15{core,hdata}/test: Add prlog to stubJoel Stanley1-1/+12
We are missing a prlog for tests. This adds a dumb version that ignores the log level and uses printf to display all messages. Signed-off-by: Joel Stanley <joel@jms.id.au> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
2014-08-01ATTN: Set up attention area to handle attentionAruna Balakrishnaiah2-0/+68
At present CPU control area ntuple in SPIRA structure is NULL. ATTN component in Service Processor side checks for this field and if its empty, it logs hardcoded SRC (0xBB821410) and generates SYSDUMP. So we have 1 SRC for all failure (assert call) from OPAL side. This makes difficult to debug the issue. Service processor provides attention area interface (FIPS PHyp Attentions spec), so that we can pass SRC and user data (error message) to service processor. This will helps us identify different failures in OPAL. This patch enables attention area and provides interface (update_sp_attn_area()) to add src and user data (error message) through assert macro. Attention SRC format: 1st byte - Opal src type 2-4 bytes - Holds the address of the assert function call Signed-off-by: Aruna Balakrishnaiah <aruna@linux.vnet.ibm.com> Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
2014-07-31MDST: Rename function name and variable orderVasant Hegde1-2/+2
- Better function naming - Better macro naming - Re-ordered function variables - calculate max_mdst_entry after allocating memory for mdst table. No change in functionality. Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Acked-by: Stewart Smith <stewart@linux.vnet.ibm.com> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
2014-07-02Initial commit of Open Source releaseBenjamin Herrenschmidt19-0/+5657
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>