Review given to v2 [1] of commit 76cbb3eabf0b ("dt-bindings: usb: realtek,rts5411: Adapt usb-hub.yaml") asked to use unevaluatedProperties, but this was ignored by the author probably because current dtschema does not allow to use both additionalProperties and unevaluatedProperties. As an effect, this binding does not end with unevaluatedProperties and allows any properties to be added.
Fix this by reverting the approach suggested at v2 review and using simpler definition of "reg" constraints.
Link: https://lore.kernel.org/r/20250416180023.GB3327258-robh@kernel.org/ [1] Fixes: 76cbb3eabf0b ("dt-bindings: usb: realtek,rts5411: Adapt usb-hub.yaml") Cc: stable@vger.kernel.org Signed-off-by: Krzysztof Kozlowski krzysztof.kozlowski@oss.qualcomm.com --- .../devicetree/bindings/usb/realtek,rts5411.yaml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/usb/realtek,rts5411.yaml b/Documentation/devicetree/bindings/usb/realtek,rts5411.yaml index a020afaf2d6e..a86afa8fdc91 100644 --- a/Documentation/devicetree/bindings/usb/realtek,rts5411.yaml +++ b/Documentation/devicetree/bindings/usb/realtek,rts5411.yaml @@ -19,6 +19,10 @@ properties: - usbbda,5411 - usbbda,411
+ reg: + minimum: 1 + maximum: 4 + vdd-supply: description: phandle to the regulator that provides power to the hub. @@ -37,17 +41,13 @@ properties: minimum: 1 maximum: 4
-additionalProperties: - properties: - reg: - minimum: 1 - maximum: 4 - required: - peer-hub - compatible - reg
+unevaluatedProperties: false + examples: - | usb {