diff mbox series

[4/5] dt-bindings: iio: pressure: bmp085: Add BMP580 compatible string

Message ID 778a9d07ad1d88e036cc1ddd9c3fa53390b906a3.1671986815.git.ang.iglesiasg@gmail.com (mailing list archive)
State Changes Requested
Headers show
Series Add support for pressure sensor Bosch BMP580 | expand

Commit Message

Angel Iglesias Dec. 25, 2022, 4:59 p.m. UTC
Add bosch,bmp580 to compatible string for the new family of sensors.
This family includes the BMP580 and BMP581 sensors. The register map
in this family presents significant departures from previous generations.

Signed-off-by: Angel Iglesias <ang.iglesiasg@gmail.com>

Comments

Krzysztof Kozlowski Dec. 26, 2022, 12:05 p.m. UTC | #1
On 25/12/2022 17:59, Angel Iglesias wrote:
> Add bosch,bmp580 to compatible string for the new family of sensors.
> This family includes the BMP580 and BMP581 sensors. The register map
> in this family presents significant departures from previous generations.
> 
> Signed-off-by: Angel Iglesias <ang.iglesiasg@gmail.com>

I did not receive anything else - no cover letter, no rest of patches -
so difficult to judge whether driver is implemented the same way.

For this patch only, assuming it matches the driver:

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
Angel Iglesias Dec. 26, 2022, 12:43 p.m. UTC | #2
On Mon, 2022-12-26 at 13:05 +0100, Krzysztof Kozlowski wrote:
> On 25/12/2022 17:59, Angel Iglesias wrote:
> > Add bosch,bmp580 to compatible string for the new family of sensors.
> > This family includes the BMP580 and BMP581 sensors. The register map
> > in this family presents significant departures from previous generations.
> > 
> > Signed-off-by: Angel Iglesias <ang.iglesiasg@gmail.com>
> 
> I did not receive anything else - no cover letter, no rest of patches -
> so difficult to judge whether driver is implemented the same way.

Sorry about that. I'll need to send an updated version with a few corrections.
I'll be sure next time to send the complete patchset your way.

Thanks for your time!

> For this patch only, assuming it matches the driver:
> 
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> Best regards,
> Krzysztof
>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml b/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml
index 72cd2c2d3f17..f52c4794e21b 100644
--- a/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml
+++ b/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml
@@ -17,6 +17,7 @@  description: |
     https://www.bosch-sensortec.com/bst/products/all_products/bmp280
     https://www.bosch-sensortec.com/bst/products/all_products/bme280
     https://www.bosch-sensortec.com/bst/products/all_products/bmp380
+    https://www.bosch-sensortec.com/bst/products/all_products/bmp580
 
 properties:
   compatible:
@@ -26,6 +27,7 @@  properties:
       - bosch,bmp280
       - bosch,bme280
       - bosch,bmp380
+      - bosch,bmp580
 
   reg:
     maxItems: 1