[coreboot] [PATCH] drop CONFIG_USE_PRINTK_IN_CAR and CONFIG_USE_INIT

Peter Stuge peter at stuge.se
Mon Aug 2 14:59:51 CEST 2010


Stefan Reinauer wrote:
> Drop the USE_PRINTK_IN_CAR option. It's a bogus decision to make for any user /
> board porter: printk should always be available in CAR mode.
> 
> Also drop CONFIG_USE_INIT, it's only been selected on one ASROCK board but it's
> not been used there. Very odd.
> 
> There is one usage of CONFIG_USE_INIT which was always off in 
> src/cpu/intel/car/cache_as_ram.inc and we have to figure out what to do with
> those few lines.
> 
> Signed-off-by: Stefan Reinauer <stepan at coresystems.de>

With a fix as described below it's

Acked-by: Peter Stuge <peter at stuge.se>


> +++ src/mainboard/amd/serengeti_cheetah/ap_romstage.c	(working copy)
..
> @@ -65,11 +63,7 @@
>  
>  	id = get_node_core_id_x();
>  
> -#if CONFIG_USE_PRINTK_IN_CAR
>          printk(BIOS_DEBUG, "CODE IN CACHE ON NODE: %02x\n");
> -#else
> -        print_debug("CODE IN CACHE ON NODE:"); print_debug_hex8(id.nodeid); print_debug("\n");
> -#endif

The remaining printk() call was+is missing the id.nodeid parameter.


//Peter




More information about the coreboot mailing list