Re: [PATCH] arm64: dts: qcom: ipq9574: Fix 'l2' regulator min/max voltages

From: Konrad Dybcio
Date: Tue Jun 24 2025 - 10:53:35 EST


On 6/24/25 8:08 AM, Varadarajan Narayanan wrote:
> On Mon, Jun 23, 2025 at 01:34:22PM +0200, Konrad Dybcio wrote:
>> On 6/23/25 1:32 PM, Varadarajan Narayanan wrote:
>>> The min and max voltages on l2 regulator is 850000uV. This was
>>> incorrectly set at 1800000uV earlier and that affected the
>>> stability of the networking subsystem.
>>>
>>> Fixes: d5506524d9d9 ("arm64: dts: qcom: ipq9574: Add LDO regulator node")
>>> Signed-off-by: Varadarajan Narayanan <quic_varada@xxxxxxxxxxx>
>>> ---
>>
>> Shouldn't there be a consumer for it, instead of it being always-on?
>
> The uniphy block is the consumer of this voltage. The PMIC configures it
> to 850000uV based on OTP settings and s/w doesn't change it as uniphy
> doesn't do scaling.

Is there a chance something in the networking part can probe before
the regulator driver, resulting in a momentary undervolting?

Konrad