[coreboot] [PATCH] flashrom patch easier board_pciid_enables parsing

[Stefan Reinauer]

On 14.01.2009 0:33 Uhr, Peter Stuge wrote:
> ron minnich wrote:
>   
>>>> Similarly to flashchips array, this patch intends to make the
>>>> table board_pciid_enables more readable.
>>>>         
>>> For certain versions of readable. What real problem does this solve?
>>>       
>> 1. Next time someone adds a new struct member, we avoid mistakes of
>> ordering of initializers
>> 2. we avoid mistakes in the first place.
>>
>> The .x = y stuff was added for a (good) reason, I think this is an
>> improvement.
>>
>> Acked-by: Ronald G. Minnich <rminnich at gmail.com>
>>     
>
> Sorry, but I really do not like these at all and I would much rather
> see the chip table change back. There seemed to be general agreement
> about that on IRC the other day as well.
>   
Why?


-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de  • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866