linux-kernel - Re: [PATCH 0/3] ti,ads7950 device tree bindings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <f0755e7c-f7e8-d3b9-dc20-9edc47d5c171@kernel.org>
Date:   Sat, 21 Jan 2017 15:08:35 +0000
From:   Jonathan Cameron <jic23@...nel.org>
To:     David Lechner <david@...hnology.com>, devicetree@...r.kernel.org,
        linux-iio@...r.kernel.org
Cc:     Hartmut Knaack <knaack.h@....de>,
        Lars-Peter Clausen <lars@...afoo.de>,
        Peter Meerwald-Stadler <pmeerw@...erw.net>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/3] ti,ads7950 device tree bindings

On 15/01/17 23:44, David Lechner wrote:
> On 01/15/2017 07:58 AM, Jonathan Cameron wrote:
>> On 11/01/17 17:52, David Lechner wrote:
>>> This series adds device tree bindings for the TI ADS7950 family of A/DC chips.
>>> The series includes the bindings documentation and some fixes to the iio driver
>>> to make it work with the device tree bindings.
>>>
>>> FYI, the ads7950 driver has not made it into mainline yet, so no worries about
>>> breaking anyone with these changes.
>>>
>> And here's the bit I failed to read!
>>
>> As an extra point, could you confirm what /sys/bus/iio/iio\:deviceX/name for this
>> one reads?  I have a feeling this is another case of what Lars has been pointing
>> out in other drivers this morning.  That name should be the device part number..
>>
> 
> cat /sys/bus/iio/devices/iio\:device0/name
> ads7957
> 
> This is the part number for the specific chip I am using. So, I am guessing that it is correct unless it is supposed to be upper case or something like that.
> 
> 
Cool. Spot on.
> 
>> Thanks,
>>
>> Jonathan
>>> David Lechner (3):
>>>   DT/bindings: Add bindings for TI ADS7950 A/DC chips
>>>   iio: adc: ti-ads7950: Drop "ti-" prefix from module name
>>>   iio: adc: ti-ads7950: Change regulator matching string to "vref"
>>>
>>>  .../devicetree/bindings/iio/adc/ti-ads7950.txt     | 23 ++++++++++++++++
>>>  drivers/iio/adc/ti-ads7950.c                       | 32 +++++++++++-----------
>>>  2 files changed, 39 insertions(+), 16 deletions(-)
>>>  create mode 100644 Documentation/devicetree/bindings/iio/adc/ti-ads7950.txt
>>>
>>
>