[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20260205201407.7d6a6dfa@jic23-huawei>
Date: Thu, 5 Feb 2026 20:14:07 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Andy Shevchenko <andriy.shevchenko@...el.com>
Cc: Oleksij Rempel <o.rempel@...gutronix.de>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, stable@...r.kernel.org, kernel@...gutronix.de,
linux-kernel@...r.kernel.org, linux-iio@...r.kernel.org,
devicetree@...r.kernel.org, Andy Shevchenko <andy@...nel.org>, David
Lechner <dlechner@...libre.com>, Nuno Sá
<nuno.sa@...log.com>, David Jander <david@...tonic.nl>
Subject: Re: [PATCH v5 01/13] iio: dac: ds4424: reject -128 RAW value
On Wed, 4 Feb 2026 16:13:59 +0200
Andy Shevchenko <andriy.shevchenko@...el.com> wrote:
> On Wed, Feb 04, 2026 at 03:00:33PM +0100, Oleksij Rempel wrote:
> > The DS442x DAC uses sign-magnitude encoding, so -128 cannot be represented
> > in hardware (7-bit magnitude).
> >
> > Previously, passing -128 resulted in a truncated value that programmed
> > 0mA (magnitude 0) instead of the expected maximum negative current,
> > effectively failing silently.
> >
> > Reject -128 to avoid producing the wrong current.
>
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@...el.com>
>
> (as agreed to use S8_* limits for now)
>
Applied to the fixes-togreg branch of iio.git.
Note that has a weird base at the moment. I'll probably rebase
and do a pull request very early in next cycle.
We might have to wait a little while to pick up the rest of the
series as a result.
Thanks,
Jonathan
Powered by blists - more mailing lists