[coreboot] [PATCH] Drop all hardcoded ram_check() calls for now
Joseph Smith
joe at settoplinux.org
Thu Jan 28 21:52:33 CET 2010
On Thu, 28 Jan 2010 10:13:40 -0800, ron minnich <rminnich at gmail.com> wrote:
> On Thu, Jan 28, 2010 at 9:21 AM, Peter Stuge <peter at stuge.se> wrote:
>
>> In general I'm inclined to Ack, but my only reservation is that some
>> of the calls to ram_check() might point to particularly strategic
>> addresses for some boards, which were only known to the person who
>> did the port. If so, it would be nice to keep that information
>> somewhere..
>
>
> Which is my point. Where else would you put it? And, if it changes for
> some reason, you've now got to fix it in a document which people may
> not catch up with. That's a problem-generator.
>
Sorry Uwe, I am going to have to agree with Ron here. The ram_check() only
really needs to be used when developing or troubleshooting raminit. For a
regular build it is not needed, and in my opinion just complicates kconfig.
--
Thanks,
Joseph Smith
Set-Top-Linux
www.settoplinux.org
More information about the coreboot
mailing list