Re: [PATCH 9/9] arm64: dts: qcom: shikra-(cqm/cqs/iqs)-evk: Enable PCIe PHY node

[Sushrut Shree Trivedi]

On 7/1/2026 4:06 PM, Konrad Dybcio wrote:
> On 6/30/26 9:02 PM, Sushrut Shree Trivedi wrote:
> > Enable the PCIe PHY for the single PCIe intance on the Shikra
> > CQS, CQM and the IQS platforms.
> >
> > IQS platform uses a different powergrid than CQS/CQM which explain
> > the different PHY supplies for IQS variant.
> >
> > Signed-off-by: Sushrut Shree Trivedi <sushrut.trivedi@xxxxxxxxxxxxxxxx>
> > ---
> >   arch/arm64/boot/dts/qcom/shikra-cqm-evk.dts | 7 +++++++
> >   arch/arm64/boot/dts/qcom/shikra-cqs-evk.dts | 7 +++++++
> >   arch/arm64/boot/dts/qcom/shikra-iqs-evk.dts | 7 +++++++
> >   3 files changed, 21 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/shikra-cqm-evk.dts b/arch/arm64/boot/dts/qcom/shikra-cqm-evk.dts
> > index 683b5245923b..06ad32041546 100644
> > --- a/arch/arm64/boot/dts/qcom/shikra-cqm-evk.dts
> > +++ b/arch/arm64/boot/dts/qcom/shikra-cqm-evk.dts
> > @@ -60,6 +60,13 @@ vreg_pmu_ch1: ldo4 {
> >   	};
> >   };
> >   
> > +&pcie_phy {
> > +	vdda-phy-supply = <&pm4125_l13>;
> > +	vdda-pll-supply = <&pm4125_l9>;
> > +
> > +	status = "okay";
> > +};
> I think it makes sense to push the status=okay to the evk file
> (because we already describe the PCIe switch there) and only keep
> the supplies here (because they differ)
>
> Konrad

ACK'd.

Sushrut