Re: [PATCH 2/3] arm64: dts: qcom: Add Talos Lyra EVK SoM platform

From: Rakesh Kota

Date: Fri Sep 18 2026 - 06:42:44 EST

On Thu, Sep 03, 2026 at 09:54:36AM -0500, Bjorn Andersson wrote:
> On Tue, Sep 01, 2026 at 07:14:02PM +0530, Avaneesh Kumar Dwivedi wrote:
>
> Please make subject prefix arm64: dts: qcom: talos: ... so that it
> groups nicely in a git log --oneline.
>
> > Introduce the device tree for the QCS615-based Talos Lyra EVK SoM.
> > Lyra EVK SoM a compact compute module integrating the QCS615 SoC,
> > PMIC, and essential connectivity, designed to mount on carrier boards.
> >
> > The initial SoM device tree includes basic support for:
> >
> > - CPU and memory
> > - PMIC and board-level regulators
> > - UFS and SD card storage
> > - QUPv3 (I2C/SPI/UART) instances
> > - ADSP/CDSP remoteprocs
> > - GPU
> >
> > Co-developed-by: Monish Chunara <quic_mchunara@xxxxxxxxxxx>
> > Signed-off-by: Monish Chunara <quic_mchunara@xxxxxxxxxxx>
> > Co-developed-by: Rakesh Kota <rakesh.kota@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Rakesh Kota <rakesh.kota@xxxxxxxxxxxxxxxx>
> > Co-developed-by: Sayali Lokhande <sayalil@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Sayali Lokhande <sayalil@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Nirmesh Kumar Singh <nirmesh.singh@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Avaneesh Kumar Dwivedi <avaneesh.dwivedi@xxxxxxxxxxxxxxxx>
>
> This s-o-b chain says Monish, Rakesh, Sayali, and Nirmesh first wrote
> the patch, but the way you sent it claims you authored it. Retain
> Nirmesh as author or fix the chain.
>
> > ---
> > arch/arm64/boot/dts/qcom/talos-lyra-evk-som.dtsi | 282 +++++++++++++++++++++++
> > 1 file changed, 282 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/talos-lyra-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-lyra-evk-som.dtsi
> > new file mode 100644
> > index 000000000000..8c19f050d5b3
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/qcom/talos-lyra-evk-som.dtsi
> > @@ -0,0 +1,282 @@
> > +// SPDX-License-Identifier: BSD-3-Clause
> > +/*
> > + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> > + */
> > +
> > +#include <dt-bindings/gpio/gpio.h>
> > +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> > +#include "talos.dtsi"
> > +#include "pm8150.dtsi"
> > +/ {
> > + aliases {
> > + mmc0 = &sdhc_1;
> > + mmc1 = &sdhc_2;
> > + serial0 = &uart0;
> > + };
> > +
> > + chosen {
> > + stdout-path = "serial0:115200n8";
> > + };
> > +
> > + clocks {
> > + sleep_clk: sleep-clk {
> > + compatible = "fixed-clock";
> > + clock-frequency = <32764>;
> > + #clock-cells = <0>;
> > + };
> > +
> > + xo_board_clk: xo-board-clk {
> > + compatible = "fixed-clock";
> > + clock-frequency = <38400000>;
> > + #clock-cells = <0>;
> > + };
> > + };
> > +};
> > +
> > +&apps_rsc {
> > + regulators-0 {
> > + compatible = "qcom,pm8150-rpmh-regulators";
> > + qcom,pmic-id = "a";
> > +
> > + vreg_s3a: smps3 {
>
> I don't see anything referencing s3a, s5a, and s6a, are you sure that
> rpmh doesn't handle the SMPSs for us automatically?
>
> Do we actually want to vote for s4a from Linux?
>

We’ll remove these unused rails in the next patch set.

> > + regulator-name = "vreg_s3a";
> > + regulator-min-microvolt = <600000>;
> > + regulator-max-microvolt = <650000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_s4a: smps4 {
> > + regulator-name = "vreg_s4a";
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <1829000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_s5a: smps5 {
> > + regulator-name = "vreg_s5a";
> > + regulator-min-microvolt = <1896000>;
> > + regulator-max-microvolt = <2040000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_s6a: smps6 {
> > + regulator-name = "vreg_s6a";
> > + regulator-min-microvolt = <1304000>;
> > + regulator-max-microvolt = <1404000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l1a: ldo1 {
> > + regulator-name = "vreg_l1a";
> > + regulator-min-microvolt = <488000>;
> > + regulator-max-microvolt = <852000>;
>
> That's a wide span, narrow it to what the consumers need.

we will fix's min and max in the next patch set.

>
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
>
> Are you sure you want to go LPM here?

we’ll keep HPM as the initial mode in the next patch set.

regards
Rakesh Kota
>
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l2a: ldo2 {
> > + regulator-name = "vreg_l2a";
> > + regulator-min-microvolt = <1650000>;
> > + regulator-max-microvolt = <3100000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l3a: ldo3 {
> > + regulator-name = "vreg_l3a";
> > + regulator-min-microvolt = <1000000>;
> > + regulator-max-microvolt = <1248000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l5a: ldo5 {
> > + regulator-name = "vreg_l5a";
> > + regulator-min-microvolt = <875000>;
> > + regulator-max-microvolt = <975000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l7a: ldo7 {
> > + regulator-name = "vreg_l7a";
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <1900000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l8a: ldo8 {
> > + regulator-name = "vreg_l8a";
> > + regulator-min-microvolt = <1150000>;
> > + regulator-max-microvolt = <1350000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l10a: ldo10 {
> > + regulator-name = "vreg_l10a";
> > + regulator-min-microvolt = <2950000>;
> > + regulator-max-microvolt = <3312000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l11a: ldo11 {
> > + regulator-name = "vreg_l11a";
> > + regulator-min-microvolt = <1232000>;
> > + regulator-max-microvolt = <1260000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l12a: ldo12 {
> > + regulator-name = "vreg_l12a";
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <1890000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l13a: ldo13 {
> > + regulator-name = "vreg_l13a";
> > + regulator-min-microvolt = <3000000>;
> > + regulator-max-microvolt = <3230000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l15a: ldo15 {
> > + regulator-name = "vreg_l15a";
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <1904000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l16a: ldo16 {
> > + regulator-name = "vreg_l16a";
> > + regulator-min-microvolt = <3000000>;
> > + regulator-max-microvolt = <3312000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
> > + regulator-allow-set-load;
> > + regulator-allowed-modes = <RPMH_REGULATOR_MODE_LPM
> > + RPMH_REGULATOR_MODE_HPM>;
> > + };
> > +
> > + vreg_l17a: ldo17 {
> > + regulator-name = "vreg_l17a";
> > + regulator-min-microvolt = <2950000>;
> > + regulator-max-microvolt = <3312000>;
> > + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> > + };
> > + };
> > +};
> > +
> > +&gcc {
> > + clocks = <&rpmhcc RPMH_CXO_CLK>,
> > + <&rpmhcc RPMH_CXO_CLK_A>,
> > + <&sleep_clk>;
>
> I'm failing to see how that differs from that talos.dtsi already says.
>
> That said, this list is incomplete and needs to be corrected - can be
> done separately.
>
> > +};
> > +
> > +&gpu {
> > + status = "okay";
> > +};
> > +
> > +&gpu_zap_shader {
> > + firmware-name = "qcom/qcs615/a612_zap.mbn";
> > +};
> > +
> > +&qupv3_id_0 {
> > + status = "okay";
> > +};
> > +
> > +&qupv3_id_1 {
> > + status = "okay";
> > +};
> > +
> > +&remoteproc_adsp {
> > + firmware-name = "qcom/qcs615/adsp.mbn";
> > +
> > + status = "okay";
> > +};
> > +
> > +&remoteproc_cdsp {
> > + firmware-name = "qcom/qcs615/cdsp.mbn";
> > +
> > + status = "okay";
> > +};
> > +
> > +&rpmhcc {
> > + clocks = <&xo_board_clk>;
>
> That's what it says in talos.dtsi already. I must be misunderstanding
> something?
>
> > +};
> > +
> > +&sdhc_1 {
> > + pinctrl-0 = <&sdc1_state_on>;
> > + pinctrl-1 = <&sdc1_state_off>;
> > + pinctrl-names = "default", "sleep";
> > + bus-width = <8>;
> > + mmc-ddr-1_8v;
> > + mmc-hs200-1_8v;
> > + mmc-hs400-1_8v;
> > + mmc-hs400-enhanced-strobe;
> > + vmmc-supply = <&vreg_l17a>;
> > + vqmmc-supply = <&vreg_s4a>;
> > + non-removable;
> > + no-sd;
> > + no-sdio;
> > +
> > + status = "okay";
> > +};
> > +
> > +&sdhc_2 {
> > + pinctrl-0 = <&sdc2_state_on>;
> > + pinctrl-1 = <&sdc2_state_off>;
> > + pinctrl-names = "default", "sleep";
> > + bus-width = <4>;
> > + cd-gpios = <&tlmm 99 GPIO_ACTIVE_LOW>;
> > + vmmc-supply = <&vreg_l10a>;
> > + vqmmc-supply = <&vreg_s4a>;
> > +
> > + status = "okay";
> > +};
> > +
> > +&uart0 {
> > + status = "okay";
> > +};
> > +
> > +&ufs_mem_hc {
> > + reset-gpios = <&tlmm 123 GPIO_ACTIVE_LOW>;
> > + vcc-supply = <&vreg_l17a>;
> > + vcc-max-microamp = <600000>;
> > + vccq2-supply = <&vreg_s4a>;
> > + vccq2-max-microamp = <600000>;
> > +
> > + status = "okay";
> > +};
> > +
> > +&ufs_mem_phy {
> > + vdda-phy-supply = <&vreg_l5a>;
> > + vdda-pll-supply = <&vreg_l12a>;
> > +
> > + status = "okay";
> > +};
> >
> > --
> > 2.34.1
> >