Thread (17 messages) 17 messages, 3 authors, 2021-02-09

RE: [PATCH 1/4] iio: adis: add helpers for locking

From: "Sa, Nuno" <Nuno.Sa@analog.com>
Date: 2021-01-25 10:25:41
Also in: linux-iio

From: Jonathan Cameron <jic23@kernel.org>
Sent: Sunday, January 24, 2021 2:30 PM
To: Sa, Nuno <Nuno.Sa@analog.com>
Cc: devicetree@vger.kernel.org; linux-iio@vger.kernel.org; Rob
Herring [off-list ref]; Peter Meerwald-Stadler
[off-list ref]; Lars-Peter Clausen [off-list ref];
Hennerich, Michael [off-list ref]; Ardelean,
Alexandru [off-list ref]
Subject: Re: [PATCH 1/4] iio: adis: add helpers for locking


On Thu, 21 Jan 2021 12:49:51 +0100
Nuno Sá [off-list ref] wrote:
quoted
Add some helpers to lock and unlock the device.

Signed-off-by: Nuno Sá <nuno.sa@analog.com>
Hi Nuno
Hi Jonathan,
For a patch like this I'd also expect to see it immediately used in
all relevant places in the driver. I don't want a mixture
going forwards of this vs direct access to the lock.
Fully agreed...
No need to separate that into two patches for such a simple case
just introduce this and put it to use.  There aren't that
many call sites anyway from a quick grep.
Ok, will do that in v2.

- Nuno Sá


Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help