mbox series

[v2,0/6] iio:pressure:ms5637: add ms5803 support

Message ID 20210109231148.1168104-1-alexandre.belloni@bootlin.com
Headers show
Series iio:pressure:ms5637: add ms5803 support | expand

Message

Alexandre Belloni Jan. 9, 2021, 11:11 p.m. UTC
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(-)

Comments

Jonathan Cameron Jan. 16, 2021, 6:06 p.m. UTC | #1
On Sun, 10 Jan 2021 00:11:42 +0100
Alexandre Belloni <alexandre.belloni@bootlin.com> 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(-)

>