Re: [PATCH v2 3/3] arm64: dts: qcom: eliza: Enable Eliza MTP board support

From: Abel Vesa

Date: Tue Mar 17 2026 - 04:50:12 EST


On 26-03-13 17:51:37, Dmitry Baryshkov wrote:
> On Fri, Mar 13, 2026 at 03:23:52PM +0200, Abel Vesa wrote:
> > The MTP is a one of the boards that comes with the Eliza SoC.
> > So add dedicated board dts for it.
> >
> > The initial support enables:
> > - UART debug console
> > - Ob-board UFS storage
> > - Qualcomm RPMh regulators (PMIC) and VPH_PWR
> > - board specific clocks & reserved GPIO ranges
> >
> > Co-developed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Abel Vesa <abel.vesa@xxxxxxxxxxxxxxxx>
> > ---
> > arch/arm64/boot/dts/qcom/Makefile | 1 +
> > arch/arm64/boot/dts/qcom/eliza-mtp.dts | 407 +++++++++++++++++++++++++++++++++
> > 2 files changed, 408 insertions(+)
> >
> > +
> > + vreg_l12b: ldo12 {
> > + regulator-name = "vreg_l12b";
> > + /*
> > + * Skip voltage voting for UFS VCC.
>
> Please continue with 'why?'

Actually, I think I'm going to drop the comment and add the voltages range
in here.

For some reason, when I added these, I considered that if other platforms
are skipping the voltage voting, Eliza should do that too. But now that I
took another look, I really don't think the voltage voting should be
skipped at all.

Downstream does the voting as well.

So I'm going to add the following in the next version:

regulator-min-microvolt = <2400000>;
regulator-max-microvolt = <3300000>;

Thanks for pointing this out.