Hi, On Fri, Dec 9, 2016 at 7:01 AM, Rob Herring <robh@xxxxxxxxxx> wrote: > On Fri, Dec 9, 2016 at 8:36 AM, Jiri Kosina <jikos@xxxxxxxxxx> wrote: >> On Thu, 8 Dec 2016, Rob Herring wrote: >> >>> > And if tomorrow there is Elan device that is drop-in compatible (same >>> > connector, etc) with Wacom i2c-hid, will you ask for Elan-specific >>> > binding? Atmel? Weida? They all need to be powered up ultimately. >>> >>> Yes, I will. >> >> What advantage does that bring? >> >>> That in no way means the OS driver has to know about each and every one. >>> If they can all claim compatibility with Wacom (including power >>> control), then they can have a Wacom compatible string too. Or you can >>> just never tell me that there's a different manufacturer and I won't >>> care as long you don't need different control. But soon as a device >>> needs another power rail, GPIO or different timing, then you'd better >>> have a new compatible string. >> >> Again, I simply don't understand what advantage does the aproach you are >> trying to use bring. > > This is simply how DT works. HID-over-I2C devices are no different > than any other I2C device or any other component. You are not special. ...but once you say that it's HID over I2C then it becomes probe-able, right? Said another way: we need to specify just enough to power the device up properly and then we can ask it what kind of device it is and then we can make quirk decisions based on that. So specifying what kind of device it is in the device tree is somewhat redundant and it also means that you make it needlessly difficult to build a system with dual-sourced components. One of the major points of probe-able connections is that you could put anything you want there and the kernel _doesn't_ need to describe it. ...and the whole point of device tree (I thought) was to specifically handle connections that _aren't_ probe-able. For instance, if a board has a USB bus on it but you need to assert a special reset or turn on a special regulator (besides vbus) before you can probe the USB bus, you wouldn't think that the board should specify exactly what device was stuffed in the connection, would you? >> HID over I2C is a generic protocol. > > DT describes h/w, not protocols. Isn't it a little of each, though? When you say that there's a USB port or an SDIO port or a serial port on the board, you're saying more than just that there are 2, 4, or 8 wires coming out of the board. You're saying that you're expecting to talk a certain protocol over those wires. -Doug -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html