Myles Watson mylesgw at gmail.com
Fri Oct 16 18:20:55 CEST 2009

On Fri, Oct 16, 2009 at 10:12 AM, Peter Stuge <peter at stuge.se> wrote:

> Myles Watson wrote:
> > There was only one place where I had to do (RAMBASE >> 10)!

> Maybe change that one to / 1024 instead?
Do you think it's clearer to use /1024?  I just meant it was surprising to
multiply the value by 1024 and only have to put it back once.

In a few places CONFIG_RAMTOP is used without (), but none of them
> looked bad. Maybe also search for ",CONFIG_RAMTOP" and add a space
> there.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091016/9f48f117/attachment.html>

More information about the coreboot mailing list