Re: [PATCH v9 0/2] iio: accel: add support for ADXL355

[Puranjay Mohan]

Please Ignore this I will resend this properly.
I made some mistakes while sending the patches.
I am really sorry for the noise.

On Mon, Aug 9, 2021 at 1:15 PM Puranjay Mohan <puranjay12@xxxxxxxxx> wrote:
>
> Add the dt-bindings and the driver for ADXL355 3-axis MEMS Accelerometer.
>
> Changes since v8:
> 1. Make scale and offset defines inline and remove them.
> 2. Change dt-binding doc to state interrupt polarity only for DRDY pin.
> 3. Remove triggered buffer support from this patch series.
>
> Changes since v7:
> 1. Update MAINTAINERS to show all driver files.
> 2. Set CONFIGS for buffered support in Kconfig.
>
> Changes since v6:
> 1. Use interrupt-names property in device tree document.
> 2. Add triggered buffer support.
> 3. Use a static table for offset and data registers.
> 4. Fix coding style issues.
> 5. move defines from header to c file.
>
> Changes since v5:
> 1. Used get_unaligned_be24() and  get_unaligned_be16() to parse
> acceleration and temperature data. This solves sparse errors and also
> make the code more understandable.
>
> Changes since v4:
> 1. Fix errors reported by sparse.
>
> Changes since v3:
> 1. Fix errors in yaml DT doc.
> 2. Change SPDX-License-Identifier to GPL-2.0-only OR BSD-2-Clause
>
> Changes since v2:
> 1. Add separate DT binding doc in yaml.
> 2. Use ____cacheline_aligned buffer for regmap_bulk_read/write calls.
> 3. Make code consistent by using same style in switch case.
> 4. Use FIELD_PREP in place of custom macros.
> 5. Make Kconfig description more informative.
>
> Changes since v1:
> 1. Remove the declarations for static regmap structures from adxl355.h.
> This was missed in the v1 and caused errors.
> 2. Make switch case statements consistent by directly returning from
> each case rather than saving the return in a variable.
> 3. Some coding style changes.
>
> Changes since v0:
> 1. Move adxl355_hpf_3db_table to adxl355_data structure. This is done to make
> sure that each device gets its own table.
> 2. Make local regmap definitions private to adxl355_core.c.
> 3. Other minor coding style changes.
>
> Puranjay Mohan (2):
>   iio: accel: Add driver support for ADXL355
>   iio: accel: adxl355: Add triggered buffer support
>
>  MAINTAINERS                      |  10 +
>  drivers/iio/accel/Kconfig        |  33 ++
>  drivers/iio/accel/Makefile       |   3 +
>  drivers/iio/accel/adxl355.h      |  19 +
>  drivers/iio/accel/adxl355_core.c | 676 +++++++++++++++++++++++++++++++
>  drivers/iio/accel/adxl355_i2c.c  |  65 +++
>  drivers/iio/accel/adxl355_spi.c  |  67 +++
>  7 files changed, 873 insertions(+)
>  create mode 100644 drivers/iio/accel/adxl355.h
>  create mode 100644 drivers/iio/accel/adxl355_core.c
>  create mode 100644 drivers/iio/accel/adxl355_i2c.c
>  create mode 100644 drivers/iio/accel/adxl355_spi.c
>
> --
> 2.30.1
>


-- 
Thanks and Regards

Yours Truly,

Puranjay Mohan