diff mbox series

[1/3] dt-bindings: iio: st,lsm6dsx: reference common schema for mount-matrix

Message ID 20230507173923.263741-1-krzysztof.kozlowski@linaro.org (mailing list archive)
State Accepted
Headers show
Series [1/3] dt-bindings: iio: st,lsm6dsx: reference common schema for mount-matrix | expand

Commit Message

Krzysztof Kozlowski May 7, 2023, 5:39 p.m. UTC
Reference iio.yaml schema from dtschema to allow already used
mount-matrix property:

  msm8953-xiaomi-daisy.dtb: imu@6a: Unevaluated properties are not allowed ('mount-matrix' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Conor Dooley May 7, 2023, 9:06 p.m. UTC | #1
On Sun, May 07, 2023 at 07:39:21PM +0200, Krzysztof Kozlowski wrote:
> Reference iio.yaml schema from dtschema to allow already used
> mount-matrix property:
> 
>   msm8953-xiaomi-daisy.dtb: imu@6a: Unevaluated properties are not allowed ('mount-matrix' was unexpected)
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Conor Dooley <conor.dooley@microchip.com>

Thanks,
Conor.
Krzysztof Kozlowski May 13, 2023, 6:10 p.m. UTC | #2
On 13/05/2023 20:13, Jonathan Cameron wrote:
> On Sun, 7 May 2023 22:06:56 +0100
> Conor Dooley <conor@kernel.org> wrote:
> 
>> On Sun, May 07, 2023 at 07:39:21PM +0200, Krzysztof Kozlowski wrote:
>>> Reference iio.yaml schema from dtschema to allow already used
>>> mount-matrix property:
>>>
>>>   msm8953-xiaomi-daisy.dtb: imu@6a: Unevaluated properties are not allowed ('mount-matrix' was unexpected)
>>>
>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>  
>>
>> Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
>>
>> Thanks,
>> Conor.
> 
> Krzysztof,
> 
> These look fine to me, but I guess I should wait for the
> dt-schema patch to land before taking them?

You can grab first patch independently. #2 and #3 should indeed wait a
bit for Rob's ack. They do not have any conflicts but they will fix
nothing without dtschema patch.

Best regards,
Krzysztof
Jonathan Cameron May 13, 2023, 6:13 p.m. UTC | #3
On Sun, 7 May 2023 22:06:56 +0100
Conor Dooley <conor@kernel.org> wrote:

> On Sun, May 07, 2023 at 07:39:21PM +0200, Krzysztof Kozlowski wrote:
> > Reference iio.yaml schema from dtschema to allow already used
> > mount-matrix property:
> > 
> >   msm8953-xiaomi-daisy.dtb: imu@6a: Unevaluated properties are not allowed ('mount-matrix' was unexpected)
> > 
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>  
> 
> Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
> 
> Thanks,
> Conor.

Krzysztof,

These look fine to me, but I guess I should wait for the
dt-schema patch to land before taking them?

Jonathan
Jonathan Cameron May 14, 2023, 5:13 p.m. UTC | #4
On Sat, 13 May 2023 20:10:40 +0200
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:

> On 13/05/2023 20:13, Jonathan Cameron wrote:
> > On Sun, 7 May 2023 22:06:56 +0100
> > Conor Dooley <conor@kernel.org> wrote:
> >   
> >> On Sun, May 07, 2023 at 07:39:21PM +0200, Krzysztof Kozlowski wrote:  
> >>> Reference iio.yaml schema from dtschema to allow already used
> >>> mount-matrix property:
> >>>
> >>>   msm8953-xiaomi-daisy.dtb: imu@6a: Unevaluated properties are not allowed ('mount-matrix' was unexpected)
> >>>
> >>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>    
> >>
> >> Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
> >>
> >> Thanks,
> >> Conor.  
> > 
> > Krzysztof,
> > 
> > These look fine to me, but I guess I should wait for the
> > dt-schema patch to land before taking them?  
> 
> You can grab first patch independently. #2 and #3 should indeed wait a
> bit for Rob's ack. They do not have any conflicts but they will fix
> nothing without dtschema patch.
Applied patch 1.  Thanks,

J
> 
> Best regards,
> Krzysztof
>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
index b39f5217d8ff..ee8724ad33ab 100644
--- a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
@@ -98,6 +98,7 @@  required:
   - reg
 
 allOf:
+  - $ref: /schemas/iio/iio.yaml#
   - $ref: /schemas/spi/spi-peripheral-props.yaml#
 
 unevaluatedProperties: false