linux-kernel - Re: [PATCH v2 0/4] Support ROHM BD7910[0,1,2,3]

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250818191932.42c22df3@jic23-huawei>
Date: Mon, 18 Aug 2025 19:19:32 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: Matti Vaittinen <mazziesaccount@...il.com>
Cc: Matti Vaittinen <matti.vaittinen@...rohmeurope.com>, David Lechner
 <dlechner@...libre.com>, Nuno Sá <nuno.sa@...log.com>, Andy
 Shevchenko <andy@...nel.org>, Rob Herring <robh@...nel.org>, Krzysztof
 Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, David
 Heidelberg <david@...t.cz>, linux-iio@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, Sukrut Bellary
 <sbellary@...libre.com>, Lothar Rubusch <l.rubusch@...il.com>
Subject: Re: [PATCH v2 0/4] Support ROHM BD7910[0,1,2,3]

On Mon, 18 Aug 2025 11:11:56 +0300
Matti Vaittinen <mazziesaccount@...il.com> wrote:

> Add support for ROHM BD7910[0,1,2,3] ADCs.
> 
> The ROHM BD79100, BD79101, BD79102 and BD79103 are ADCs derived from the
> BD79104. According to the data-sheets, the BD79103 is compatible with the
> BD79104. Rest of the ICs have different number of analog input channels.
> 
> This series adds support for these ICs using the ti-adc128s052.c.
> 
> NOTE: There has been work on couple of other patch series [1][2] touching
> this same driver. I haven't considered those changes because, AFAICS,
> there has been no new revisions of these series since mid June.
> 
> [1]: https://lore.kernel.org/all/20250614091504.575685-1-sbellary@baylibre.com/
> [2]: https://lore.kernel.org/all/20250625170218.545654-2-l.rubusch@gmail.com/

Pretty simple replies to the changes requested in v1 reviews and nice and clean
so I'll gamble (a tiny bit) that everyone is happy with this and at least
queue it up for build testing.

Applied to the togreg branch of iio.git and pushed out as testing for
0-day to have it's fun.

Jonathan

> 
> Revision history:
>   v1 => v2:
>     dt-bindings:
>     - Fix the fallback compatible for BD79103.
>     - Drop the excess 'items'
>     other:
>     - Rename the channel structs as discussed during v1 review.
> 
> Matti Vaittinen (4):
>   dt-bindings: iio: adc: Add BD7910[0,1,2,3]
>   iio: adc: adc128s052: Simplify matching chip_data
>   iio: adc: adc128s052: Rename channel structs
>   iio: adc: adc128s052: Support ROHM BD7910[0,1,2,3]
> 
>  .../bindings/iio/adc/rohm,bd79104.yaml        |  10 +-
>  drivers/iio/adc/ti-adc128s052.c               | 132 ++++++++++++------
>  2 files changed, 95 insertions(+), 47 deletions(-)
> 
> 
> base-commit: 856d7be7f3c459a6d646b1f8432c6f616ade0d10