[coreboot] [PATCH 2/7] Geode GX2 cleanup next round

[Nils]

Change the print_conf activation from >=BIOS_ERR to >=BIOS_DEBUG.

Signed-off-by: Nils Jacobs <njacobs8 at hetnet.nl>


Op donderdag 30 december 2010 02:44:12 schreef u:
> On Wed, Dec 29, 2010 at 4:46 PM, Nils <njacobs8 at hetnet.nl> wrote:
> > Op woensdag 29 december 2010 21:44:08 schreef u:
> >> On Wed, Dec 29, 2010 at 12:04 PM, Nils <njacobs8 at hetnet.nl> wrote:
> >> > Add a MSR printing function to northbridge.c like in the Geode LX
> >> > code.
> >> 
> >> It's surprising to have it included
> >> 
> >> +#if CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_ERR
> >> 
> >> but then have all the print statements be BIOS_DEBUG.
> >> 
> >> Thanks,
> >> Myles
> > 
> > You are right, i copied it from Geode LX and didn't notice it.
> > Should i make a patch to change it on GX2 and LX to BIOS_DEBUG ?
> 
> It's up to you.  I think there could be a better way to handle it, but
> there are many ways to change it.
> 
> Thanks,
> Myles

This patch is boot tested on my Wyse S50.
If one of the professionals comes up with a better patch i will be happy to 
test it. :)

Thanks, Nils.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 6221_msrprint2.patch
Type: text/x-patch
Size: 1555 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20101230/f8c5f28d/attachment.patch>