dt-bindings: usb: ci-hdrc-usb2: allow multiple PHYs

Message ID 20230420065051.22994-1-krzysztof.kozlowski@linaro.org
State New
Headers
Series dt-bindings: usb: ci-hdrc-usb2: allow multiple PHYs |

Commit Message

Krzysztof Kozlowski April 20, 2023, 6:50 a.m. UTC
  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(-)
  

Comments

Rob Herring April 20, 2023, 4:57 p.m. UTC | #1
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>
  
Peng Fan (OSS) April 21, 2023, 1:13 a.m. UTC | #2
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>
  

Patch

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