Thread (21 messages) 21 messages, 5 authors, 2025-06-25

Re: [PATCH v6 2/9] dt-bindings: soc: imx-blk-ctrl: add i.MX91 blk-ctrl compatible

From: Frank Li <Frank.li@nxp.com>
Date: 2025-06-24 19:09:06
Also in: imx, linux-arm-kernel, linux-devicetree, lkml

On Tue, Jun 24, 2025 at 05:39:43PM +0200, Krzysztof Kozlowski wrote:
On 24/06/2025 16:51, Frank Li wrote:
quoted
quoted
Why imx91 now has 10 clocks?

v6 and this has basic issues. The quality of NXP patches decreases :/
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.

Frank
quoted
If put two lists at top, clock-names list have to dupliate at both top and
if-else branch.

If there are better solution, please point me which file have good method
to share two totally difference list.
Best regards,
Krzysztof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help