[coreboot] progress on lx

[ron minnich]

On Jan 22, 2008 10:46 AM, Peter Stuge <peter at stuge.se> wrote:

> Small thing; I would prefer something else than a comma after the
> type, so that it is separated from the real information. Maybe:
> id="pci:vendor,device";
>
> I think that would make the syntax much more clear.

Thats fine with me, it is not really the "standard" for the dts but I
think we'd be better off making it clear.
>
>
> > > Further, in the dtc makefile I would like to preprocess pci_ids.h
> > > to produce arrays of char * with a list of vendor and device
> > > names. Then the  dtc tool can do the following:
> > > check the path type (pci). Check that it has the proper arguments
> > > (device, path). Then check the vendor name and device name to
> > > make sure they are valid for that path type.
> >
> > I'm undecided about this one. It adds complexity, but the
> > complexity helps verification of the code.
>
> But it's buildtime complexity which must be better than runtime
> complexity, and definately better than runtime failure.
>

That's my hope. I'm happy to make the buildtime more complex, as long
as it is more work for US, not the USERS :-)

I think we're going to benefit from increased error checking in the dts.

thanks

ron