On 24/06/2025 21:08, Frank Li wrote:

quoted

quoted

quoted

quoted

+        clock-names:
+          items:
+            - const: apb
+            - const: axi
+            - const: nic
+            - const: disp
+            - const: cam
+            - const: lcdif
+            - const: isi
+            - const: csi

No, look at other bindings how they share clock lists.

Sorry, this method is what I suggested. becuase there are pxp between cam
and lcdif, can't use simple minItems/maxItems to limit list.

The point is to put new items, so pxp, at the end.

There are already a list for imx93. If change list order, it will break
ABI. This was rejected at other binding doc review.

I see, I mixed the SoCs. It is a pity you upstream that way and do not
try to make the list common. Anyway names indeed need to be constrained
per variant, but the rest of the comments stay.

Best regards,
Krzysztof