Re: [PATCH 2/3] arm64: qcom: sc7280: Move phy, perst to root port node

[Krishna Chaitanya Chundru]

On 3/25/2025 12:45 AM, Konrad Dybcio wrote:
> On 3/22/25 4:00 AM, Krishna Chaitanya Chundru wrote:
> > Move phy, perst, to root port from the controller node.
> >
> > Rename perst-gpios to reset-gpios to align with the expected naming
> > convention of pci-bus-common.yaml.
> >
> > Signed-off-by: Krishna Chaitanya Chundru <krishna.chundru@xxxxxxxxxxxxxxxx>
> > ---
>
> [...]
>
> > diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> > index 0f2caf36910b..6c21c320a2b5 100644
> > --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> > @@ -2271,9 +2271,6 @@ pcie1: pcie@1c08000 {
> >   
> >   			power-domains = <&gcc GCC_PCIE_1_GDSC>;
> >   
> > -			phys = <&pcie1_phy>;
> > -			phy-names = "pciephy";
> > -
> >   			pinctrl-names = "default";
> >   			pinctrl-0 = <&pcie1_clkreq_n>;
> >   
> > @@ -2284,7 +2281,7 @@ pcie1: pcie@1c08000 {
> >   
> >   			status = "disabled";
> >   
> > -			pcie@0 {
> > +			pcieport1: pcie@0 {
>
> pcie1_port0 (or pcie1_port), please
pcie1_port0 makes more sense as pcie1 indicates pcie instance
and port0 indicates root port0. I will update this in next
series.

- Krishna Chaitanya.
> Konrad