mbox series

[v3,0/3] iio: light: add support for VEML6075 UVA and UVB light sensor

Message ID 20231110-veml6075-v3-0-6ee46775b422@gmail.com (mailing list archive)
Headers show
Series iio: light: add support for VEML6075 UVA and UVB light sensor | expand

Message

Javier Carrasco Nov. 27, 2023, 5:34 p.m. UTC
This series adds support for the Vishay VEML6075 ultraviolet sensor,
which offers UVA and UVB measurement channels and I2C communication.

The device bindings and a simple example are also provided.

This driver has been tested with a Gravity VEML6075 UV Sensor Module in
open air conditions.

Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
---
Changes in v3:
- veml6075.c: use MICRO instead of 1000000LL (add linux/units.h).
- veml6075.c: improve lock description.
- veml6075.c: remove unnecessary i2c_get_client_data() from the probe
  function.
- veml6075.c: remove shutdown function and do the bit update inline.
- veml6075.c: improve return path in read/write_raw().
- veml6075.c: remove -ENODEV return check for regulator_get_enable().
- Link to v2: https://lore.kernel.org/r/20231110-veml6075-v2-0-d04efbc8bf51@gmail.com

Changes in v2:
- General: swap patch order (bindings first).
- iio core: add uva and uvb modifiers.
- veml6075.c: use uva and uvb modifiers instead of extend_name
- veml6075.c: remove redundant information from the description.
- veml6075.c: inline device name.
- veml6075.c: use read_avail() for available attributes.
- veml6075.c: use guard(mutex) instead of lock/unlock().
- veml6075.c: use regulator_get_enable() without _optional.
- veml6075.c: register managed iio device and delete remove().
- veml6075.c: remove remaining debug messages.
- veml6075.c: error path cleanup (return type after val assignment).
- veml6075.c: remove zero from i2c_device_id.
- MAINTAINERS: fix bindings name.
- vishay,veml6075.yaml: remove vdd-supply description and mark property
  as true.
- Link to v1: https://lore.kernel.org/r/20231110-veml6075-v1-0-354b3245e14a@gmail.com

---
Javier Carrasco (3):
      iio: add modifiers for A and B ultraviolet light
      dt-bindings: iio: light: add support for Vishay VEML6075
      iio: light: add VEML6075 UVA and UVB light sensor driver

 Documentation/ABI/testing/sysfs-bus-iio            |   7 +-
 .../bindings/iio/light/vishay,veml6075.yaml        |  39 ++
 MAINTAINERS                                        |   6 +
 drivers/iio/industrialio-core.c                    |   2 +
 drivers/iio/light/Kconfig                          |  11 +
 drivers/iio/light/Makefile                         |   1 +
 drivers/iio/light/veml6075.c                       | 474 +++++++++++++++++++++
 include/uapi/linux/iio/types.h                     |   2 +
 tools/iio/iio_event_monitor.c                      |   2 +
 9 files changed, 542 insertions(+), 2 deletions(-)
---
base-commit: b85ea95d086471afb4ad062012a4d73cd328fa86
change-id: 20231110-veml6075-321522ceaca9

Best regards,

Comments

Jonathan Cameron Dec. 4, 2023, 1:59 p.m. UTC | #1
On Mon, 27 Nov 2023 18:34:27 +0100
Javier Carrasco <javier.carrasco.cruz@gmail.com> wrote:

> This series adds support for the Vishay VEML6075 ultraviolet sensor,
> which offers UVA and UVB measurement channels and I2C communication.
> 
> The device bindings and a simple example are also provided.
> 
> This driver has been tested with a Gravity VEML6075 UV Sensor Module in
> open air conditions.
> 
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
Hi Javier,

Applied to the togreg branch of iio.git and pushed out as testing for 0-day
to see if it can find anything we missed.

Thanks,

Jonathan