[coreboot] [PATCH] Factor out ROM size calculation from Config.lb

Myles Watson mylesgw at gmail.com
Thu Apr 23 00:26:49 CEST 2009


On Wed, Apr 22, 2009 at 4:13 PM, Carl-Daniel Hailfinger
<c-d.hailfinger.devel.2006 at gmx.net> wrote:
> On 21.04.2009 05:37, Myles Watson wrote:
>>
>>> -----Original Message-----
>>> From: ron minnich [mailto:rminnich at gmail.com]
>>> On Mon, Apr 20, 2009 at 7:50 PM, ron minnich <rminnich at gmail.com> wrote:
>>>
>>>> the issue is that include files need an end statement. Long story. My
>>>>
>>> fault.
>>>
>>> that's not quite it either, sorry, it's been a long day.
>>>
>>
>> I think that this small patch fixes it.  It seems that | is special to the
>> tokenizer and needs to be [|], so "||" => "[|][|]".  There are no
>> occurrences of || that I can find in Config.lb files, which probably means
>> that this bug has stopped people from using it.  I made up a test case, but
>> nothing real.  I'd love for someone to think of a place they'd like to use
>> || in a Config.lb file and try it out.
>>
>> Signed-off-by: Myles Watson <mylesgw at gmail.com>
>>
>
> I can't verify the correctness of this, but it fixes a real issue.
> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>
>
>>>> +
>>>> +## This is needed to work around a parser bug.
>>>> +if USE_FALLBACK_IMAGE
>>>> +end
>>>> +
>>>>
>>
>> This snippet will also disappear from config/nofailovercalculation.lb with
>> an ack.
>>
>
> Could you tackle config/nofailovercalculation128.lb as well? It has the
> same workaround. Thanks.

Sure.  Rev 4187.

Thanks,
Myles




More information about the coreboot mailing list