Re: [PATCH v3 2/3] arm64: dts: qcom: sm6115-xiaomi-lemon: Split out a common dtsi

[Amir Hanip <killdeath104@gmail.com>]

On 2026-11-07 19:26, Barnabas Czeman wrote:
> You should not override rpm_request node here, you can use the labels
> from the common to change the ranges like:
> &pm6125_s7 {
>     regulator-min-microvolt = <1280000>;
>     regulator-max-microvolt = <2080000>;
> };
> [...]
> Please add the missing regulators.

Hi Barnabas,

Thanks, understood. In v4 the common dtsi will define all the PM6125
regulators with their labels, and lemon and citrus will only change
the ranges that differ through those labels, without overriding
rpm_requests. I assume that by the missing regulators you mean the
ones that lemon.dts still defines (s7, l5, l7, l11, l17, l18, l22 and
l24). Is that right?

About your previous mail: thanks for the dumps. In v4 I will move the
ADSP and GPU carve-outs to the common dtsi with the values from your
MIUI dump (ADSP 0x53800000/0x2200000, GPU 0x55a15000/0x2000), as a
separate patch from the pure split, so it can be reviewed on its own.
It changes lemon's compiled DT: the ADSP region shrinks by 4 MiB and
the GPU region moves away from 0xfa400000.

I can only test citrus. Would you, or Adam, be able to test the lemon
patches in v4? A Tested-by would help a lot.

Sashiko points out that WiFi bypasses the wcn3950-pmu regulators.
WiFi works on citrus as is, but I will route its supplies through
vreg_pmu_xo/rf/ch0 like qrb2210-rb1 does and test it before v4.

Thanks,
Amir