[PATCH v2 0/2] Xilinx XADC fixes

[O'Griofa, Conall]

[AMD Official Use Only - General]

Acked-by: O'Griofa, Conall <conall.ogriofa at amd.com>
Tested-by: O'Griofa, Conall <conall.ogriofa at amd.com>

Hi Jonathan,

Thanks for the patches, changes look good, I have tested on board and not seen any issues from my testing.

Cheers,
Conall.
> -----Original Message-----
> From: Jonathan Cameron <jic23 at kernel.org>
> Sent: Tuesday, October 10, 2023 5:19 PM
> To: Simek, Michal <michal.simek at amd.com>
> Cc: Robert Hancock <robert.hancock at calian.com>; O'Griofa, Conall
> <conall.ogriofa at amd.com>; Lars-Peter Clausen <lars at metafoo.de>; linux-
> iio at vger.kernel.org; linux-arm-kernel at lists.infradead.org
> Subject: Re: [PATCH v2 0/2] Xilinx XADC fixes
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> On Sun, 24 Sep 2023 17:32:10 +0100
> Jonathan Cameron <jic23 at kernel.org> wrote:
>
> > On Fri, 15 Sep 2023 08:52:49 +0200
> > Michal Simek <michal.simek at amd.com> wrote:
> >
> > > +Conall
> > >
> > > On 9/15/23 02:10, Robert Hancock wrote:
> > > > Fixes for a couple of issues in the Xilinx XADC driver: one where
> > > > preconfigured temperature/voltage thresholds were being clobbered
> > > > and potentially breaking overtemperature shutdown, and another for
> > > > inaccurate temperature readings on UltraScale family devices.
> > > >
> > > > Changed since v2: Updated to also remove disabling XADC alarm bits.
> > > >
> > > > Robert Hancock (2):
> > > >    iio: adc: xilinx-xadc: Don't clobber preset voltage/temperature
> > > >      thresholds
> > > >    iio: adc: xilinx-xadc: Correct temperature offset/scale for
> > > > UltraScale
> > > >
> > > >   drivers/iio/adc/xilinx-xadc-core.c | 39 +++++++++++-------------------
> > > >   drivers/iio/adc/xilinx-xadc.h      |  2 ++
> > > >   2 files changed, 16 insertions(+), 25 deletions(-)
> > > >
> > >
> > > Conall: Please test and review.
> >
> > I'm sitting on this one until I hear back.   No huge rush, but if you
> > can estimate when you'll get to this I know to leave you alone until
> > after that!
>
> I'll ask one more time, then probably just assume these are fine and apply.
>
> So Conall, have you had a chance to look at these?
>
> Thanks,
>
> Jonathan
>
> >
> > Jonathan
> >
> > >
> > > Thanks,
> > > Michal
> >