[RESEND PATCH] dt-bindings: usb: samsung,exynos-usb2: include usb-hcd schema

[RESEND PATCH] dt-bindings: usb: samsung,exynos-usb2: include usb-hcd schema

[Krzysztof Kozlowski]

Remove parts duplicated with usb-hcd.yaml DT schema and include it
directly.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Rob Herring <robh@kernel.org>
---
 .../bindings/usb/samsung,exynos-usb2.yaml         | 15 ++-------------
 1 file changed, 2 insertions(+), 13 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/samsung,exynos-usb2.yaml b/Documentation/devicetree/bindings/usb/samsung,exynos-usb2.yaml
index 340dff8d19c3..d51f91d1139f 100644
--- a/Documentation/devicetree/bindings/usb/samsung,exynos-usb2.yaml
+++ b/Documentation/devicetree/bindings/usb/samsung,exynos-usb2.yaml
@@ -15,9 +15,6 @@ properties:
       - samsung,exynos4210-ehci
       - samsung,exynos4210-ohci
 
-  '#address-cells':
-    const: 1
-
   clocks:
     maxItems: 1
 
@@ -46,15 +43,6 @@ properties:
       Only for controller in EHCI mode, if present, specifies the GPIO that
       needs to be pulled up for the bus to be powered.
 
-  '#size-cells':
-    const: 0
-
-patternProperties:
-  "^.*@[0-9a-f]{1,2}$":
-    description: The hard wired USB devices
-    type: object
-    $ref: /usb/usb-device.yaml
-
 required:
   - compatible
   - clocks
@@ -64,6 +52,7 @@ required:
   - phy-names
 
 allOf:
+  - $ref: usb-hcd.yaml#
   - if:
       properties:
         compatible:
@@ -73,7 +62,7 @@ allOf:
       properties:
         samsung,vbus-gpio: false
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
-- 
2.32.0

RE: [RESEND PATCH] dt-bindings: usb: samsung,exynos-usb2: include usb-hcd schema

[Alim Akhtar]

Hi Krzysztof

>-----Original Message-----
>From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org]
>Sent: Friday, April 8, 2022 12:56 AM
>To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>; Rob Herring
><robh+dt@kernel.org>; Krzysztof Kozlowski
><krzysztof.kozlowski+dt@linaro.org>; Alim Akhtar
><alim.akhtar@samsung.com>; linux-usb@vger.kernel.org;
>devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
>samsung-soc@vger.kernel.org; linux-kernel@vger.kernel.org
>Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>; Rob Herring
><robh@kernel.org>
>Subject: [RESEND PATCH] dt-bindings: usb: samsung,exynos-usb2: include
>usb-hcd schema
>
>Remove parts duplicated with usb-hcd.yaml DT schema and include it
directly.
>
>Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>Reviewed-by: Rob Herring <robh@kernel.org>
>---

Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>

> .../bindings/usb/samsung,exynos-usb2.yaml         | 15 ++-------------
> 1 file changed, 2 insertions(+), 13 deletions(-)
>
>diff --git a/Documentation/devicetree/bindings/usb/samsung,exynos-
>usb2.yaml b/Documentation/devicetree/bindings/usb/samsung,exynos-
>usb2.yaml
>index 340dff8d19c3..d51f91d1139f 100644
>--- a/Documentation/devicetree/bindings/usb/samsung,exynos-usb2.yaml
>+++ b/Documentation/devicetree/bindings/usb/samsung,exynos-usb2.yaml
>@@ -15,9 +15,6 @@ properties:
>       - samsung,exynos4210-ehci
>       - samsung,exynos4210-ohci
>
>-  '#address-cells':
>-    const: 1
>-
>   clocks:
>     maxItems: 1
>
>@@ -46,15 +43,6 @@ properties:
>       Only for controller in EHCI mode, if present, specifies the GPIO
that
>       needs to be pulled up for the bus to be powered.
>
>-  '#size-cells':
>-    const: 0
>-
>-patternProperties:
>-  "^.*@[0-9a-f]{1,2}$":
>-    description: The hard wired USB devices
>-    type: object
>-    $ref: /usb/usb-device.yaml
>-
> required:
>   - compatible
>   - clocks
>@@ -64,6 +52,7 @@ required:
>   - phy-names
>
> allOf:
>+  - $ref: usb-hcd.yaml#
>   - if:
>       properties:
>         compatible:
>@@ -73,7 +62,7 @@ allOf:
>       properties:
>         samsung,vbus-gpio: false
>
>-additionalProperties: false
>+unevaluatedProperties: false
>
> examples:
>   - |
>--
>2.32.0