Re: [PATCH] dt: Update I2C trivial devices list

[Ben Gamari <bgamari.foss-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1393 bytes --]

Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org> writes:

> On Tue, Dec 17, 2013 at 11:47:19AM -0500, Ben Gamari wrote:
>> Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> writes:
>> 
>> > On 12/16/2013 05:12 PM, Ben Gamari wrote:
>> >
>> >> diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
>> >
>> >>  This is a list of trivial i2c devices that have simple device tree
>> >>  bindings, consisting only of a compatible field, an address and
>> >> -possibly an interrupt line.
>> >> +possibly an interrupt line. The compatible field is used to lookup the
>> >> +modalias of the driver which will handle the device. The compatible
>> >> +string may begin with a manufacturer prefix (separated from the
>> >> +modalias by a comma) which will be stripped off during lookup.
>> >
>> > This part of the patch described Linux-specific behaviour, whereas DT
>> > bindings should be OS-agnostic.
>> >
>> I see your point but it would be nice to have language like this
>> somewhere. One of the biggest challenges in diving in to writing a
>> devicetree is figuring out details like this.
>
> What about putting this into a seperate paragraph prefixed with "Current
> Linux specific implementation:"?
>
This would address my concern quite nicely.

Cheers,

- Ben

[-- Attachment #2: Type: application/pgp-signature, Size: 489 bytes --]