mbox series

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

Message ID 20210109231148.1168104-1-alexandre.belloni@bootlin.com (mailing list archive)
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(-)
>