回复: 回复: [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 commentsquoted
EXTERNAL EMAIL On 10/03/2026 07:53, Gary Yang wrote:quoted
add power-domains and power-domain-names propertyWhy? 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 beforethe controller can be accessed.quoted
Describe the power-domains property so that DTS files can reference thecorrect 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.yamlb/Documentation/devicetree/bindings/pci/cix,sky1-pcie-host.yamlquoted
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: 1There is no such syntax. Open any existing bindinh.Sorry, They come from Documentation/devicetree/bindings/pci/fsl,imx6q-pcie-common.yamlThere 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