On Tue, Sep 03, 2024 at 08:34:22AM +0200, Krzysztof Kozlowski wrote: > On Mon, Sep 02, 2024 at 08:42:20PM +0200, Vasileios Amoiridis wrote: > > Add interrupt options for BMP3xx and BMP5xx devices as well. > > > > Signed-off-by: Vasileios Amoiridis <vassilisamir@xxxxxxxxx> > > --- > > .../bindings/iio/pressure/bmp085.yaml | 22 ++++++++++++++++++- > > 1 file changed, 21 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml b/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml > > index 6fda887ee9d4..20b75865e02f 100644 > > --- a/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml > > +++ b/Documentation/devicetree/bindings/iio/pressure/bmp085.yaml > > @@ -48,14 +48,34 @@ properties: > > > > interrupts: > > description: > > - interrupt mapping for IRQ (BMP085 only) > > + interrupt mapping for IRQ. Supported in BMP085, BMP3xx, BMP5xx > > maxItems: 1 > > > > + drive-open-drain: > > + description: > > + set if the interrupt pin should be configured as open drain. > > + If not set, defaults to push-pull configuration. > > + type: boolean > > + > > + > > Just one blank liine. > Hi Krzysztof, Thanks for the review. I can change that. > > required: > > - compatible > > - vddd-supply > > - vdda-supply > > > > +allOf: > > + - if: > > + properties: > > + compatible: > > + not: > > + enum: > > + - bosch,bmp085 > > + - bosch,bmp380 > > + - bosch,bmp580 > > Are you sure you tested this patch? > Well, before I run only make dt_binding_check but with the option make dtbs_check I now found what you meant. I think I miss a "contains" here. In any case, make dtbs_check showed some other warnings as well which I will address in a follow-up series. > Best regards, > Krzysztof > Cheers, Vasilis