On Jan 22, 2008 10:46 AM, Peter Stuge peter@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