[v10,0/5] Expand Semtech SAR Sensors support

Message

Gwendal Grignou Jan. 1, 2022, 8:38 p.m. UTC

Add a new Semtech SAR sensor SX9324.
Instead of recopying 1/3 of the sx9310 driver, move common code in a new
file. It will be used again for the next sensor, SX9360.

Major changes in v10:
  Fix spelling in comment, unnecessary empty lines.
  Add symbol namespace.
Major changes in v9:
  Use simple pm macro
  Unify use of returning on default
  Add space before '}'
  Use sysfs_emit instead of printf.

Major changes in v8:
  Fix mismatch between documentation and code (setting vs setup)
  Use dev_get_drvdata to access iio device structure.

Major changes in v7:
  Fix one remaining syntax error in device tree binding documentation.

Major changes in v6:
  Fix syntax errors in device tree binding documentation.
  Drop of_match_ptr and ACPI_PTR protections.
  Fix unused variable warning.

Major changes in v5:
  Use iwyu to cleanup include files.
  Use dev_err_probe() in probe routine
  Add attribute to shift irq status register, not common among all
    sensors.
  Fix long line.

Major changes in v4:
  Use chip_info instead of info in common data.
  Returns an error when setting negative sysfs entries
  Fix cut and paste errors, credit.

Major changes in v3:
  Fix some error in binding descriptions and setting
  Fix invalid register constant name.

Major changes in v2:
  Better interface between common code and drivers
  Document SX9324 phase configuration

Gwendal Grignou (5):
  iio: sx9310: Add frequency in read_avail
  iio: sx9310: Extract common Semtech sensor logic
  iio: proximity: Add SX9324 support
  dt-bindings: iio: Add sx9324 binding
  iio: sx9324: Add dt_binding support

 .../ABI/testing/sysfs-bus-iio-sx9324          |   28 +
 .../iio/proximity/semtech,sx9324.yaml         |  161 +++
 drivers/iio/proximity/Kconfig                 |   20 +
 drivers/iio/proximity/Makefile                |    2 +
 drivers/iio/proximity/sx9310.c                |  741 ++----------
 drivers/iio/proximity/sx9324.c                | 1068 +++++++++++++++++
 drivers/iio/proximity/sx_common.c             |  572 +++++++++
 drivers/iio/proximity/sx_common.h             |  157 +++
 8 files changed, 2120 insertions(+), 629 deletions(-)
 create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-sx9324
 create mode 100644 Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml
 create mode 100644 drivers/iio/proximity/sx9324.c
 create mode 100644 drivers/iio/proximity/sx_common.c
 create mode 100644 drivers/iio/proximity/sx_common.h

Comments

Jonathan Cameron Jan. 9, 2022, 10:52 a.m. UTC | #1

On Sat,  1 Jan 2022 12:38:12 -0800
Gwendal Grignou <[e-mail]> wrote:

> Add a new Semtech SAR sensor SX9324.
> Instead of recopying 1/3 of the sx9310 driver, move common code in a new
> file. It will be used again for the next sensor, SX9360.
Series applied to the togreg branch of iio.git.
Until the merge window is done this will only be pushed out as testing
and I'll be rebasing on rc1 once available.

Thanks,

Jonathan

> 
> Major changes in v10:
>   Fix spelling in comment, unnecessary empty lines.
>   Add symbol namespace.
> Major changes in v9:
>   Use simple pm macro
>   Unify use of returning on default
>   Add space before '}'
>   Use sysfs_emit instead of printf.
> 
> Major changes in v8:
>   Fix mismatch between documentation and code (setting vs setup)
>   Use dev_get_drvdata to access iio device structure.
> 
> Major changes in v7:
>   Fix one remaining syntax error in device tree binding documentation.
> 
> Major changes in v6:
>   Fix syntax errors in device tree binding documentation.
>   Drop of_match_ptr and ACPI_PTR protections.
>   Fix unused variable warning.
> 
> Major changes in v5:
>   Use iwyu to cleanup include files.
>   Use dev_err_probe() in probe routine
>   Add attribute to shift irq status register, not common among all
>     sensors.
>   Fix long line.
> 
> Major changes in v4:
>   Use chip_info instead of info in common data.
>   Returns an error when setting negative sysfs entries
>   Fix cut and paste errors, credit.
> 
> Major changes in v3:
>   Fix some error in binding descriptions and setting
>   Fix invalid register constant name.
> 
> Major changes in v2:
>   Better interface between common code and drivers
>   Document SX9324 phase configuration
> 
> Gwendal Grignou (5):
>   iio: sx9310: Add frequency in read_avail
>   iio: sx9310: Extract common Semtech sensor logic
>   iio: proximity: Add SX9324 support
>   dt-bindings: iio: Add sx9324 binding
>   iio: sx9324: Add dt_binding support
> 
>  .../ABI/testing/sysfs-bus-iio-sx9324          |   28 +
>  .../iio/proximity/semtech,sx9324.yaml         |  161 +++
>  drivers/iio/proximity/Kconfig                 |   20 +
>  drivers/iio/proximity/Makefile                |    2 +
>  drivers/iio/proximity/sx9310.c                |  741 ++----------
>  drivers/iio/proximity/sx9324.c                | 1068 +++++++++++++++++
>  drivers/iio/proximity/sx_common.c             |  572 +++++++++
>  drivers/iio/proximity/sx_common.h             |  157 +++
>  8 files changed, 2120 insertions(+), 629 deletions(-)
>  create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-sx9324
>  create mode 100644 Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml
>  create mode 100644 drivers/iio/proximity/sx9324.c
>  create mode 100644 drivers/iio/proximity/sx_common.c
>  create mode 100644 drivers/iio/proximity/sx_common.h
>