mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jan Remes <ciakval@gmail.com>
To: Jonathan Cameron <jic23@kernel.org>
Cc: "Andy Shevchenko" <andriy.shevchenko@intel.com>,
	"David Lechner" <dlechner@baylibre.com>,
	"Nuno Sá" <nuno.sa@analog.com>,
	"Andy Shevchenko" <andy@kernel.org>,
	"Joe Sandom" <joe.g.sandom@gmail.com>,
	linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,
	martin.triska@mycroftmind.com,
	"Jan Remes" <jan.remes@mycroftmind.com>
Subject: Re: [PATCH v2 0/4] iio: light: tsl2591: fix lux calculation and precision, add I2C ID table, report saturation
Date: Mon, 19 Oct 2026 10:17:08 +0200	[thread overview]
Message-ID: <atXSBHbOPYYltHUW@mm-ntb-00805> (raw)
In-Reply-To: <20261017031411.05dee323@jic23-hlaptop>

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@intel.com> wrote:
>
> > From my view (haven't checked the calculations, though) this looks good,
> > Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>
> >
> 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).

And on Arm, the net result is getting false illumination reading of 0.0,
which isn't great, but isn't terrible either.

Thanks for applying the series to iio/testing.

Kind regards,
  Jan

  reply	other threads:[~2026-10-19  8:17 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-14 14:16 Jan Remes
2026-10-14 14:16 ` [PATCH v2 1/4] iio: light: tsl2591: don't divide by zero Jan Remes
2026-10-14 14:16 ` [PATCH v2 2/4] iio: light: tsl2591: add I2C device ID table Jan Remes
2026-10-14 14:16 ` [PATCH v2 3/4] iio: light: tsl2591: report saturation Jan Remes
2026-10-14 14:16 ` [PATCH v2 4/4] iio: light: tsl2591: improve lux calculation precision Jan Remes
2026-10-15 18:36 ` [PATCH v2 0/4] iio: light: tsl2591: fix lux calculation and precision, add I2C ID table, report saturation Andy Shevchenko
2026-10-17  2:14   ` Jonathan Cameron
2026-10-19  8:17     ` Jan Remes [this message]
2026-10-19  8:50       ` Andy Shevchenko
2026-10-20 11:31         ` Jan Remes
2026-10-21  8:00           ` Andy Shevchenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=atXSBHbOPYYltHUW@mm-ntb-00805 \
    --to=ciakval@gmail.com \
    --cc=andriy.shevchenko@intel.com \
    --cc=andy@kernel.org \
    --cc=dlechner@baylibre.com \
    --cc=jan.remes@mycroftmind.com \
    --cc=jic23@kernel.org \
    --cc=joe.g.sandom@gmail.com \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=martin.triska@mycroftmind.com \
    --cc=nuno.sa@analog.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®