[coreboot] [LinuxBIOS] adding "--list-supported" feature to superiotool #91

[Robinson Tryon]

On Jan 12, 2008 8:57 PM, Uwe Hermann <uwe at hermann-uwe.de> wrote:
>
...
> > Actually, here's another question for you:
> > Why is all of the data for superiotool stored in arrays terminated
> > with EOT?  Can't we just use ARRAY_SIZE (or some similar construct) to
> > figure out the length of a given array before we iterate over it?
>
> Not all entries have the same length. Using the EOT method is nice, IMO.

Okay, cool.
Is it common for C programmers to terminate arrays like this?  (I
haven't written much C code, so I'm just wondering for my own
edification... :-)

>
>

Here's a new patch.  I believe that I addressed everything that Uwe
brought up, including adding an entry in the superiotool manpage.  Let
me know how it looks.
(I noticed that there are a few references to "LinuxBIOS" in the
manpage, etc... that need to be changed to "coreboot"; I'll do that in
a separate patch)

Signed-off-by: Robinson P. Tryon <bishop.robinson at gmail.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: superiotool-list-supported3.patch
Type: text/x-patch
Size: 9377 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20080113/30292a45/attachment.patch>