Re: [PATCH 09/24] arm64: dts: qcom: glymur-crd: Add RPMH regulator rails

[Kamal Wadhwa]

On Thu, Sep 25, 2025 at 01:01:56PM +0200, Konrad Dybcio wrote:
> On 9/25/25 8:32 AM, Pankaj Patil wrote:
> > From: Kamal Wadhwa <kamal.wadhwa@xxxxxxxxxxxxxxxx>
> > 
> > Add RPMH regulator rails for Glymur CRD.
> > 
> > Signed-off-by: Kamal Wadhwa <kamal.wadhwa@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Pankaj Patil <pankaj.patil@xxxxxxxxxxxxxxxx>
> > ---
> 
> [...]
> 
> > +	regulators-1 {
> > +		compatible = "qcom,pmcx0102-rpmh-regulators";
> > +		qcom,pmic-id = "C_E0";
> > +		vdd-s1-supply = <&vph_pwr>;
> > +		vdd-s8-supply = <&vph_pwr>;
> > +
> > +		vreg_s1c_e0_0p3: smps1 {
> > +			regulator-name = "vreg_s1c_e0_0p3";
> > +			regulator-min-microvolt = <300000>;
> > +			regulator-max-microvolt = <1200000>;
> > +			regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > +		};
> > +
> > +		vreg_s8c_e0_0p3: smps8 {
> > +			regulator-name = "vreg_s8c_e0_0p3";
> > +			regulator-min-microvolt = <300000>;
> > +			regulator-max-microvolt = <1200000>
> 
> Both of these regulators, having no consumers, will be parked to 0.3 V
> (the lower bound)
> 
> There are other similar cases in this patch

Ok. I will remove the unused rails.

But just wanted to let you know currently i have exposed all the rails that
are allowed to be controlled from APPS, mostly these rails will be staying
OFF if no clients in SW are there to vote on them.

But do note that some of the clients may be getting added as more features
get added, as lot of these rails are not unused in the HW. The client driver
just isnt enabled as of now.

So wanted to check if I should remove ALL rails that are unused in SW?
 or
Can i keep the ones for which clients will be getting added in near future.

(i would prefer the later option, if that is ok with you?)

> 
> Does the board still boot with all the expected functionality with only
> patches 1-9 applied?

No. just tested, it seems not able to boot properly with just 1-9 patches.
is your concern about squashing of the patches?
(just trying to understand)

> 
> Konrad

Regards,
Kamal