| Message ID | 20250309170633.1347476-1-jic23@kernel.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | iio: light: Sparse friendly claim of direct mode | expand |
On 3/9/25 12:06 PM, Jonathan Cameron wrote: > From: Jonathan Cameron <Jonathan.Cameron@huawei.com> > > Note that a number of the drivers touched in this series have no > obvious active maintainer, so it would be much appreciated if anyone > has time to take a look! It is a large series so feel free to review > any you have time to look at rather than feeling you need to look > at the whole thing! > > This is effectively part 4 of what will probably be around 7 series > focused on moving from iio_device_claim/release_direct_mode() to > iio_device_claim/release_direct(). The new form is more consistent > with conditional locking semantics and sparse markings have been > added that let us detect miss-balance between claim and release. > > More details can be found in the cover letter of the first series: > https://lore.kernel.org/all/20250209180624.701140-1-jic23@kernel.org/ > > This series focuses on the light sensor drivers. > Made a couple of minor comments, but overall looks OK. Reviewed-by: David Lechner <dlechner@baylibre.com>
From: Jonathan Cameron <Jonathan.Cameron@huawei.com> Note that a number of the drivers touched in this series have no obvious active maintainer, so it would be much appreciated if anyone has time to take a look! It is a large series so feel free to review any you have time to look at rather than feeling you need to look at the whole thing! This is effectively part 4 of what will probably be around 7 series focused on moving from iio_device_claim/release_direct_mode() to iio_device_claim/release_direct(). The new form is more consistent with conditional locking semantics and sparse markings have been added that let us detect miss-balance between claim and release. More details can be found in the cover letter of the first series: https://lore.kernel.org/all/20250209180624.701140-1-jic23@kernel.org/ This series focuses on the light sensor drivers. Jonathan Cameron (18): iio: light: apds9306: Switch to sparse friendly iio_device_claim/release_direct() iio: light: gp2ap020a00f: Switch to sparse friendly iio_device_claim/release_direct() iio: light: isl29125: Switch to sparse friendly iio_device_claim/release_direct() iio: light: as73211: Use guard() and move mode switch into inner write_raw fucntion iio: light: as73211: Switch to sparse friendly iio_device_claim/release_direct() iio: light: ltr501: Factor out IIO_INFO_RAW leg of read_raw() callback. iio: light: ltr501: Factor out core of write_raw() where direct mode claim is held. iio: light: ltr501: Switch to sparse friendly iio_device_claim/release_direct() iio: light: opt4060: Switch to sparse friendly iio_device_claim/release_direct() iio: light: rohm-bu27034: Switch to sparse friendly iio_device_claim/release_direct() iio: light: rpr0521: Factor out handling of IIO_INFO_RAW and use guard() iio: light: rpr0521: Switch to sparse friendly iio_device_claim/release_direct() iio: light: si1145: Switch to sparse friendly iio_device_claim/release_direct() iio: light: st_uvis25: Switch to sparse friendly iio_device_claim/release_direct() iio: light: tcs3414: Switch to sparse friendly iio_device_claim/release_direct() iio: light: tcs3472: Switch to sparse friendly iio_device_claim/release_direct() iio: light: vcnl4000: Switch to sparse friendly iio_device_claim/release_direct() iio: light: vcnl4035: Switch to sparse friendly iio_device_claim/release_direct() drivers/iio/light/apds9306.c | 7 +- drivers/iio/light/as73211.c | 42 ++++---- drivers/iio/light/gp2ap020a00f.c | 7 +- drivers/iio/light/isl29125.c | 7 +- drivers/iio/light/ltr501.c | 164 +++++++++++++++-------------- drivers/iio/light/opt4060.c | 5 +- drivers/iio/light/rohm-bu27034.c | 14 ++- drivers/iio/light/rpr0521.c | 63 ++++++----- drivers/iio/light/si1145.c | 25 ++--- drivers/iio/light/st_uvis25_core.c | 7 +- drivers/iio/light/tcs3414.c | 9 +- drivers/iio/light/tcs3472.c | 9 +- drivers/iio/light/vcnl4000.c | 78 +++++++------- drivers/iio/light/vcnl4035.c | 42 +++++--- 14 files changed, 245 insertions(+), 234 deletions(-)