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

Peter Stuge peter at stuge.se
Thu Nov 4 08:46:31 CET 2010


Ward Vandewege wrote:
> I'm not sure adding a post code to 'die' is a good idea. The
> problem with doing that is that it would clobber any previous post
> codes, which might be a better indicator for what's going wrong.

I think it is important to have a special POST code for use in die()
to positively identify that that is really where the code has hung.


Stefan Reinauer wrote:
> We could add a post code to the parameters of the die() function.

I think this is a good idea, and I would like to suggest that die()
does the following:

void die(u8 post) {
  while(1) {
    post_code(0xdd);
    mdelay(700);
    post_code(post);
    mdelay(700);
  }
}

There is no printk() which is sad. I think it's better to only use
POST codes if that is the only reliable output method, but I think
it's a lot better if the print functions can just be made to not
hang if called before they can actually output something. That would
need a bit more work though, so for now I'd like to suggest the
above.


//Peter




More information about the coreboot mailing list