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

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

[Krzysztof Kozlowski]

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(+)

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
-- 
2.34.1

[PATCH 2/3] dt-bindings: iio: semtech,sx9310: reference common schema for label

[Krzysztof Kozlowski]

Reference iio.yaml schema from dtschema to allow already used
label property:

  sc7180-trogdor-homestar-r4.dtb: proximity@28: 'label' does not match any of the regexes: 'pinctrl-[0-9]+'

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

---

Patch for dtschema:
https://lore.kernel.org/linux-devicetree/20230507171219.232216-1-krzk@kernel.org/T/#u
---
 .../devicetree/bindings/iio/proximity/semtech,sx9310.yaml    | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/iio/proximity/semtech,sx9310.yaml b/Documentation/devicetree/bindings/iio/proximity/semtech,sx9310.yaml
index 5de0bb2180e6..775555d147bf 100644
--- a/Documentation/devicetree/bindings/iio/proximity/semtech,sx9310.yaml
+++ b/Documentation/devicetree/bindings/iio/proximity/semtech,sx9310.yaml
@@ -15,6 +15,9 @@ description: |
   Specifications about the devices can be found at:
   https://www.semtech.com/products/smart-sensing/sar-sensors/sx9310
 
+allOf:
+  - $ref: /schemas/iio/iio.yaml#
+
 properties:
   compatible:
     enum:
@@ -102,7 +105,7 @@ required:
   - reg
   - "#io-channel-cells"
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
-- 
2.34.1

Re: [PATCH 2/3] dt-bindings: iio: semtech,sx9310: reference common schema for label

[Rob Herring]

On Sun, 07 May 2023 19:39:22 +0200, Krzysztof Kozlowski wrote:
> Reference iio.yaml schema from dtschema to allow already used
> label property:
> 
>   sc7180-trogdor-homestar-r4.dtb: proximity@28: 'label' does not match any of the regexes: 'pinctrl-[0-9]+'
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> Patch for dtschema:
> https://lore.kernel.org/linux-devicetree/20230507171219.232216-1-krzk@kernel.org/T/#u
> ---
>  .../devicetree/bindings/iio/proximity/semtech,sx9310.yaml    | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 

Acked-by: Rob Herring <robh@kernel.org>

[PATCH 3/3] dt-bindings: iio: semtech,sx9324: reference common schema for label

[Krzysztof Kozlowski]

Reference iio.yaml schema from dtschema to allow already used
label property:

  sc7180-trogdor-pazquel360-lte.dtb: proximity@28: 'label' does not match any of the regexes: 'pinctrl-[0-9]+'

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

---

Patch for dtschema:
https://lore.kernel.org/linux-devicetree/20230507171219.232216-1-krzk@kernel.org/T/#u
---
 .../devicetree/bindings/iio/proximity/semtech,sx9324.yaml    | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml b/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml
index b3aa2ebf9661..48f221463166 100644
--- a/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml
+++ b/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml
@@ -13,6 +13,9 @@ maintainers:
 description: |
   Semtech's SX9324 proximity sensor.
 
+allOf:
+  - $ref: /schemas/iio/iio.yaml#
+
 properties:
   compatible:
     const: semtech,sx9324
@@ -167,7 +170,7 @@ required:
   - reg
   - "#io-channel-cells"
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
-- 
2.34.1

Re: [PATCH 3/3] dt-bindings: iio: semtech,sx9324: reference common schema for label

[Rob Herring]

On Sun, 07 May 2023 19:39:23 +0200, Krzysztof Kozlowski wrote:
> Reference iio.yaml schema from dtschema to allow already used
> label property:
> 
>   sc7180-trogdor-pazquel360-lte.dtb: proximity@28: 'label' does not match any of the regexes: 'pinctrl-[0-9]+'
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> Patch for dtschema:
> https://lore.kernel.org/linux-devicetree/20230507171219.232216-1-krzk@kernel.org/T/#u
> ---
>  .../devicetree/bindings/iio/proximity/semtech,sx9324.yaml    | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 

Acked-by: Rob Herring <robh@kernel.org>

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

[Conor Dooley]

[-- Attachment #1: Type: text/plain, Size: 427 bytes --]

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.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

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

[Jonathan Cameron]

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

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

[Krzysztof Kozlowski]

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

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

[Jonathan Cameron]

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
>