[PATCH v2 0/4] Support ROHM BD7910[0,1,2,3]

Matti Vaittinen posted 4 patches 1 month, 2 weeks ago
.../bindings/iio/adc/rohm,bd79104.yaml        |  10 +-
drivers/iio/adc/ti-adc128s052.c               | 132 ++++++++++++------
2 files changed, 95 insertions(+), 47 deletions(-)
[PATCH v2 0/4] Support ROHM BD7910[0,1,2,3]
Posted by Matti Vaittinen 1 month, 2 weeks ago
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/

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
-- 
2.50.1

Re: [PATCH v2 0/4] Support ROHM BD7910[0,1,2,3]
Posted by Jonathan Cameron 1 month, 2 weeks ago
On Mon, 18 Aug 2025 11:11:56 +0300
Matti Vaittinen <mazziesaccount@gmail.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
Re: [PATCH v2 0/4] Support ROHM BD7910[0,1,2,3]
Posted by David Lechner 1 month, 2 weeks ago
On 8/18/25 1:19 PM, Jonathan Cameron wrote:
> On Mon, 18 Aug 2025 11:11:56 +0300
> Matti Vaittinen <mazziesaccount@gmail.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.
> 
Reviewed-by: David Lechner <dlechner@baylibre.com>
Re: [PATCH v2 0/4] Support ROHM BD7910[0,1,2,3]
Posted by Jonathan Cameron 1 month, 2 weeks ago
On Mon, 18 Aug 2025 13:46:12 -0500
David Lechner <dlechner@baylibre.com> wrote:

> On 8/18/25 1:19 PM, Jonathan Cameron wrote:
> > On Mon, 18 Aug 2025 11:11:56 +0300
> > Matti Vaittinen <mazziesaccount@gmail.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.
> >   
> Reviewed-by: David Lechner <dlechner@baylibre.com>
> 

Thanks. Added to the 3 you hadn't tagged already.