[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aLbPx8yPP_EgzTmW@smile.fi.intel.com>
Date: Tue, 2 Sep 2025 14:06:47 +0300
From: Andy Shevchenko <andriy.shevchenko@...el.com>
To: Jonathan Cameron <jic23@...nel.org>
Cc: Krzysztof Kozlowski <krzk@...nel.org>,
Lakshay Piplani <lakshay.piplani@....com>,
linux-kernel@...r.kernel.org, linux-iio@...r.kernel.org,
dlechner@...libre.com, nuno.sa@...log.com, andy@...nel.org,
marcelo.schmitt1@...il.com, gregkh@...uxfoundation.org,
viro@...iv.linux.org.uk, peterz@...radead.org,
jstephan@...libre.com, robh@...nel.org, krzk+dt@...nel.org,
conor+dt@...nel.org, devicetree@...r.kernel.org,
ilpo.jarvinen@...ux.intel.com, jonathan.cameron@...wei.com,
akpm@...ux-foundation.org, chao@...nel.org, jaegeuk@...nel.org,
vikash.bansal@....com, priyanka.jain@....com,
shashank.rebbapragada@....com, Frank.Li@....com
Subject: Re: [PATCH v2 2/2] iio: temperature: Add driver for NXP P3T175x
temperature sensor
On Mon, Sep 01, 2025 at 04:47:17PM +0100, Jonathan Cameron wrote:
> On Sun, 31 Aug 2025 18:46:32 +0200
> Krzysztof Kozlowski <krzk@...nel.org> wrote:
...
> This device does have a novel definition of register. There
> are 4 of them, 3 of which are 12 bits zero padded to 16 and
> the other 8 bits.
>
> So, I think only way to wrap that up fully in regmap would be
> a pair of regmaps one of which has only a single register in it.
>
> Agreed though that using bulk accesses is not a good plan.
> I'd been assuming that was actually a pair of registers, not
> a single larger one.
Why is it a problem? We have PMICs which have 16- and 8-bit registers and we do
use the bulk transfers. Yes, we have also an exception when it's required to
use byte transfers to make device work, but otherwise it works fine.
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists