Re: [coreboot] New patch to review for coreboot: c3f4e05 Google/snow: romstage that turns on memory and loads a ram stage with cbfs

[Bernhard Urban]

I can't log into gerrit at the moment, so I reply here.

On Thu, Jan 31, 2013 at 1:08 AM, Ronald G. Minnich <ger...@coreboot.org> wrote:
> +       /* wow, did it work? */
> +       int i;
> +       u32 *c = (void *)0x40000000;

what is at 0x40000000? what is "c"?
Please use a define or a more descriptive name for the pointer.


Thanks,
Bernhard

-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot