2021-01-09 23:18:29

by Alexandre Belloni

[permalink] [raw]
Subject: [PATCH v2 0/6] iio:pressure:ms5637: add ms5803 support

Hello,

This series adds support for the Measurement Specialities ms5803. It is
very similar to the ms5805 but has a different PROM layout (which I
suspect predates the ms5805 PROM layout). Also it supports less
frequency sampling options.

After a bit of preparatory work in the ms5637 driver and its common
library, mainly to handle the PROM layout and sample frequencies, adding
support is trivial.

Changes in v2:
- Dropped "iio:pressure:ms5637: switch to probe_new" to keep the i2c_device_id
table.
- Reorder trivial-devices.yaml

Alexandre Belloni (6):
dt-bindings: trivial-devices: reorder memsic devices
iio:pressure:ms5637: introduce hardware differentiation
iio:pressure:ms5637: limit available sample frequencies
iio:common:ms_sensors:ms_sensors_i2c: rework CRC calculation helper
iio:common:ms_sensors:ms_sensors_i2c: add support for alternative PROM
layout
iio:pressure:ms5637: add ms5803 support

.../devicetree/bindings/trivial-devices.yaml | 10 ++-
.../iio/common/ms_sensors/ms_sensors_i2c.c | 76 ++++++++++++++----
.../iio/common/ms_sensors/ms_sensors_i2c.h | 15 +++-
drivers/iio/pressure/ms5637.c | 77 +++++++++++++++----
4 files changed, 143 insertions(+), 35 deletions(-)

--
2.29.2


2021-01-16 18:09:01

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH v2 0/6] iio:pressure:ms5637: add ms5803 support

On Sun, 10 Jan 2021 00:11:42 +0100
Alexandre Belloni <[email protected]> wrote:

> Hello,
>
> This series adds support for the Measurement Specialities ms5803. It is
> very similar to the ms5805 but has a different PROM layout (which I
> suspect predates the ms5805 PROM layout). Also it supports less
> frequency sampling options.
>
> After a bit of preparatory work in the ms5637 driver and its common
> library, mainly to handle the PROM layout and sample frequencies, adding
> support is trivial.
>
> Changes in v2:
> - Dropped "iio:pressure:ms5637: switch to probe_new" to keep the i2c_device_id
> table.
> - Reorder trivial-devices.yaml
LGTM. Series applied to the togreg branch of iio.git and pushed out as testing
to see if the autobuilders can find anything we missed.

thanks,

Jonathan

>
> Alexandre Belloni (6):
> dt-bindings: trivial-devices: reorder memsic devices
> iio:pressure:ms5637: introduce hardware differentiation
> iio:pressure:ms5637: limit available sample frequencies
> iio:common:ms_sensors:ms_sensors_i2c: rework CRC calculation helper
> iio:common:ms_sensors:ms_sensors_i2c: add support for alternative PROM
> layout
> iio:pressure:ms5637: add ms5803 support
>
> .../devicetree/bindings/trivial-devices.yaml | 10 ++-
> .../iio/common/ms_sensors/ms_sensors_i2c.c | 76 ++++++++++++++----
> .../iio/common/ms_sensors/ms_sensors_i2c.h | 15 +++-
> drivers/iio/pressure/ms5637.c | 77 +++++++++++++++----
> 4 files changed, 143 insertions(+), 35 deletions(-)
>