mbox series

[0/7] iio: light: veml6070: update code to current IIO best practices

Message ID 20240929-veml6070-cleanup-v1-0-a9350341a646@gmail.com (mailing list archive)
Headers show
Series iio: light: veml6070: update code to current IIO best practices | expand

Message

Javier Carrasco Sept. 29, 2024, 8:38 p.m. UTC 
This series updates the driver in preparation to add new features. The
cleanup consists of:

1. Device-managed registering for:
- iio device
- action (unregister i2c device)
- regulator

2. Code update to use a guard for the mutex handling and
   dev_err_probe in the probe function.

3. Devicetree support (document the device bindings and register the
compatible in the driver).

Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
---
Javier Carrasco (7):
      iio: light: veml6070: add action for i2c_unregister_device
      iio: light: veml6070: use guard to handle mutex
      iio: light: veml6070: use device managed iio_device_register
      iio: light: veml6070: add support for a regulator
      dt-bindings: iio: light: vishay,veml6075: add vishay,veml6070
      iio: light: veml6070: add devicetree support
      iio: light: veml6070: use dev_err_probe in probe function

 .../bindings/iio/light/vishay,veml6075.yaml        |  3 +-
 drivers/iio/light/veml6070.c                       | 63 +++++++++++-----------
 2 files changed, 35 insertions(+), 31 deletions(-)
---
base-commit: 4057951fb272efda718dca665f6607c348d5785b
change-id: 20240929-veml6070-cleanup-70b609bb1f0f

Best regards,

Comments

Jonathan Cameron Sept. 30, 2024, 9 a.m. UTC | #1 
On Sun, 29 Sep 2024 22:38:45 +0200
Javier Carrasco <javier.carrasco.cruz@gmail.com> wrote:

> This series updates the driver in preparation to add new features. The
> cleanup consists of:
> 
> 1. Device-managed registering for:
> - iio device
> - action (unregister i2c device)
> - regulator
> 
> 2. Code update to use a guard for the mutex handling and
>    dev_err_probe in the probe function.
> 
> 3. Devicetree support (document the device bindings and register the
> compatible in the driver).
> 
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
Nice.  Applied to the togreg branch of iio.git and pushed out as testing.

thanks,

Jonathan

> ---
> Javier Carrasco (7):
>       iio: light: veml6070: add action for i2c_unregister_device
>       iio: light: veml6070: use guard to handle mutex
>       iio: light: veml6070: use device managed iio_device_register
>       iio: light: veml6070: add support for a regulator
>       dt-bindings: iio: light: vishay,veml6075: add vishay,veml6070
>       iio: light: veml6070: add devicetree support
>       iio: light: veml6070: use dev_err_probe in probe function
> 
>  .../bindings/iio/light/vishay,veml6075.yaml        |  3 +-
>  drivers/iio/light/veml6070.c                       | 63 +++++++++++-----------
>  2 files changed, 35 insertions(+), 31 deletions(-)
> ---
> base-commit: 4057951fb272efda718dca665f6607c348d5785b
> change-id: 20240929-veml6070-cleanup-70b609bb1f0f
> 
> Best regards,