[coreboot] [PATCH] don't print too early on mcp55-based boards

Uwe Hermann uwe at hermann-uwe.de
Tue Nov 2 23:20:43 CET 2010

On Mon, Nov 01, 2010 at 11:01:41PM +0100, Peter Stuge wrote:
> Ward Vandewege wrote:
> > See attached. Perhaps we should also print a post code if the SMBus
> > controller can't be found - suggestions for a value?
> 0x5B ?
> > We can't print this early.
> > 
> > This patch fixes a hang on 
> > 
> >   supermicro/h8dme
> >   supermicro/h8dmr
> >   supermicro/h8dmr_fam10
> > 
> > and possibly on other mcp55-based boards.
> > 
> > Signed-off-by: Ward Vandewege <ward at gnu.org>
> Acked-by: Peter Stuge <peter at stuge.se>

I don't object to the patch, and we should probably fix this in all
other southbridges, I think the same problem applies there.

But: the die() call itself also does a printk(), so that'll still hang
if the error path is chosen (at that point it probably doesn't matter
much, though, as we die anyway).

I also agree that die() should have a POST code, preferrably something
easy to remember. It already has a commented-out "//post_code(0xff);".
Not sure why it's disabled, but I think it should be something other
than 0xff, that's a bit too "special" for my taste.

We have "0xee: Not supposed to get here" as per documentation/POSTCODES,
so maybe we can use 0xdd ("d" as in die), if that's not already used elsewhere.

http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org

More information about the coreboot mailing list