[LinuxBIOS] [patch 08/10] Geode LX: Northbridge
Peter Stuge
stuge-linuxbios at cdy.org
Fri May 4 03:25:17 CEST 2007
On Thu, May 03, 2007 at 12:15:56PM -0600, Marc Jones wrote:
> This patch adds support for the northbridge integrated into the AMD
> Geode LX platform, including memory and graphics.
Nak again due to whitespace although it is not as severe as in the
CPU patch.
> -int
> -sizeram(void)
> +int sizeram(void)
It would be nice to have a small comment on the algorithm used by
this function and a reference to some documentation.
> - printk_debug("sizeram: sizem 0x%x\n", sizem);
> + printk_debug("sizeram: sizem 0x%xMB\n", sizem);
I think %d would be nicer. :)
> + print_debug("\r\n");
Please use only \n for line endings.
//Peter
More information about the coreboot
mailing list