From 324289829343bd11c2c0cc4cdbec80ce81bbf89d Mon Sep 17 00:00:00 2001 From: Stewart Smith Date: Mon, 13 Oct 2014 15:15:35 +1100 Subject: decrease verbosity of logging CPU information. 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 --- hdata/cpu-common.c | 12 ++++++------ hdata/pcia.c | 23 ++++++++++++----------- 2 files changed, 18 insertions(+), 17 deletions(-) (limited to 'hdata') diff --git a/hdata/cpu-common.c b/hdata/cpu-common.c index f08fa27..e1aa607 100644 --- a/hdata/cpu-common.c +++ b/hdata/cpu-common.c @@ -33,12 +33,12 @@ struct dt_node * add_core_common(struct dt_node *cpus, const uint8_t pa_features[] = { 6, 0, 0xf6, 0x3f, 0xc7, 0x00, 0x80, 0xc0 }; - printf(" Cache: I=%u D=%u/%u/%u/%u\n", - be32_to_cpu(cache->icache_size_kb), - be32_to_cpu(cache->l1_dcache_size_kb), - be32_to_cpu(cache->l2_dcache_size_kb), - be32_to_cpu(cache->l3_dcache_size_kb), - be32_to_cpu(cache->l35_dcache_size_kb)); + prlog(PR_INFO, " Cache: I=%u D=%u/%u/%u/%u\n", + be32_to_cpu(cache->icache_size_kb), + be32_to_cpu(cache->l1_dcache_size_kb), + be32_to_cpu(cache->l2_dcache_size_kb), + be32_to_cpu(cache->l3_dcache_size_kb), + be32_to_cpu(cache->l35_dcache_size_kb)); /* Use the boot CPU PVR to make up a CPU name in the device-tree * since the HDAT doesn't seem to tell.... diff --git a/hdata/pcia.c b/hdata/pcia.c index d128a31..29a81c1 100644 --- a/hdata/pcia.c +++ b/hdata/pcia.c @@ -124,11 +124,11 @@ static struct dt_node *add_core_node(struct dt_node *cpus, >> CPU_ID_NUM_SECONDARY_THREAD_SHIFT) + 1; assert(threads <= PCIA_MAX_THREADS); - printf("CORE[%i]: PIR=%i RES=%i %s %s(%u threads)\n", - pcia_index(pcia), t->pir, t->proc_int_line, - ve_flags & CPU_ID_PACA_RESERVED - ? "**RESERVED**" : cpu_state(ve_flags), - be32_to_cpu(t->pir) == boot_cpu->pir ? "[boot] " : "", threads); + prlog(PR_INFO, "CORE[%i]: PIR=%i RES=%i %s %s(%u threads)\n", + pcia_index(pcia), t->pir, t->proc_int_line, + ve_flags & CPU_ID_PACA_RESERVED + ? "**RESERVED**" : cpu_state(ve_flags), + be32_to_cpu(t->pir) == boot_cpu->pir ? "[boot] " : "", threads); timebase = HDIF_get_idata(pcia, SPPCIA_IDATA_TIMEBASE, &size); if (!timebase || size < sizeof(*timebase)) { @@ -198,7 +198,7 @@ bool pcia_parse(void) if (!pcia) return false; - printf("Got PCIA !\n"); + prlog(PR_INFO, "Got PCIA !\n"); got_pcia = true; @@ -229,11 +229,12 @@ bool pcia_parse(void) okay = false; } - printf("CORE[%i]: HW_PROC_ID=%i PROC_CHIP_ID=%i EC=0x%x %s\n", - pcia_index(pcia), be32_to_cpu(id->hw_proc_id), - be32_to_cpu(id->proc_chip_id), - be32_to_cpu(id->chip_ec_level), - okay ? "OK" : "UNAVAILABLE"); + prlog(okay ? PR_INFO : PR_WARNING, + "CORE[%i]: HW_PROC_ID=%i PROC_CHIP_ID=%i EC=0x%x %s\n", + pcia_index(pcia), be32_to_cpu(id->hw_proc_id), + be32_to_cpu(id->proc_chip_id), + be32_to_cpu(id->chip_ec_level), + okay ? "OK" : "UNAVAILABLE"); if (!add_core_node(cpus, pcia, id, okay)) break; -- cgit v1.1