[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAL_Jsq+aysVR9U2Wquwwv7cjjO7wO8EXBg1Ja82u6s5t1yeJUw@mail.gmail.com>
Date: Thu, 16 Nov 2017 14:36:01 -0600
From: Rob Herring <robh@...nel.org>
To: Pavel Machek <pavel@....cz>
Cc: Dan Murphy <dmurphy@...com>, Mark Rutland <mark.rutland@....com>,
Richard Purdie <rpurdie@...ys.net>,
Jacek Anaszewski <jacek.anaszewski@...il.com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Linux LED Subsystem <linux-leds@...r.kernel.org>
Subject: Re: dts: fun with chip names Re: [PATCH v3 1/2] dt: bindings:
lm3692x: Add bindings for lm3692x LED driver
On Thu, Nov 16, 2017 at 2:11 PM, Pavel Machek <pavel@....cz> wrote:
> Hi!
>
>> >> +Required properties:
>> >> + - compatible:
>> >> + "ti,lm3692x"
>> >
>> > Don't use wildcards in compatible strings.
>>
>> Do you mean to remove the x? How do we denote a family of parts
>> then?
>
> I guess you should specify the exact chip.
>
> Which will present interesting problem for me on Nokia N9/N950; in one
> case, compatible chip is produced by two companies, and it looks like
> some machines have one and some have the other; but we'd like to share
> the dts as user has no chance telling them apart (and it is not
> important, anyway).
>
> In second case, chip is refered as APDS990X and I don't know where to
> get more exact data.
There's always exceptions to rules. Just make the case for it. IIRC,
there was the same case for the BT chip.
Rob
Powered by blists - more mailing lists