Received-SPF: pass (google.com: domain of linux-kernel+bounces-100001-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: Xu Yang <xu.yang_2@nxp.com>
To: gregkh@linuxfoundation.org,
	robh+dt@kernel.org,
	krzysztof.kozlowski+dt@linaro.org,
	shawnguo@kernel.org,
	conor+dt@kernel.org
Cc: s.hauer@pengutronix.de,
	kernel@pengutronix.de,
	festevam@gmail.com,
	linux-imx@nxp.com,
	peter.chen@kernel.org,
	xu.yang_2@nxp.com,
	jun.li@nxp.com,
	linux-usb@vger.kernel.org,
	devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	imx@lists.linux.dev,
	linux-kernel@vger.kernel.org
Subject: [PATCH v8 05/10] dt-bindings: usb: ci-hdrc-usb2-imx: add restrictions for reg, interrupts, clock and clock-names properties
Date: Tue, 12 Mar 2024 17:16:58 +0800
Message-Id: <20240312091703.1220649-5-xu.yang_2@nxp.com>
In-Reply-To: <20240312091703.1220649-1-xu.yang_2@nxp.com>
References: <20240312091703.1220649-1-xu.yang_2@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?yT+S7dCaPv3gHZqus+trDBZipTtf7sCEerPFW6+g9+7vk2CUyIh572QUlLf0?=
 =?us-ascii?Q?lEbrYN7xybph+3H+0b0llfl/AXDUMDM03nzDkbFM+jVquIWJjZOxYl2X0y/x?=
 =?us-ascii?Q?WOLKVExnyUZ9uFs9ORx/F5g/kG6+pZQJYWCtxV9Rw7+VNugWocymlpV+1a7z?=
 =?us-ascii?Q?LNw7M6Z5CKNsHCmWQoU7bgujGc4wH+O1VLuEy6vihVYnZzvfdHj7j+UP3frU?=
 =?us-ascii?Q?IE1vo5L3zvcZQENQ/29e/9HJ3z5E6jsSTfaaL1P/E3cTTnCyBygQbqX6QmFQ?=
 =?us-ascii?Q?AGzBB2pPVwcMG6GhAKsTyac/uM28MUzoMIoyEFjtJKpthSLzuopkqkBJqw5W?=
 =?us-ascii?Q?jJBaMamoau2GrEk4dldDeUHnQngYFigKxzYfJdozwQH1ze/JFSgtCHqXxNAQ?=
 =?us-ascii?Q?YCNSwoPUKLJFiaqDkjfwglHV2NPw6xaFFaUOkCkHhCEo24DH2YwsW+vESAx9?=
 =?us-ascii?Q?Lowwh+WTOknP6H+uz3ULoM2kf/NmP8kOqyj0zMicSn7yF4/Zl+DiA2uilfLW?=
 =?us-ascii?Q?TgiOB3t/Mpyhy07ubmnRotk6HORWgCP6W4+iNUpkvhMFfavoWE2CIDighgZS?=
 =?us-ascii?Q?nqoQK4PyGBW7VR7WXuwXeu3yKZk2eo+7i53xvqNj7Ugxcjtwj+9AiOtUNjaZ?=
 =?us-ascii?Q?SMVd69cDX9vJjd5NDhpQjhnemN7prM3k7mrBYrYsUmk2Ch6T3W5E6HcykGEp?=
 =?us-ascii?Q?EZzZ17KAn6b4fuCzWsTKuynv4IgWihtbbOF3kWfSG2HP2T5bPZTq6izT43dg?=
 =?us-ascii?Q?f2GsIN8faicITqw8QzuMSUpcFrpXVmRrYkJoDo3GNz0VV/Ls+IfQRINsGL8T?=
 =?us-ascii?Q?3XcG14b6ArGW/P/u9YyhpCAEzdxBRPF0d1rLjToNhWybxUARtiUBZTu6w6TD?=
 =?us-ascii?Q?yWz89v/wQcobZvVjqEVHEriMAJ6HXlDk3gTgkNRE4vpDeGDGL1ekXn3jD58D?=
 =?us-ascii?Q?8G9mLaHcIWc/zj8jYvFJdBoacEWnODEv9uCeDsBXAcUa31youPWt8KY5TRMG?=
 =?us-ascii?Q?1C40uQNUV20w3k0A4v/U46xFe+VZAnmxBkvBmBm5k59pgY+7LpanbTKqqtCa?=
 =?us-ascii?Q?YGnEcz6+rskUqQpO5inbD23E8xoT1+Aa9Uptj2Uv8Bw6vGTkC81BCtFcjg6s?=
 =?us-ascii?Q?TcggCFwdEBc/GuRpw9V4R/Y9jORbFez+dVRjyetteet/P0kYnHdKERm/1Psb?=
 =?us-ascii?Q?IZ5EObJgpssYfX5AqhOuP1D3ZgVqFfTvLzxZWKjAqnwceSolSTP+NK2LJJNe?=
 =?us-ascii?Q?GPZfIwCgq+Z/Mx35HcMn/ou7xZ9nZs3M/c352QO9SDCIjW6K0KfQS6dUdDzG?=
 =?us-ascii?Q?2K5Fa76ACCbFTkGgm5BQzJSrrE3aNxAp9vO6LZRRlroZfR9le1fgamOjpE/j?=
 =?us-ascii?Q?XfhcHvAQD3BoFRffEIZuEXVi7i7l9uZTlMHAfZoTftVaeTjb3B96NKW7sRBh?=
 =?us-ascii?Q?lWznncsh9sHOvS/oHwuBuVXI87HUxaqZAJ5pvZLz35rHAxIqfeyUuGYNw4gR?=
 =?us-ascii?Q?9fzSs3O7/Mm+5N4mG3F7cZPPtzeCGSX8iZ/UGZ44ACsVE0lh6PjjapVuIyWL?=
 =?us-ascii?Q?yY9CoaTfE3LGk/qWY/OqBXXfERNUA4vbFeWoeq0f?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: febb59af-774c-4d09-5109-08dc427535b1
X-MS-Exchange-CrossTenant-AuthSource: DU2PR04MB8822.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Mar 2024 09:17:16.0875
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 7U0UW4b/Js3HMeSsI5tNQQpyJP6vI8nKYV7Euo5EZRG2m2Lkd/1A2sw2VS7lJB/em8PVnwLkNLZP8zLhcGdAsg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM8PR04MB7876

Add restrictions for reg, interrupts, clock and clock-names properties
for imx Socs.

Signed-off-by: Xu Yang <xu.yang_2@nxp.com>

---
Changes in v4:
 - new patch since v3's discussion
 - split the reg, interrupts, clock and clock-names properties into
   common part and device-specific
Changes in v5:
 - keep common property unchanged
 - make if-then more readable
 - remove non imx part
Changes in v6:
 - new patch based on ci-hdrc-usb2-imx.yaml
Changes in v7:
 - no changes
Changes in v8:
 - remove if:else:if:else:if:else block
---
 .../bindings/usb/chipidea,usb2-imx.yaml       | 80 +++++++++++++++++++
 1 file changed, 80 insertions(+)

diff --git a/Documentation/devicetree/bindings/usb/chipidea,usb2-imx.yaml b/Documentation/devicetree/bindings/usb/chipidea,usb2-imx.yaml
index cdbb224e9f68..fb1c378dfe88 100644
--- a/Documentation/devicetree/bindings/usb/chipidea,usb2-imx.yaml
+++ b/Documentation/devicetree/bindings/usb/chipidea,usb2-imx.yaml
@@ -49,6 +49,12 @@ properties:
           - const: fsl,imx6ul-usb
           - const: fsl,imx27-usb
 
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
   clocks:
     minItems: 1
     maxItems: 3
@@ -144,6 +150,80 @@ allOf:
             - const: idle
             - const: active
 
+  # imx27 Soc needs three clocks
+  - if:
+      properties:
+        compatible:
+          const: fsl,imx27-usb
+    then:
+      properties:
+        clocks:
+          minItems: 3
+          maxItems: 3
+        clock-names:
+          items:
+            - const: ipg
+            - const: ahb
+            - const: per
+
+  # imx25 and imx35 Soc need three clocks
+  - if:
+      properties:
+        compatible:
+          contains:
+            enum:
+              - fsl,imx25-usb
+              - fsl,imx35-usb
+    then:
+      properties:
+        clocks:
+          minItems: 3
+          maxItems: 3
+        clock-names:
+          items:
+            - const: ipg
+            - const: ahb
+            - const: per
+
+  # imx7d Soc need one clock
+  - if:
+      properties:
+        compatible:
+          items:
+            - const: fsl,imx7d-usb
+            - const: fsl,imx27-usb
+    then:
+      properties:
+        clocks:
+          maxItems: 1
+        clock-names:
+          maxItems: 1
+
+  # other Soc need one clock
+  - if:
+      properties:
+        compatible:
+          contains:
+            enum:
+              - fsl,imx23-usb
+              - fsl,imx28-usb
+              - fsl,imx50-usb
+              - fsl,imx51-usb
+              - fsl,imx53-usb
+              - fsl,imx6q-usb
+              - fsl,imx6sl-usb
+              - fsl,imx6sx-usb
+              - fsl,imx6ul-usb
+              - fsl,imx8mm-usb
+              - fsl,imx8mn-usb
+              - fsl,vf610-usb
+    then:
+      properties:
+        clocks:
+          maxItems: 1
+        clock-names:
+          maxItems: 1
+
 unevaluatedProperties: false
 
 examples:
-- 
2.34.1