Re: [PATCH v2 0/4] iio: light: tsl2591: fix lux calculation and precision, add I2C ID table, report saturation
From: Jan Remes
Date: Tue Oct 20 2026 - 07:39:50 EST
On Mon, 19 Oct 2026 11:50:10 +0300, Andy Shevchenko wrote:
>
> On Mon, Oct 19, 2026 at 10:17:08AM +0200, Jan Remes wrote:
> > On Sat, 17 Oct 2026 03:14:11 +0100, Jonathan Cameron wrote:
> > > On Thu, 15 Oct 2026 21:36:35 +0300
> > > Andy Shevchenko <andriy.shevchenko@xxxxxxxxx> wrote:
> > >
> > > > From my view (haven't checked the calculations, though) this looks good,
> > > > Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>
> > > >
> > > I'm guessing not huge urgency on patch 1, so to keep things simple I've
> > > applied the lot to the testing branch of iio.git. That will get rebased
> > > on rc1 once available in a couple of weeks.
> > >
> > > If you want the fix to go in faster, shout and I'll do that at risk
> > > of slowing down the rest.
> > >
> > FWIW, I don't think it to be very urgent, as people probably don't use
> > it on x86-64 (seeing both as it had only DT probe support and that
> > impacted users are likely to have noticed the oops).
>
> What do you mean by the second part of the sentence? It's as easy to use on
> ACPI platforms as on DT, we have so nice trick called PRP0001, which allows to
> enumerate "OF-only" drivers on ACPI platforms via SSDT overlays (or other means
> of DSDT/SSDT provisioning).
[ Sorry for the delay and potential malformed e-mail, I was trying to
interface mutt with my $DAYJOB Outlook e-mail until I got the idea to
just export this one e-mail into mbox. ]
I did not know about the PRP0001 (thanks), so please disregard that
argument. Still, I believe x86 users to be few if any, since the oops is
quite easy to trigger and AFAIK most x86 distro kernels ship without
CONFIG_OF.
Best regards,
Jan Remes