Re: [coreboot] [PATCH] CONFIG_LB_MEM_TOPK -> CONFIG_RAMTOP

[Myles Watson]

On Fri, Oct 16, 2009 at 10:12 AM, Peter Stuge  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.

Fixed.

Thanks,
Myles