Re: [PATCH 2/2] regulator: qcom-refgen: add support for the IPQ9650 SoC

[Kathiravan Thirumoorthy]

On 6/9/2026 7:18 PM, Konrad Dybcio wrote:
> On 6/2/26 11:22 AM, Kathiravan Thirumoorthy wrote:
> > IPQ9650 SoC has 2 REFGEN blocks providing the reference current to the
> > PCIe and USB, UNIPHY PHYs. For the other SoCs, clocks for this block is
> > enabled on power up but that's not the case for IPQ9650 and we have to
> > enable those clocks explicitly to bring up the PHYs properly.
> >
> > As per the design team, REFGEN block provides the reference current.
> > Hence marked the regulator type as REGULATOR_CURRENT.
> >
> > Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@xxxxxxxxxxxxxxxx>
> > ---
> I'm slightly confused. Is there no register programming needed?

Yeah, no programming is needed. Similar to Kodiak, PHYs in IPQ9650 does 
the HW votes. No SW vote is needed.

> Can we at least retrieve the running state in .is_enabled() by reading
> some value?

Based on the discussion with IP team, I can use the REFGEN_STATUS 
register. Let me update the is_enabled() to query this information.

> Konrad