[coreboot] [PATCH] flashrom - board enable - reconstruct table.
Peter Stuge
peter at stuge.se
Mon Apr 20 14:45:14 CEST 2009
Luc Verhaegen wrote:
> This patch restores the pciid based board matching table. It makes this
> table readable and hackable again, and the only disadvantage is that the
> right margin is way beyond the rather dogmatic 80. All 0x0000 pci ids have
> been string replaced by 0 to more easily spot missing ids, and extra
> comments have been added to explain how the various entries are used.
>
> Signed-Off-By: Luc Verhaegen <libv at skynet.be>
Thanks! r4142.
Carl-Daniel Hailfinger wrote:
> The documentation part can be committed after one more iteration.
I took the liberty of improving the wording a little.
> The rest of the patch makes board entries unreadable.
I disagree very much, and I welcome this patch because I naked the
original change.
> In the past few weeks, we had quite a few people in #coreboot who
> wanted to add support for their boards. The presence of struct
> member names helped immensely to explain how to add board enables.
Yes, and I added most of them after quickly looking over lspci -vnn
output to find the most suitable PCI ids.
> Lowering the barrier for possible future developers is very
> important.
A table is a table. Those who understand the concept still understand
it whether the table is really long or really wide.
However, a high degree of noise, such as many redundant member names,
really makes it more difficult to process the signal, such as
different hex PCI ids.
> feel free to submit any new board enables in the single-line
> layout. I'll fix them up.
While a very noble offer, it's not so practical for each change to
have to pass you.
Thanks Luc.
//Peter
More information about the coreboot
mailing list