Re: [PATCH 0/2] iio: adc: ti-ads112c14: add burnout current support

Jonathan Cameron <[email protected]>
Newsgroups org.kernel.vger.linux-iio,org.kernel.vger.linux-kernel
Message-ID <20260823232018.6fad4bbd@jic23-huawei>
On Mon, 27 Jul 2026 03:08:22 +0100
Jonathan Cameron <[email protected]> wrote:

> On Fri, 24 Jul 2026 15:49:16 -0500
> "David Lechner (TI)" <[email protected]> wrote:
> 
> > The TI ADS112C14 ADC has a feature to enable a "burnout" current when
> > taking a measurement. This can be used to detect open/short circuit
> > conditions of a connected sensor. However, enabling this affects the
> > accuracy of the reading. So we add a custom "burnoutraw" attribute for
> > this. This works like the standard raw attribute to do a direct read
> > but does so with the burnout current enabled.
> > 
> > If this sounds familiar, it is because we discussed this on a similar
> > chip recently [1]. And this is why I put the ABI docs in a common ADC
> > file rather than one specific to this chip.
> > 
> > [1]: https://lore.kernel.org/linux-iio/20260622110223.7e854dde@jic23-huawei/
> > 
> > Signed-off-by: David Lechner (TI) <[email protected]>  
> 
> Seems fine to me, but given it is new ABI, I'd like some more eyes on the documentation
> patch in particular.

Seems no interest in this from others. I've no idea what this was on top
of so please rebase or resend alongside those other series.

All I know is it doesn't apply to my tree  - which isn't that surprising
given all the series around for this driver!

Jonathan

> 
> Thanks
> 
> Jonathan
> 
> > ---
> > David Lechner (TI) (2):
> >       iio: adc: ti-ads112c14: add burnout current support
> >       iio: ABI: add sysfs attribute for _burnoutraw
> > 
> >  Documentation/ABI/testing/sysfs-bus-iio-adc |   9 ++
> >  drivers/iio/adc/ti-ads112c14.c              | 126 ++++++++++++++++++++++++++--
> >  2 files changed, 126 insertions(+), 9 deletions(-)
> > ---
> > base-commit: 0b5e142ced4bcf20532da051934bd694d1bbd470
> > change-id: 20260724-iio-adc-ti-ads112c14-burnout-184304669165
> > prerequisite-change-id: 20260714-iio-adc-ti-ads112c14-buffered-read-41e5d1ca0dc3:v2
> > prerequisite-patch-id: 8723aa599dcf5b836fb81cf1efb6c9096bf60b7a
> > prerequisite-patch-id: 12754335ccf210bcdeaa8a5d60b2280618d50993
> > prerequisite-change-id: 20260724-iio-adc-ti-ads112c14-continuous-mode-dd1580754609:v1
> > prerequisite-patch-id: 8723aa599dcf5b836fb81cf1efb6c9096bf60b7a
> > prerequisite-patch-id: 12754335ccf210bcdeaa8a5d60b2280618d50993
> > prerequisite-patch-id: ed5d4c1628ef60ed542339fe5fb490383ee8074f
> > prerequisite-patch-id: b92f5609fcf39e958e78b4dd74c3d323ebb6a161
> > 
> > Best regards,
> > --  
> > David Lechner (TI) <[email protected]>
> >   
> 
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.