Re: [PATCH v3 06/12] dt-bindings: power: supply: smb347-charger: Document USB VBUS regulator

On Tue, Jul 13, 2021 at 03:22:40AM +0300, Dmitry Osipenko wrote:

12.07.2021 18:39, Rob Herring пишет:

quoted

quoted

+  summit,inok-polarity:
+    description: |
+      Polarity of INOK signal indicating presence of external power supply.
+    $ref: /schemas/types.yaml#/definitions/uint32
+    enum:
+      - 0 # SMB3XX_SYSOK_INOK_ACTIVE_LOW
+      - 1 # SMB3XX_SYSOK_INOK_ACTIVE_HIGH
+
+  usb-vbus:
+    $ref: "../../regulator/regulator.yaml#"
+    type: object

       unevaluatedProperties: false

With that,

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

I tried to add the unevaluatedProperties + a random unrelated property
to the example usb-vbus node and dt_binding_check is happy with that. So
the unevaluatedProperties has no effect, is it supposed to be so?

Yes, until support lands upstream[1].

Rob

[1] https://github.com/Julian/jsonschema/pull/817