[coreboot] [commit] r6149 - in trunk/src: mainboard/advantech/pcm-5820 mainboard/amd/db800 mainboard/amd/dbm690t mainboard/amd/mahogany mainboard/amd/mahogany_fam10 mainboard/amd/norwich mainboard/amd/pistach...
Stefan Reinauer
stefan.reinauer at coresystems.de
Wed Dec 8 19:00:08 CET 2010
On 08.12.2010, at 03:17, Patrick Georgi <patrick at georgi-clan.de> wrote:
> Am 08.12.2010 07:20, schrieb Stefan Reinauer:
>> On 12/7/10 10:00 PM, Peter Stuge wrote:
>>> repository service wrote:
>>>> first round name simplification. drop the <component>_ prefix.
>>> Beautiful.
>>>
>>>> +++ trunk/src/mainboard/advantech/pcm-5820/romstage.c Wed Dec 8 06:42:47 2010 (r6149)
>>>> @@ -27,7 +27,7 @@
>>>> #include "northbridge/amd/gx1/raminit.c"
>>>> #include "cpu/x86/bist.h"
>>>> #include "superio/winbond/w83977f/w83977f_early_serial.c"
>>> Maybe the early_* files should be renamed to romstage as well?
>>>
>>>
>>> //Peter
>>>
>> I like the idea... Will finish the northbridge and superios first, and
>> let someone else do the early->romstage deed :)
> Sounds good. Also, the bootblock.c files usually include another .c file
> for now (so we are able to use the same southbridge code with or without
> bootblock). Once a southbridge is entirely tinybootblocked, that could
> be inlined (and the early_rom stuff dropped).
Great! That would also automatically prevent people from calling the rom enable code again in romstage by accident.
Stefan
More information about the coreboot
mailing list