Thread (14 messages) 14 messages, 2 authors, 2024-08-13

Re: [PATCH v3 1/3] dt-bindings: pinctrl: Add support for Xilinx Versal platform

From: Krzysztof Kozlowski <hidden>
Date: 2024-08-13 10:14:44
Also in: linux-devicetree, linux-gpio, lkml

On 13/08/2024 07:40, Potthuri, Sai Krishna wrote:
quoted
sd1_wp_0_grp, sd1_wp_1_grp,
quoted
+                         ospi0_0_grp, ospi0_ss_0_grp, qspi0_0_grp,
qspi0_fbclk_0_grp, qspi0_ss_0_grp,
quoted
+                         test_clk_0_grp, test_scan_0_grp, tamper_trigger_0_grp]
+            maxItems: 78
You miss minItems... I have doubts this was really tested.
groups/pins are mentioned as required property.
Do we still need to define minItems?
These are two unrelated concepts. How many items can be listed at once
in each property?
....
quoted
quoted
+
+      drive-strength:
drive-strength-microamp
In this case, values are in mA.
Ah, no, if it is mA, then existing property is correct.
Do we still need to use drive-strength-microamp?


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