Thread (9 messages) 9 messages, 3 authors, 2026-03-13

回复: 回复: [PATCH v2 1/2] dt-bindings: PCI: update CIX Sky1 PCIe Root Complex bindings

From: Gary Yang <gary.yang@cixtech.com>
Date: 2026-03-13 02:13:38
Also in: linux-devicetree, linux-pci, lkml

Hi Krzysztof:

Thanks for your comments
EXTERNAL EMAIL

On 11/03/2026 03:31, Gary Yang wrote:
quoted
Hi Krzysztof:

Thanks for your comments
quoted
EXTERNAL EMAIL

On 10/03/2026 07:53, Gary Yang wrote:
quoted
add power-domains and power-domain-names property
Why? Please write complete sentences explaining why you are doing this.
Sorry, I did not describe in detail.

The Sky1 PCIe controller sits in a power domain that must be enabled before
the controller can be accessed.
quoted
Describe the power-domains property so that DTS files can reference the
correct power domain provider.
quoted
quoted
quoted
Signed-off-by: Gary Yang <gary.yang@cixtech.com>
---
 .../devicetree/bindings/pci/cix,sky1-pcie-host.yaml         | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git
a/Documentation/devicetree/bindings/pci/cix,sky1-pcie-host.yaml
b/Documentation/devicetree/bindings/pci/cix,sky1-pcie-host.yaml
quoted
index b910a42e0843..1d8ee8310588 100644
--- a/Documentation/devicetree/bindings/pci/cix,sky1-pcie-host.yaml
+++ b/Documentation/devicetree/bindings/pci/cix,sky1-pcie-host.yaml
@@ -38,6 +38,12 @@ properties:
   ranges:
     maxItems: 3

+  power-domains:
+    minItems: 1
+
+  power-domain-names:
+    minItems: 1
There is no such syntax. Open any existing bindinh.
Sorry, They come from
Documentation/devicetree/bindings/pci/fsl,imx6q-pcie-common.yaml
There is no such syntax there.

Just open the file.
Maybe we see it from the different views. Anyway, it's not important.
Do you agree this scheme? Like this:
  power-domains:
    maxItems: 1
  power-domain-names:
    maxItems: 1
quoted
Execute the two commands below before submit patch every time.

make O=$OUTKNL dt_binding_check
make O=$OUTKNL dt_binding_check
DT_SCHEMA_FILES=cix,sky1-pcie-host.yaml

we don't find any warning or error. So we think all is ok.
No, because there is no such code nowhere. Please take other bindings as an
example. You miss here maxItems.
Ok, thanks for your kindly remind. We get it.
Thanks for your suggestions

Best regards
Gary
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