[lkml]   [2017]   [Nov]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: dts: fun with chip names Re: [PATCH v3 1/2] dt: bindings: lm3692x: Add bindings for lm3692x LED driver
On 11/16/2017 02:36 PM, Rob Herring wrote:
> On Thu, Nov 16, 2017 at 2:11 PM, Pavel Machek <> 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.

Not sure how I got looped into the dts fun. But I am going to call out the specific chips out in the
compatible strings. There are currently only 2 devices supported by this driver. The delta
between the chips is one supports a third string of LEDs. 1 bit in 1 register.


> Rob

Dan Murphy
 \ /
  Last update: 2017-11-17 17:18    [W:0.080 / U:0.792 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site