On 07/27/2016 12:24 PM, Jason Gunthorpe wrote: > On Wed, Jul 27, 2016 at 10:30:09AM -0400, Dave Heller wrote: >>> So you might do >>> >>> compatible = "nuvoton,npct650", "nuvoton,npct601" >>> >>> Andrew, is 601 even the right name? >> >> Hi Jason, I'm not sure I'm understanding your answer here. Looks like >> we will have to add a couple of lines to specifically list the npct650, >> why not just using the existing syntax, one device id per line? Or you >> are saying, since the npct601 and npct650 are similar models, it's >> better to list them on the same line? > > The above example would go in your device tree, are you using device tree? > > Jason
Thanks, I get in now: you are saying we should add that to our device tree and leave the driver code alone. Here is were my understanding of the device tree entry is a bit lacking" maybe this is the convention that is normally followed. And yes, the device tree property is named "compatible". But this seems intuitively backward to me: Should not the device tree tell you what is actually *there* and let the driver code decide what is compatible? Rather than the firmware trying to make a guess as to which Nuvoton device id to try to match in the Nuvoton driver code? Put another way, should the "smarts" as to which Nuvoton part numbers are considered compatible to the driver, be done by FW? Or left to the driver? ------------------------------------------------------------------------------ What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic patterns at an interface-level. Reveals which users, apps, and protocols are consuming the most bandwidth. Provides multi-vendor support for NetFlow, J-Flow, sFlow and other flows. Make informed decisions using capacity planning reports.http://sdm.link/zohodev2dev _______________________________________________ tpmdd-devel mailing list tpmdd-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/tpmdd-devel
