[PATCH] dt-bindings: usb: ci-hdrc-usb2: allow multiple PHYs

Krzysztof Kozlowski posted 1 patch 2 years, 8 months ago
Documentation/devicetree/bindings/usb/ci-hdrc-usb2.yaml | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
[PATCH] dt-bindings: usb: ci-hdrc-usb2: allow multiple PHYs
Posted by Krzysztof Kozlowski 2 years, 8 months ago
Qualcomm MSM8974 comes with USB HS phy in two variants, although final
DTS chooses only one.  Allow such combination in the ChipIdea USB2
bindings and also disallow any other properties in the ulpi node.

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

---

DTS will be fixed separately.
---
 Documentation/devicetree/bindings/usb/ci-hdrc-usb2.yaml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/ci-hdrc-usb2.yaml b/Documentation/devicetree/bindings/usb/ci-hdrc-usb2.yaml
index e5e4dbc5a7a0..b26d26c2b023 100644
--- a/Documentation/devicetree/bindings/usb/ci-hdrc-usb2.yaml
+++ b/Documentation/devicetree/bindings/usb/ci-hdrc-usb2.yaml
@@ -322,8 +322,9 @@ properties:
 
   ulpi:
     type: object
-    properties:
-      phy:
+    additionalProperties: false
+    patternProperties:
+      "^phy(-[0-9])?$":
         description: The phy child node for Qcom chips.
         type: object
         $ref: /schemas/phy/qcom,usb-hs-phy.yaml
-- 
2.34.1
Re: [PATCH] dt-bindings: usb: ci-hdrc-usb2: allow multiple PHYs
Posted by Peng Fan 2 years, 8 months ago

On 4/20/2023 2:50 PM, Krzysztof Kozlowski wrote:
> Qualcomm MSM8974 comes with USB HS phy in two variants, although final
> DTS chooses only one.  Allow such combination in the ChipIdea USB2
> bindings and also disallow any other properties in the ulpi node.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Peng Fan <peng.fan@nxp.com>
Re: [PATCH] dt-bindings: usb: ci-hdrc-usb2: allow multiple PHYs
Posted by Rob Herring 2 years, 8 months ago
On Thu, 20 Apr 2023 08:50:51 +0200, Krzysztof Kozlowski wrote:
> Qualcomm MSM8974 comes with USB HS phy in two variants, although final
> DTS chooses only one.  Allow such combination in the ChipIdea USB2
> bindings and also disallow any other properties in the ulpi node.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> DTS will be fixed separately.
> ---
>  Documentation/devicetree/bindings/usb/ci-hdrc-usb2.yaml | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 

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