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

Gary Yang gary.yang at cixtech.com
Tue Mar 10 19:31:37 PDT 2026


Hi Krzysztof:

Thanks for your comments

> EXTERNAL EMAIL
> 
> On 10/03/2026 07:53, Gary Yang wrote:
> > 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.
Describe the power-domains property so that DTS files can reference the correct power domain provider.

> >
> > Signed-off-by: Gary Yang <gary.yang at 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
> > 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

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.

DTEX    Documentation/devicetree/bindings/pci/cix,sky1-pcie-host.example.dts
DTC [C] Documentation/devicetree/bindings/pci/cix,sky1-pcie-host.example.dtb

and

CHKDT   ./Documentation/devicetree/bindings
LINT    ./Documentation/devicetree/bindings
DTEX    Documentation/devicetree/bindings/pci/cix,sky1-pcie-host.example.dts
DTC [C] Documentation/devicetree/bindings/pci/cix,sky1-pcie-host.example.dtb

If take any mistakes and miss any information, please let us know.

Best Ragards
Gary

> Best regards,
> Krzysztof


More information about the linux-arm-kernel mailing list