Re: [PATCH v2 0/4] iio: adc: ade9000: add support for ADE9078

[Jonathan Cameron]

On Wed, 5 Aug 2026 17:45:48 +0100
Jonathan Cameron <jic23@xxxxxxxxxx> wrote:

> On Wed, 5 Aug 2026 11:45:37 +0000
> "Miclaus, Antoniu" <Antoniu.Miclaus@xxxxxxxxxx> wrote:
> 
> > Hi Jonathan,
> >   
> > > -----Original Message-----
> > > From: Jonathan Cameron <jic23@xxxxxxxxxx>
> > > Sent: Sunday, August 2, 2026 4:44 AM
> > > To: Miclaus, Antoniu <Antoniu.Miclaus@xxxxxxxxxx>
> > > Cc: Sa, Nuno <Nuno.Sa@xxxxxxxxxx>; Hennerich, Michael
> > > <Michael.Hennerich@xxxxxxxxxx>; David Lechner <dlechner@xxxxxxxxxxxx>;
> > > Rob Herring <robh@xxxxxxxxxx>; Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>;
> > > Conor Dooley <conor+dt@xxxxxxxxxx>; Jonathan Corbet <corbet@xxxxxxx>;
> > > Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>; Sabau, Radu bogdan
> > > <Radu.Sabau@xxxxxxxxxx>; Schmitt, Marcelo
> > > <Marcelo.Schmitt@xxxxxxxxxx>; Marques, Jorge
> > > <Jorge.Marques@xxxxxxxxxx>; Salih Erim <salih.erim@xxxxxxx>; Matti
> > > Vaittinen <mazziesaccount@xxxxxxxxx>; Jishnu Prakash
> > > <jishnu.prakash@xxxxxxxxxxxxxxxx>; linux-iio@xxxxxxxxxxxxxxx; linux
> > > <linux@xxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx; linux-
> > > kernel@xxxxxxxxxxxxxxx; linux-doc@xxxxxxxxxxxxxxx
> > > Subject: Re: [PATCH v2 0/4] iio: adc: ade9000: add support for ADE9078
> > > 
> > > [External]
> > > 
> > > On Fri, 31 Jul 2026 11:29:46 +0300
> > > Antoniu Miclaus <antoniu.miclaus@xxxxxxxxxx> wrote:
> > >     
> > > > The ADE9078 is a polyphase energy metering device from the same family as
> > > > the ADE9000 and is register compatible with it. The main differences
> > > > relevant to the driver are the absence of the on-chip dip/swell detection,
> > > > a lower maximum SPI clock (10 MHz), and slightly different full-scale ADC
> > > > codes.    
> > > 
> > > This looks fine to me, but will need to sit on list until the dt maintainers
> > > have gotten to it.  Sashiko did call out some possible preexisting issues
> > > in the driver.  I haven't looked at them, but if you have some time, it may
> > > be worth a look the power vs interrupt registration one smells plausible.
> > >     
> > Want me to send the fixes in a separate patch or within this series?  
> 
> Separate should be fine for any preexisting issues.  I'm not going to
> hold this series up waiting for them (assuming we get the dt ack - it's
> listed on their patchwork as Needs Ack so no one has gotten to it yet).
> https://patchwork.kernel.org/project/devicetree/patch/20260731082951.1228-3-antoniu.miclaus@xxxxxxxxxx/

Applied this series to the testing branch of iio.git.  Expecting to see
patches dealing with any of the Sashiko found prexisting issues on top.

Thanks,

Jonathan

> 
> Jonathan
> 
> >   
> > > https://urldefense.com/v3/__https://sashiko.dev/*/patchset/20260731082
> > > 951.1228-1-
> > > antoniu.miclaus*40analog.com__;IyU!!A3Ni8CS0y2Y!78esTjrnDgvdH9h_Pruv
> > > IpxZPJggP6_ge3-ecPnYKxN8b32xrwH3oUABtFHCRmEGtnicycd5eqRTn7EkeQ$    
> > > >
> > > > This series adds support for the ADE9078 to the existing ade9000 driver.
> > > > It first refactors the driver to move the part-specific values (device
> > > > name, channel table, and full-scale codes) into a chip_info structure
> > > > retrieved via device match data, then adds the new device tree compatible
> > > > and the ADE9078 support itself, and finally updates the documentation.
> > > >
> > > > Changes in v2:
> > > > - move the "respective datasheets" comment wording to avoid churn.
> > > > - use a generic Kconfig title and list the supported parts.
> > > > - duplicate the RMS voltage macro instead of a shared base macro.
> > > >
> > > > v1: https://urldefense.com/v3/__https://lore.kernel.org/linux-    
> > > iio/__;!!A3Ni8CS0y2Y!78esTjrnDgvdH9h_PruvIpxZPJggP6_ge3-
> > > ecPnYKxN8b32xrwH3oUABtFHCRmEGtnicycd5eqTALKwajg$ <FILL-IN-v1-
> > > MESSAGE-ID>/    
> > > >
> > > > Antoniu Miclaus (4):
> > > >   iio: adc: ade9000: introduce chip_info structure
> > > >   dt-bindings: iio: adc: adi,ade9000: add adi,ade9078 compatible
> > > >   iio: adc: ade9000: add support for ADE9078
> > > >   docs: iio: ade9000: document ADE9078 support
> > > >
> > > >  .../bindings/iio/adc/adi,ade9000.yaml         |  30 ++-
> > > >  Documentation/iio/ade9000.rst                 |  28 +--
> > > >  drivers/iio/adc/Kconfig                       |  16 +-
> > > >  drivers/iio/adc/ade9000.c                     | 180 ++++++++++++------
> > > >  4 files changed, 175 insertions(+), 79 deletions(-)
> > > >
> > > >
> > > > base-commit: 0b5e142ced4bcf20532da051934bd694d1bbd470    
> >   
> 
>