Commit 2c48a7d6 authored by Michael Neuling's avatar Michael Neuling Committed by Benjamin Herrenschmidt

powerpc: Print decimal values in prom_init.c

Currently we look pretty stupid when printing out a bunch of things in
prom_init.c.  eg.

  Max number of cores passed to firmware: 0x0000000000000080

So I've change this to print in decimal:

  Max number of cores passed to firmware: 128 (NR_CPUS = 256)

This required adding a prom_print_dec() function and changing some
prom_printk() calls from %x to %lu.
Signed-off-by: default avatarMichael Neuling <mikey@neuling.org>
Signed-off-by: default avatarBenjamin Herrenschmidt <benh@kernel.crashing.org>
parent ceddee23
...@@ -311,6 +311,24 @@ static void __init prom_print_hex(unsigned long val) ...@@ -311,6 +311,24 @@ static void __init prom_print_hex(unsigned long val)
call_prom("write", 3, 1, _prom->stdout, buf, nibbles); call_prom("write", 3, 1, _prom->stdout, buf, nibbles);
} }
/* max number of decimal digits in an unsigned long */
#define UL_DIGITS 21
static void __init prom_print_dec(unsigned long val)
{
int i, size;
char buf[UL_DIGITS+1];
struct prom_t *_prom = &RELOC(prom);
for (i = UL_DIGITS-1; i >= 0; i--) {
buf[i] = (val % 10) + '0';
val = val/10;
if (val == 0)
break;
}
/* shift stuff down */
size = UL_DIGITS - i;
call_prom("write", 3, 1, _prom->stdout, buf+i, size);
}
static void __init prom_printf(const char *format, ...) static void __init prom_printf(const char *format, ...)
{ {
...@@ -350,6 +368,14 @@ static void __init prom_printf(const char *format, ...) ...@@ -350,6 +368,14 @@ static void __init prom_printf(const char *format, ...)
v = va_arg(args, unsigned long); v = va_arg(args, unsigned long);
prom_print_hex(v); prom_print_hex(v);
break; break;
case 'l':
++q;
if (*q == 'u') { /* '%lu' */
++q;
v = va_arg(args, unsigned long);
prom_print_dec(v);
}
break;
} }
} }
} }
...@@ -835,11 +861,11 @@ static int __init prom_count_smt_threads(void) ...@@ -835,11 +861,11 @@ static int __init prom_count_smt_threads(void)
if (plen == PROM_ERROR) if (plen == PROM_ERROR)
break; break;
plen >>= 2; plen >>= 2;
prom_debug("Found 0x%x smt threads per core\n", (unsigned long)plen); prom_debug("Found %lu smt threads per core\n", (unsigned long)plen);
/* Sanity check */ /* Sanity check */
if (plen < 1 || plen > 64) { if (plen < 1 || plen > 64) {
prom_printf("Threads per core 0x%x out of bounds, assuming 1\n", prom_printf("Threads per core %lu out of bounds, assuming 1\n",
(unsigned long)plen); (unsigned long)plen);
return 1; return 1;
} }
...@@ -869,12 +895,12 @@ static void __init prom_send_capabilities(void) ...@@ -869,12 +895,12 @@ static void __init prom_send_capabilities(void)
cores = (u32 *)PTRRELOC(&ibm_architecture_vec[IBM_ARCH_VEC_NRCORES_OFFSET]); cores = (u32 *)PTRRELOC(&ibm_architecture_vec[IBM_ARCH_VEC_NRCORES_OFFSET]);
if (*cores != NR_CPUS) { if (*cores != NR_CPUS) {
prom_printf("WARNING ! " prom_printf("WARNING ! "
"ibm_architecture_vec structure inconsistent: 0x%x !\n", "ibm_architecture_vec structure inconsistent: %lu!\n",
*cores); *cores);
} else { } else {
*cores = DIV_ROUND_UP(NR_CPUS, prom_count_smt_threads()); *cores = DIV_ROUND_UP(NR_CPUS, prom_count_smt_threads());
prom_printf("Max number of cores passed to firmware: 0x%x\n", prom_printf("Max number of cores passed to firmware: %lu (NR_CPUS = %lu)\n",
(unsigned long)*cores); *cores, NR_CPUS);
} }
/* try calling the ibm,client-architecture-support method */ /* try calling the ibm,client-architecture-support method */
...@@ -1482,7 +1508,7 @@ static void __init prom_hold_cpus(void) ...@@ -1482,7 +1508,7 @@ static void __init prom_hold_cpus(void)
reg = -1; reg = -1;
prom_getprop(node, "reg", &reg, sizeof(reg)); prom_getprop(node, "reg", &reg, sizeof(reg));
prom_debug("cpu hw idx = 0x%x\n", reg); prom_debug("cpu hw idx = %lu\n", reg);
/* Init the acknowledge var which will be reset by /* Init the acknowledge var which will be reset by
* the secondary cpu when it awakens from its OF * the secondary cpu when it awakens from its OF
...@@ -1492,7 +1518,7 @@ static void __init prom_hold_cpus(void) ...@@ -1492,7 +1518,7 @@ static void __init prom_hold_cpus(void)
if (reg != _prom->cpu) { if (reg != _prom->cpu) {
/* Primary Thread of non-boot cpu */ /* Primary Thread of non-boot cpu */
prom_printf("starting cpu hw idx %x... ", reg); prom_printf("starting cpu hw idx %lu... ", reg);
call_prom("start-cpu", 3, 0, node, call_prom("start-cpu", 3, 0, node,
secondary_hold, reg); secondary_hold, reg);
...@@ -1507,7 +1533,7 @@ static void __init prom_hold_cpus(void) ...@@ -1507,7 +1533,7 @@ static void __init prom_hold_cpus(void)
} }
#ifdef CONFIG_SMP #ifdef CONFIG_SMP
else else
prom_printf("boot cpu hw idx %x\n", reg); prom_printf("boot cpu hw idx %lu\n", reg);
#endif /* CONFIG_SMP */ #endif /* CONFIG_SMP */
} }
...@@ -2420,7 +2446,7 @@ static void __init prom_find_boot_cpu(void) ...@@ -2420,7 +2446,7 @@ static void __init prom_find_boot_cpu(void)
prom_getprop(cpu_pkg, "reg", &getprop_rval, sizeof(getprop_rval)); prom_getprop(cpu_pkg, "reg", &getprop_rval, sizeof(getprop_rval));
_prom->cpu = getprop_rval; _prom->cpu = getprop_rval;
prom_debug("Booting CPU hw index = 0x%x\n", _prom->cpu); prom_debug("Booting CPU hw index = %lu\n", _prom->cpu);
} }
static void __init prom_check_initrd(unsigned long r3, unsigned long r4) static void __init prom_check_initrd(unsigned long r3, unsigned long r4)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment