Re: [PATCH 2/5] power: supply: bd71828: Fix current direction

Andy Shevchenko <[email protected]>
Newsgroups org.kernel.vger.linux-pm,org.kernel.vger.linux-kernel
Organization Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo
Message-ID <[email protected]>
On Wed, Aug 12, 2026 at 10:10:50AM +0200, Andreas Kemnade wrote:
> On Mon, 10 Aug 2026 13:09:07 +0300
> Matti Vaittinen <[email protected]> wrote:

...

> That code is complicated. What about a generic 16bit non_2complemented sign
> to 2-complemented sign function/macro. Such would have prevented the error
> in the first places. So &vals[i] = signbit_to_2complement(be16_to_cpu(tmp_curr), 15) * pwr->curr_factor

Do you mean existing sign_extend32() from bitops.h?

-- 
With Best Regards,
Andy Shevchenko
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.