On 12/31/25 5:19 PM, Dmitry Baryshkov wrote:
> On Wed, Dec 31, 2025 at 01:09:49PM +0100, Konrad Dybcio wrote:
>> On 12/31/25 12:35 AM, Dmitry Baryshkov wrote:
>>> Specify power supply for the second chain / antenna output of the
>>> onboard WiFi chip.
>>>
>>> Fixes: 3f72e2d3e682 ("arm64: dts: qcom: Add Dragonboard 845c")
>>> Signed-off-by: Dmitry Baryshkov <[email protected]>
>>> ---
>>
>> I don't have schematics nearby, but jfyi downstream doesn't do that
>
> I checked in the schematics, it is L23A.
>
>>
>>> arch/arm64/boot/dts/qcom/sdm845-db845c.dts | 7 +++++++
>>> 1 file changed, 7 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
>>> b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
>>> index ad283a79bcdb..5118b776a9bb 100644
>>> --- a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
>>> +++ b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
>>> @@ -379,6 +379,12 @@ vreg_l21a_2p95: ldo21 {
>>> regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>> };
>>>
>>> + vreg_l23a_3p3: ldo23 {
>>> + regulator-min-microvolt = <3300000>;
>>> + regulator-max-microvolt = <3312000>;
>>
>> Do we benefit from the 0.012V spread?
>>
>> (i.e. is there anything else on this rail?)
>
> I must admt jere, I just copied defition of the ch0 regulator. I think,
> they should be the same.
Alright
Reviewed-by: Konrad Dybcio <[email protected]>
Konrad
