[PATCH v11 0/4] Add Qualcomm Technologies, Inc. Talos EVK SMARC support

[PATCH v11 0/4] Add Qualcomm Technologies, Inc. Talos EVK SMARC support

[Sudarshan Shetty]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=y, Size: 4929 bytes --]

Hi all,

This patch series adds device tree binding and board support for the
Qualcomm Technologies, Inc. Talos EVK SMARC platform based on the
QCS615 SoC.

The first patch introduces the DT binding entry for the Talos EVK
SMARC board, and the second patch adds the corresponding DTS
files for the platform.

Note:
USB(usb_1_dwc3) supports host-only mode based on the switch SW1 on
the SoM, which is purely a hardware controlled as USB-ID and USB-VBUS
is not connected the switching cannot be handled from SW.
Hence from SW Host-only mode is supported on Linux boot up.

Changes in v11:
 - Fixed inconsistent USB PHY node naming in qcs615-ride.dts to
   align with the expected naming convention. (suggested by
   yuanjiey)
 - Reordered properties in property-n and property-names for
   consistency across DT files.
 - Removed unnecessary output-low from the bt_en_state node.
 - Used a &label_reference for hdmi_connector.
 - Added pcieport0 device node, which is required for WLAN to
   power on.
 - No functional change in bindings file.

Changes in v10:
 - This series extends the gpio-backlight driver and its DT bindings
   to support multiple control GPIOs. (suggested by Dmitry)
 - Addressed DT node labeling feedback.
 - Polished overall device tree structure and minor formattings.
 - Corrected pinctrl configuration for the active-low PERST# pin.
 - Includes a fix for inconsistent USB PHY node naming in
   talos.dtsi. (suggested by Dmitry)
 - Enabled WiFi and Bluetooth functionality.
 - Updated commit message.
 - No functional change in bindings file.

Changes in v9:
 - The updates in the following patch url is taken care in this
   patch series.
   https://lists.freedesktop.org/archives/dri-devel/2025-November/534473.html
 - Ignore v8 patch

Changes in v8:
 - This patch series depends on the below url
   https://lists.freedesktop.org/archives/dri-devel/2025-November/534473.html
 - Moved the contents of `talos-evk-cb.dtsi` to `talos-evk.dtsi`
   which will describe all the standard configuration of the EVK.
 - Introduced 'talos-evk-lvds-auo,g133han01.dtso' which has overlay
   to disable HDMI and adds the LVDS-related on top of the base
   dts file. (suggested by Bjorn)
 - Ethernet and Wi-Fi/BT are not (yet) supported.
 - Updated commit message.
 - No functional change in bindings file.

Changes in v7:
 - Added i2c clock frequency for i2c5 node.
 - Moved adv7535 controller section to talos-evk.dts file.
 - Reorder the usb nodes.
 - GPI_DMA nodes added in the device tree files.
 - No functional change in bindings file.

Changes in v6:
 - Clarified description of `talos-evk-cb.dtsi` to explain that it
   defines common carrier board interfaces shared between HDMI and
   LVDS variants, And It’s a single carrier board. HDMI and LVDS
   share the same DSI interface, and only one can be active at a
   time depending on a DIP switch.
 - Added missing newlines before 'status' and between the last property
   and subnode (suggested by Konrad).
 - Updated commit messages for clarity.

Changes in v5:
 - Updated commit message. (suggested by Krzysztof)
 - Introduced generic node name for can, dp, hdmi-bridge. (suggested by
   Krzysztof)
 - Introduced talos-evk-cb.dtsi, which has common carrier board
   interfaces.
 - No functional change in bindings file.

Changes in v4:
 - Updated product name to full form per Krzysztof’s feedback in
   dt-binding comment.
 - Hook up the ADV7535 DSI-to-HDMI bridge to base DTS file.
 - Add DP connector node and MDSS DisplayPort controller.
 - Added USB note in the cover letter for maintainers' awareness.

Changes in v3:
 - Addressed comments from Dmitry regarding USB1 `dr_mode` and
   added a DTS comment.
 - No functional change in bindings file.

Changes in v2:
 - Renamed compatible to "qcom,talos-evk" (suggested by Dmitry/Bjorn)
 - Merged enum entry with existing `qcs615-ride` block (Krzysztof)
 - Fixed subject and commit message to use imperative mood.

Thanks,
Anusha

Sudarshan Shetty (4):
  dt-bindings: arm: qcom: talos-evk: Add QCS615 Talos EVK SMARC platform
  arm64: dts: qcom: talos: Fix inconsistent USB PHY node naming
  arm64: dts: qcom: qcs615-ride: Fix USB PHY label reference
  arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

 .../devicetree/bindings/arm/qcom.yaml         |   1 +
 arch/arm64/boot/dts/qcom/Makefile             |   4 +
 arch/arm64/boot/dts/qcom/qcs615-ride.dts      |   2 +-
 .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
 arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
 arch/arm64/boot/dts/qcom/talos.dtsi           |   4 +-
 7 files changed, 889 insertions(+), 3 deletions(-)
 create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
 create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts

-- 
2.34.1

[PATCH v11 1/4] dt-bindings: arm: qcom: talos-evk: Add QCS615 Talos EVK SMARC platform

[Sudarshan Shetty]

Add binding support for the Qualcomm Technologies, Inc. Talos EVK
SMARC platform based on the QCS615 SoC.

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 4eb0a7a9ee4a..c081746636d1 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -876,6 +876,7 @@ properties:
       - items:
           - enum:
               - qcom,qcs615-ride
+              - qcom,talos-evk
           - const: qcom,qcs615
           - const: qcom,sm6150
 
-- 
2.34.1

[PATCH v11 2/4] arm64: dts: qcom: talos: Fix inconsistent USB PHY node naming

[Sudarshan Shetty]

The USB PHY nodes has inconsistent labels as 'usb_1_hspy'
and 'usb_hsphy_2'. This patch renames them to follow
a consistent naming scheme.

No functional changes, only label renaming.

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
---
 arch/arm64/boot/dts/qcom/talos.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/talos.dtsi b/arch/arm64/boot/dts/qcom/talos.dtsi
index 95d26e313622..c7dc507a50b5 100644
--- a/arch/arm64/boot/dts/qcom/talos.dtsi
+++ b/arch/arm64/boot/dts/qcom/talos.dtsi
@@ -4304,7 +4304,7 @@ osm_l3: interconnect@18321000 {
 			#interconnect-cells = <1>;
 		};
 
-		usb_1_hsphy: phy@88e2000 {
+		usb_hsphy_1: phy@88e2000 {
 			compatible = "qcom,qcs615-qusb2-phy";
 			reg = <0x0 0x88e2000 0x0 0x180>;
 
@@ -4412,7 +4412,7 @@ usb_1_dwc3: usb@a600000 {
 				iommus = <&apps_smmu 0x140 0x0>;
 				interrupts = <GIC_SPI 133 IRQ_TYPE_LEVEL_HIGH>;
 
-				phys = <&usb_1_hsphy>, <&usb_qmpphy>;
+				phys = <&usb_hsphy_1>, <&usb_qmpphy>;
 				phy-names = "usb2-phy", "usb3-phy";
 
 				snps,dis-u1-entry-quirk;
-- 
2.34.1

[PATCH v11 3/4] arm64: dts: qcom: qcs615-ride: Fix USB PHY label reference

[Sudarshan Shetty]

Update the USB high-speed PHY label from 'usb_1_hsphy' to
'usb_hsphy_1' to maintain consistency with the naming
convention used in talos.dtsi, where the second PHY is
labeled 'usb_hsphy_2'.

No functional changes, only label renaming.

Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
---
 arch/arm64/boot/dts/qcom/qcs615-ride.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
index be67eb173046..0ca6b50a6de1 100644
--- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
+++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
@@ -609,7 +609,7 @@ bluetooth {
 	};
 };
 
-&usb_1_hsphy {
+&usb_hsphy_1 {
 	vdd-supply = <&vreg_l5a>;
 	vdda-pll-supply = <&vreg_l12a>;
 	vdda-phy-dpdm-supply = <&vreg_l13a>;
-- 
2.34.1

Re: [PATCH v11 3/4] arm64: dts: qcom: qcs615-ride: Fix USB PHY label reference

[Konrad Dybcio]

On 1/9/26 10:52 AM, Sudarshan Shetty wrote:
> Update the USB high-speed PHY label from 'usb_1_hsphy' to
> 'usb_hsphy_1' to maintain consistency with the naming
> convention used in talos.dtsi, where the second PHY is
> labeled 'usb_hsphy_2'.
> 
> No functional changes, only label renaming.
> 
> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
> ---
>  arch/arm64/boot/dts/qcom/qcs615-ride.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> index be67eb173046..0ca6b50a6de1 100644
> --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> @@ -609,7 +609,7 @@ bluetooth {
>  	};
>  };
>  
> -&usb_1_hsphy {
> +&usb_hsphy_1 {
>  	vdd-supply = <&vreg_l5a>;
>  	vdda-pll-supply = <&vreg_l12a>;
>  	vdda-phy-dpdm-supply = <&vreg_l13a>;

This must be squashed with patch 2, otherwise patch 2 is going to
cause a build failure which must not happen at any point in the
git history (to make bisecting possible)

Konrad

Re: [PATCH v11 3/4] arm64: dts: qcom: qcs615-ride: Fix USB PHY label reference

[tessolveupstream]

On 09-01-2026 15:29, Konrad Dybcio wrote:
> On 1/9/26 10:52 AM, Sudarshan Shetty wrote:
>> Update the USB high-speed PHY label from 'usb_1_hsphy' to
>> 'usb_hsphy_1' to maintain consistency with the naming
>> convention used in talos.dtsi, where the second PHY is
>> labeled 'usb_hsphy_2'.
>>
>> No functional changes, only label renaming.
>>
>> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
>> ---
>>  arch/arm64/boot/dts/qcom/qcs615-ride.dts | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>> index be67eb173046..0ca6b50a6de1 100644
>> --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>> +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>> @@ -609,7 +609,7 @@ bluetooth {
>>  	};
>>  };
>>  
>> -&usb_1_hsphy {
>> +&usb_hsphy_1 {
>>  	vdd-supply = <&vreg_l5a>;
>>  	vdda-pll-supply = <&vreg_l12a>;
>>  	vdda-phy-dpdm-supply = <&vreg_l13a>;
> 
> This must be squashed with patch 2, otherwise patch 2 is going to
> cause a build failure which must not happen at any point in the
> git history (to make bisecting possible)
> 

Okay, I will squash these changes with patch 2 in the next patch 
series.
> Konrad

[PATCH v11 4/4] arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

[Sudarshan Shetty]

Add the device tree for the QCS615-based Talos EVK platform. The
platform is composed of a System-on-Module following the SMARC
standard, and a Carrier Board.

The Carrier Board supports several display configurations, HDMI and
LVDS. Both configurations use the same base hardware, with the display
selection controlled by a DIP switch.

Use a DTBO file, talos-evk-lvds-auo,g133han01.dtso, which defines an
overlay that disables HDMI and adds LVDS. The DTs file talos-evk
can describe the HDMI display configurations.

The initial device tree includes support for:
- CPU and memory
- UART
- GPIOs
- Regulators
- PMIC
- Early console
- AT24MAC602 EEPROM
- MCP2515 SPI to CAN
- ADV7535 DSI-to-HDMI bridge
- DisplayPort interface
- SN65DSI84ZXHR DSI-to-LVDS bridge
- Wi-Fi/BT

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
---
 arch/arm64/boot/dts/qcom/Makefile             |   4 +
 .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
 arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
 4 files changed, 885 insertions(+)
 create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
 create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts

diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index 76cf0115a00a..289b651ef0c5 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -324,6 +324,10 @@ dtb-$(CONFIG_ARCH_QCOM)	+= sm8650-mtp.dtb
 dtb-$(CONFIG_ARCH_QCOM)	+= sm8650-qrd.dtb
 dtb-$(CONFIG_ARCH_QCOM)	+= sm8750-mtp.dtb
 dtb-$(CONFIG_ARCH_QCOM)	+= sm8750-qrd.dtb
+dtb-$(CONFIG_ARCH_QCOM)	+= talos-evk.dtb
+talos-evk-lvds-auo,g133han01-dtbs	:= \
+	talos-evk.dtb talos-evk-lvds-auo,g133han01.dtbo
+dtb-$(CONFIG_ARCH_QCOM)	+= talos-evk-lvds-auo,g133han01.dtb
 x1e001de-devkit-el2-dtbs	:= x1e001de-devkit.dtb x1-el2.dtbo
 dtb-$(CONFIG_ARCH_QCOM)	+= x1e001de-devkit.dtb x1e001de-devkit-el2.dtb
 x1e78100-lenovo-thinkpad-t14s-el2-dtbs	:= x1e78100-lenovo-thinkpad-t14s.dtb x1-el2.dtbo
diff --git a/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
new file mode 100644
index 000000000000..ad058cf4cd93
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
@@ -0,0 +1,126 @@
+// SPDX-License-Identifier: BSD-3-Clause
+/*
+ * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
+ */
+/dts-v1/;
+/plugin/;
+
+#include <dt-bindings/gpio/gpio.h>
+
+&{/} {
+	backlight: backlight {
+		compatible = "gpio-backlight";
+		gpios = <&tlmm 59 GPIO_ACTIVE_HIGH>,
+			<&tlmm 115 GPIO_ACTIVE_HIGH>;
+		default-on;
+	};
+
+	panel-lvds {
+		compatible = "auo,g133han01";
+
+		ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+
+			/* LVDS A (Odd pixels) */
+			port@0 {
+				reg = <0>;
+				dual-lvds-odd-pixels;
+
+				lvds_panel_out_a: endpoint {
+					remote-endpoint = <&sn65dsi84_out_a>;
+				};
+			};
+
+			/* LVDS B (Even pixels) */
+			port@1 {
+				reg = <1>;
+				dual-lvds-even-pixels;
+
+				lvds_panel_out_b: endpoint {
+					remote-endpoint = <&sn65dsi84_out_b>;
+				};
+			};
+		};
+	};
+};
+
+&hdmi_connector {
+	status = "disabled";
+};
+
+&i2c1 {
+	clock-frequency = <400000>;
+
+	status = "okay";
+
+	hdmi_bridge: bridge@3d {
+		status = "disabled";
+	};
+
+	lvds_bridge: bridge@2c {
+		compatible = "ti,sn65dsi84";
+		reg = <0x2c>;
+		enable-gpios = <&tlmm 42 GPIO_ACTIVE_HIGH>;
+		ti,dsi-lanes = <4>;
+		ti,lvds-format = "jeida-24";
+		ti,lvds-bpp = <24>;
+
+		ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+
+			port@0 {
+				reg = <0>;
+
+				sn65dsi84_in: endpoint {
+					data-lanes = <1 2 3 4>;
+					remote-endpoint = <&mdss_dsi0_out>;
+				};
+			};
+
+			port@2 {
+				reg = <2>;
+
+				sn65dsi84_out_a: endpoint {
+					data-lanes = <1 2 3 4>;
+					remote-endpoint = <&lvds_panel_out_a>;
+				};
+			};
+
+			port@3 {
+				reg = <3>;
+
+				sn65dsi84_out_b: endpoint {
+					data-lanes = <1 2 3 4>;
+					remote-endpoint = <&lvds_panel_out_b>;
+				};
+			};
+		};
+	};
+};
+
+&mdss_dsi0 {
+	vdda-supply = <&vreg_l11a>;
+
+	status = "okay";
+};
+
+&mdss_dsi0_out {
+	remote-endpoint = <&sn65dsi84_in>;
+	data-lanes = <0 1 2 3>;
+};
+
+&tlmm {
+	lcd_bklt_en: lcd-bklt-en-state {
+		pins = "gpio115";
+		function = "gpio";
+		bias-disable;
+	};
+
+	lcd_bklt_pwm: lcd-bklt-pwm-state {
+		pins = "gpio59";
+		function = "gpio";
+		bias-disable;
+	};
+};
diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
new file mode 100644
index 000000000000..95ed335bcb08
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
@@ -0,0 +1,616 @@
+// SPDX-License-Identifier: BSD-3-Clause
+/*
+ * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
+ */
+/dts-v1/;
+
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
+#include "talos.dtsi"
+#include "pm8150.dtsi"
+/ {
+	aliases {
+		mmc0 = &sdhc_1;
+		serial0 = &uart0;
+		serial1 = &uart7;
+	};
+
+	chosen {
+		stdout-path = "serial0:115200n8";
+	};
+
+	clocks {
+		can_osc: can-oscillator {
+			compatible = "fixed-clock";
+			clock-frequency = <20000000>;
+			#clock-cells = <0>;
+		};
+
+		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>;
+		};
+	};
+
+	regulator-usb2-vbus {
+		compatible = "regulator-fixed";
+		regulator-name = "USB2_VBUS";
+		gpio = <&pm8150_gpios 10 GPIO_ACTIVE_HIGH>;
+		pinctrl-0 = <&usb2_en>;
+		pinctrl-names = "default";
+		enable-active-high;
+		regulator-always-on;
+	};
+
+	vreg_conn_1p8: regulator-conn-1p8 {
+		compatible = "regulator-fixed";
+		regulator-name = "vreg_conn_1p8";
+		startup-delay-us = <4000>;
+		enable-active-high;
+		gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
+	};
+
+	vreg_conn_pa: regulator-conn-pa {
+		compatible = "regulator-fixed";
+		regulator-name = "vreg_conn_pa";
+		startup-delay-us = <4000>;
+		enable-active-high;
+		gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
+	};
+
+	vreg_v3p3_can: regulator-v3p3-can {
+		compatible = "regulator-fixed";
+		regulator-name = "vreg-v3p3-can";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		regulator-boot-on;
+		regulator-always-on;
+	};
+
+	vreg_v5p0_can: regulator-v5p0-can {
+		compatible = "regulator-fixed";
+		regulator-name = "vreg-v5p0-can";
+		regulator-min-microvolt = <5000000>;
+		regulator-max-microvolt = <5000000>;
+		regulator-boot-on;
+		regulator-always-on;
+	};
+
+	wcn6855-pmu {
+		compatible = "qcom,wcn6855-pmu";
+
+		pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
+		pinctrl-names = "default";
+
+		bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
+		wlan-enable-gpios = <&tlmm 84 GPIO_ACTIVE_HIGH>;
+
+		vddio-supply = <&vreg_conn_pa>;
+		vddaon-supply = <&vreg_s5a>;
+		vddpmu-supply = <&vreg_conn_1p8>;
+		vddpmumx-supply = <&vreg_conn_1p8>;
+		vddpmucx-supply = <&vreg_conn_pa>;
+		vddrfa0p95-supply = <&vreg_s5a>;
+		vddrfa1p3-supply = <&vreg_s6a>;
+		vddrfa1p9-supply = <&vreg_l15a>;
+		vddpcie1p3-supply = <&vreg_s6a>;
+		vddpcie1p9-supply = <&vreg_l15a>;
+
+		regulators {
+			vreg_pmu_rfa_cmn: ldo0 {
+				regulator-name = "vreg_pmu_rfa_cmn";
+			};
+
+			vreg_pmu_aon_0p59: ldo1 {
+				regulator-name = "vreg_pmu_aon_0p59";
+			};
+
+			vreg_pmu_wlcx_0p8: ldo2 {
+				regulator-name = "vreg_pmu_wlcx_0p8";
+			};
+
+			vreg_pmu_wlmx_0p85: ldo3 {
+				regulator-name = "vreg_pmu_wlmx_0p85";
+			};
+
+			vreg_pmu_btcmx_0p85: ldo4 {
+				regulator-name = "vreg_pmu_btcmx_0p85";
+			};
+
+			vreg_pmu_rfa_0p8: ldo5 {
+				regulator-name = "vreg_pmu_rfa_0p8";
+			};
+
+			vreg_pmu_rfa_1p2: ldo6 {
+				regulator-name = "vreg_pmu_rfa_1p2";
+			};
+
+			vreg_pmu_rfa_1p7: ldo7 {
+				regulator-name = "vreg_pmu_rfa_1p7";
+			};
+
+			vreg_pmu_pcie_0p9: ldo8 {
+				regulator-name = "vreg_pmu_pcie_0p9";
+			};
+
+			vreg_pmu_pcie_1p8: ldo9 {
+				regulator-name = "vreg_pmu_pcie_1p8";
+			};
+		};
+	};
+
+	wifi_1p8v: regulator-wifi-1p8v {
+		compatible = "regulator-fixed";
+		regulator-name = "wifi_1p8v";
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+		gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+		pinctrl-0 = <&wifi_reg_en_pins_state>;
+		pinctrl-names = "default";
+		regulator-boot-on;
+		regulator-always-on;
+	};
+
+	wifi_3p85v: regulator-wifi-3p85v {
+		compatible = "regulator-fixed";
+		regulator-name = "wifi_3p85v";
+		regulator-min-microvolt = <3850000>;
+		regulator-max-microvolt = <3850000>;
+		gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+		pinctrl-0 = <&wifi_reg_en_pins_state>;
+		pinctrl-names = "default";
+		regulator-boot-on;
+		regulator-always-on;
+	};
+};
+
+&apps_rsc {
+	regulators-0 {
+		compatible = "qcom,pm8150-rpmh-regulators";
+		qcom,pmic-id = "a";
+
+		vreg_s3a: smps3 {
+			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>;
+			regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
+			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>;
+		};
+	};
+};
+
+&gpi_dma0 {
+	status = "okay";
+};
+
+&gpi_dma1 {
+	status = "okay";
+};
+
+&i2c5 {
+	clock-frequency = <400000>;
+	status = "okay";
+
+	eeprom@57 {
+		compatible = "atmel,24c02";
+		reg = <0x57>;
+		pagesize = <16>;
+	};
+
+	eeprom@5f {
+		compatible = "atmel,24mac602";
+		reg = <0x5f>;
+		pagesize = <16>;
+	};
+};
+
+&mdss {
+	status = "okay";
+};
+
+&mdss_dp0 {
+	status = "okay";
+};
+
+&mdss_dp0_out {
+	link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000>;
+	remote-endpoint = <&dp0_connector_in>;
+};
+
+&mdss_dsi0 {
+	vdda-supply = <&vreg_l11a>;
+	status = "okay";
+};
+
+&mdss_dsi0_phy {
+	vcca-supply = <&vreg_l5a>;
+	status = "okay";
+};
+
+&pcie {
+	perst-gpios = <&tlmm 89 GPIO_ACTIVE_LOW>;
+	wake-gpios = <&tlmm 100 GPIO_ACTIVE_HIGH>;
+
+	pinctrl-0 = <&pcie_default_state>;
+	pinctrl-names = "default";
+
+	status = "okay";
+};
+
+&pcie_phy {
+	vdda-phy-supply = <&vreg_l5a>;
+	vdda-pll-supply = <&vreg_l12a>;
+
+	status = "okay";
+};
+
+&pcie_port0 {
+	wifi@0 {
+		compatible = "pci17cb,1103";
+		reg = <0x10000 0x0 0x0 0x0 0x0>;
+
+		qcom,calibration-variant = "QC_QCS615_Ride";
+
+		vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
+		vddaon-supply = <&vreg_pmu_aon_0p59>;
+		vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
+		vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
+		vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
+		vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
+		vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
+		vddpcie0p9-supply = <&vreg_pmu_pcie_0p9>;
+		vddpcie1p8-supply = <&vreg_pmu_pcie_1p8>;
+	};
+};
+
+&pm8150_gpios {
+	usb2_en: usb2-en-state {
+		pins = "gpio10";
+		function = "normal";
+		output-enable;
+		power-source = <0>;
+	};
+};
+
+&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";
+};
+
+&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";
+};
+
+&spi6 {
+	status = "okay";
+
+	can@0 {
+		compatible = "microchip,mcp2515";
+		reg = <0>;
+		clocks = <&can_osc>;
+		interrupts-extended = <&tlmm 87 IRQ_TYPE_LEVEL_LOW>;
+		spi-max-frequency = <10000000>;
+		vdd-supply = <&vreg_v3p3_can>;
+		xceiver-supply = <&vreg_v5p0_can>;
+	};
+};
+
+&tlmm {
+	bt_en_state: bt-en-state {
+		pins = "gpio85";
+		function = "gpio";
+		bias-pull-down;
+	};
+
+	pcie_default_state: pcie-default-state {
+		clkreq-pins {
+			pins = "gpio90";
+			function = "pcie_clk_req";
+			drive-strength = <2>;
+			bias-pull-up;
+		};
+
+		perst-pins {
+			pins = "gpio89";
+			function = "gpio";
+			drive-strength = <2>;
+			bias-disable;
+		};
+
+		wake-pins {
+			pins = "gpio100";
+			function = "gpio";
+			drive-strength = <2>;
+			bias-pull-up;
+		};
+	};
+
+	wifi_reg_en_pins_state: wifi-reg-en-pins-state {
+		pins = "gpio91";
+		function = "gpio";
+		drive-strength = <8>;
+		output-high;
+		bias-pull-up;
+	};
+
+	wlan_en_state: wlan-en-state {
+		pins = "gpio84";
+		function = "gpio";
+		drive-strength = <16>;
+		bias-pull-down;
+		output-low;
+	};
+};
+
+&uart0 {
+	status = "okay";
+};
+
+&uart7 {
+	status = "okay";
+
+	bluetooth {
+		compatible = "qcom,wcn6855-bt";
+		firmware-name = "QCA6698/hpnv21", "QCA6698/hpbtfw21.tlv";
+
+		vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
+		vddaon-supply = <&vreg_pmu_aon_0p59>;
+		vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
+		vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
+		vddbtcmx-supply = <&vreg_pmu_btcmx_0p85>;
+		vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
+		vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
+		vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
+	};
+};
+
+&usb_1 {
+	status = "okay";
+};
+
+&usb_1_dwc3 {
+	dr_mode = "host";
+};
+
+&usb_hsphy_1 {
+	vdd-supply = <&vreg_l5a>;
+	vdda-pll-supply = <&vreg_l12a>;
+	vdda-phy-dpdm-supply = <&vreg_l13a>;
+
+	status = "okay";
+};
+
+&usb_2 {
+	status = "okay";
+};
+
+&usb_2_dwc3 {
+	dr_mode = "host";
+};
+
+&usb_hsphy_2 {
+	vdd-supply = <&vreg_l5a>;
+	vdda-pll-supply = <&vreg_l12a>;
+	vdda-phy-dpdm-supply = <&vreg_l13a>;
+
+	status = "okay";
+};
+
+&usb_qmpphy {
+	vdda-phy-supply = <&vreg_l5a>;
+	vdda-pll-supply = <&vreg_l12a>;
+
+	status = "okay";
+};
+
+&usb_qmpphy_2 {
+	vdda-phy-supply = <&vreg_l11a>;
+	vdda-pll-supply = <&vreg_l5a>;
+
+	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";
+};
+
+&venus {
+	status = "okay";
+};
diff --git a/arch/arm64/boot/dts/qcom/talos-evk.dts b/arch/arm64/boot/dts/qcom/talos-evk.dts
new file mode 100644
index 000000000000..af100e22beee
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/talos-evk.dts
@@ -0,0 +1,139 @@
+// SPDX-License-Identifier: BSD-3-Clause
+/*
+ * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
+ */
+/dts-v1/;
+
+#include "talos-evk-som.dtsi"
+
+/ {
+	model = "Qualcomm QCS615 IQ 615 EVK";
+	compatible = "qcom,talos-evk", "qcom,qcs615", "qcom,sm6150";
+	chassis-type = "embedded";
+
+	aliases {
+		mmc1 = &sdhc_2;
+	};
+
+	dp0-connector {
+		compatible = "dp-connector";
+		label = "DP0";
+		type = "full-size";
+
+		hpd-gpios = <&tlmm 104 GPIO_ACTIVE_HIGH>;
+
+		port {
+			dp0_connector_in: endpoint {
+				remote-endpoint = <&mdss_dp0_out>;
+			};
+		};
+	};
+
+	hdmi_connector: hdmi-out {
+		compatible = "hdmi-connector";
+		type = "d";
+
+		port {
+			hdmi_con_out: endpoint {
+			remote-endpoint = <&adv7535_out>;
+			};
+		};
+	};
+
+	vreg_v1p8_out: regulator-v1p8-out {
+		compatible = "regulator-fixed";
+		regulator-name = "vreg-v1p8-out";
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+		vin-supply = <&vreg_v5p0_out>;
+		regulator-boot-on;
+		regulator-always-on;
+	};
+
+	vreg_v3p3_out: regulator-v3p3-out {
+		compatible = "regulator-fixed";
+		regulator-name = "vreg-v3p3-out";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		vin-supply = <&vreg_v5p0_out>;
+		regulator-boot-on;
+		regulator-always-on;
+	};
+
+	vreg_v5p0_out: regulator-v5p0-out {
+		compatible = "regulator-fixed";
+		regulator-name = "vreg-v5p0-out";
+		regulator-min-microvolt = <5000000>;
+		regulator-max-microvolt = <5000000>;
+		regulator-boot-on;
+		regulator-always-on;
+		/* Powered by system 20V rail (USBC_VBUS_IN) */
+	};
+};
+
+&i2c1 {
+	clock-frequency = <400000>;
+	status = "okay";
+
+	hdmi_bridge: bridge@3d {
+		compatible = "adi,adv7535";
+		reg = <0x3d>;
+		avdd-supply = <&vreg_v1p8_out>;
+		dvdd-supply = <&vreg_v1p8_out>;
+		pvdd-supply = <&vreg_v1p8_out>;
+		a2vdd-supply = <&vreg_v1p8_out>;
+		v3p3-supply = <&vreg_v3p3_out>;
+		interrupts-extended = <&tlmm 26 IRQ_TYPE_LEVEL_LOW>;
+		adi,dsi-lanes = <4>;
+
+		ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+
+			port@0 {
+				reg = <0>;
+
+				adv7535_in: endpoint {
+					remote-endpoint = <&mdss_dsi0_out>;
+				};
+			};
+
+			port@1 {
+				reg = <1>;
+
+				adv7535_out: endpoint {
+					remote-endpoint = <&hdmi_con_out>;
+				};
+			};
+		};
+	};
+};
+
+&mdss_dsi0_out {
+	remote-endpoint = <&adv7535_in>;
+	data-lanes = <0 1 2 3>;
+};
+
+&pon_pwrkey {
+	status = "okay";
+};
+
+&pon_resin {
+	linux,code = <KEY_VOLUMEDOWN>;
+
+	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";
+};
-- 
2.34.1

Re: [PATCH v11 4/4] arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

[yuanjiey]

On Fri, Jan 09, 2026 at 03:22:20PM +0530, Sudarshan Shetty wrote:
> Add the device tree for the QCS615-based Talos EVK platform. The
> platform is composed of a System-on-Module following the SMARC
> standard, and a Carrier Board.
> 
> The Carrier Board supports several display configurations, HDMI and
> LVDS. Both configurations use the same base hardware, with the display
> selection controlled by a DIP switch.
> 
> Use a DTBO file, talos-evk-lvds-auo,g133han01.dtso, which defines an
> overlay that disables HDMI and adds LVDS. The DTs file talos-evk
> can describe the HDMI display configurations.
> 
> The initial device tree includes support for:
> - CPU and memory
> - UART
> - GPIOs
> - Regulators
> - PMIC
> - Early console
> - AT24MAC602 EEPROM
> - MCP2515 SPI to CAN
> - ADV7535 DSI-to-HDMI bridge
> - DisplayPort interface
> - SN65DSI84ZXHR DSI-to-LVDS bridge
> - Wi-Fi/BT
> 
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
> ---
>  arch/arm64/boot/dts/qcom/Makefile             |   4 +
>  .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
>  arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
>  arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
>  4 files changed, 885 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>  create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>  create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts
> 
> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> index 76cf0115a00a..289b651ef0c5 100644
> --- a/arch/arm64/boot/dts/qcom/Makefile
> +++ b/arch/arm64/boot/dts/qcom/Makefile
> @@ -324,6 +324,10 @@ dtb-$(CONFIG_ARCH_QCOM)	+= sm8650-mtp.dtb
>  dtb-$(CONFIG_ARCH_QCOM)	+= sm8650-qrd.dtb
>  dtb-$(CONFIG_ARCH_QCOM)	+= sm8750-mtp.dtb
>  dtb-$(CONFIG_ARCH_QCOM)	+= sm8750-qrd.dtb
> +dtb-$(CONFIG_ARCH_QCOM)	+= talos-evk.dtb
> +talos-evk-lvds-auo,g133han01-dtbs	:= \
> +	talos-evk.dtb talos-evk-lvds-auo,g133han01.dtbo
> +dtb-$(CONFIG_ARCH_QCOM)	+= talos-evk-lvds-auo,g133han01.dtb
>  x1e001de-devkit-el2-dtbs	:= x1e001de-devkit.dtb x1-el2.dtbo
>  dtb-$(CONFIG_ARCH_QCOM)	+= x1e001de-devkit.dtb x1e001de-devkit-el2.dtb
>  x1e78100-lenovo-thinkpad-t14s-el2-dtbs	:= x1e78100-lenovo-thinkpad-t14s.dtb x1-el2.dtbo
> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> new file mode 100644
> index 000000000000..ad058cf4cd93
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> @@ -0,0 +1,126 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> + */
> +/dts-v1/;
> +/plugin/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +
> +&{/} {
> +	backlight: backlight {
> +		compatible = "gpio-backlight";
> +		gpios = <&tlmm 59 GPIO_ACTIVE_HIGH>,
> +			<&tlmm 115 GPIO_ACTIVE_HIGH>;
> +		default-on;
> +	};
> +
> +	panel-lvds {
> +		compatible = "auo,g133han01";
> +
> +		ports {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +
> +			/* LVDS A (Odd pixels) */
> +			port@0 {
> +				reg = <0>;
> +				dual-lvds-odd-pixels;
> +
> +				lvds_panel_out_a: endpoint {
> +					remote-endpoint = <&sn65dsi84_out_a>;
> +				};
> +			};
> +
> +			/* LVDS B (Even pixels) */
> +			port@1 {
> +				reg = <1>;
> +				dual-lvds-even-pixels;
> +
> +				lvds_panel_out_b: endpoint {
> +					remote-endpoint = <&sn65dsi84_out_b>;
> +				};
> +			};
> +		};
> +	};
> +};
> +
> +&hdmi_connector {
> +	status = "disabled";
> +};
> +
> +&i2c1 {
> +	clock-frequency = <400000>;
> +
> +	status = "okay";
> +
> +	hdmi_bridge: bridge@3d {
> +		status = "disabled";
> +	};
> +
> +	lvds_bridge: bridge@2c {
power-supply, if be optional, need to modify dt-binding file.
otherwise we will meet binding check error.

> +		compatible = "ti,sn65dsi84";
> +		reg = <0x2c>;
> +		enable-gpios = <&tlmm 42 GPIO_ACTIVE_HIGH>;
> +		ti,dsi-lanes = <4>;
> +		ti,lvds-format = "jeida-24";
> +		ti,lvds-bpp = <24>;

ti,dsi-lanes, ti,lvds-format, ti,lvds-bpp, do we need these property?
I don't see these property be used from  kernel 6.6 to kernel 6.19. 


Thanks,
Yuanjie


> +		ports {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +
> +			port@0 {
> +				reg = <0>;
> +
> +				sn65dsi84_in: endpoint {
> +					data-lanes = <1 2 3 4>;
> +					remote-endpoint = <&mdss_dsi0_out>;
> +				};
> +			};
> +
> +			port@2 {
> +				reg = <2>;
> +
> +				sn65dsi84_out_a: endpoint {
> +					data-lanes = <1 2 3 4>;
> +					remote-endpoint = <&lvds_panel_out_a>;
> +				};
> +			};
> +
> +			port@3 {
> +				reg = <3>;
> +
> +				sn65dsi84_out_b: endpoint {
> +					data-lanes = <1 2 3 4>;
> +					remote-endpoint = <&lvds_panel_out_b>;
> +				};
> +			};
> +		};
> +	};
> +};
> +
> +&mdss_dsi0 {
> +	vdda-supply = <&vreg_l11a>;
> +
> +	status = "okay";
> +};
> +
> +&mdss_dsi0_out {
> +	remote-endpoint = <&sn65dsi84_in>;
> +	data-lanes = <0 1 2 3>;
> +};
> +
> +&tlmm {
> +	lcd_bklt_en: lcd-bklt-en-state {
> +		pins = "gpio115";
> +		function = "gpio";
> +		bias-disable;
> +	};
> +
> +	lcd_bklt_pwm: lcd-bklt-pwm-state {
> +		pins = "gpio59";
> +		function = "gpio";
> +		bias-disable;
> +	};
> +};
> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> new file mode 100644
> index 000000000000..95ed335bcb08
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> @@ -0,0 +1,616 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> + */
> +/dts-v1/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> +#include "talos.dtsi"
> +#include "pm8150.dtsi"
> +/ {
> +	aliases {
> +		mmc0 = &sdhc_1;
> +		serial0 = &uart0;
> +		serial1 = &uart7;
> +	};
> +
> +	chosen {
> +		stdout-path = "serial0:115200n8";
> +	};
> +
> +	clocks {
> +		can_osc: can-oscillator {
> +			compatible = "fixed-clock";
> +			clock-frequency = <20000000>;
> +			#clock-cells = <0>;
> +		};
> +
> +		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>;
> +		};
> +	};
> +
> +	regulator-usb2-vbus {
> +		compatible = "regulator-fixed";
> +		regulator-name = "USB2_VBUS";
> +		gpio = <&pm8150_gpios 10 GPIO_ACTIVE_HIGH>;
> +		pinctrl-0 = <&usb2_en>;
> +		pinctrl-names = "default";
> +		enable-active-high;
> +		regulator-always-on;
> +	};
> +
> +	vreg_conn_1p8: regulator-conn-1p8 {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg_conn_1p8";
> +		startup-delay-us = <4000>;
> +		enable-active-high;
> +		gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
> +	};
> +
> +	vreg_conn_pa: regulator-conn-pa {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg_conn_pa";
> +		startup-delay-us = <4000>;
> +		enable-active-high;
> +		gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
> +	};
> +
> +	vreg_v3p3_can: regulator-v3p3-can {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg-v3p3-can";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +
> +	vreg_v5p0_can: regulator-v5p0-can {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg-v5p0-can";
> +		regulator-min-microvolt = <5000000>;
> +		regulator-max-microvolt = <5000000>;
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +
> +	wcn6855-pmu {
> +		compatible = "qcom,wcn6855-pmu";
> +
> +		pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
> +		pinctrl-names = "default";
> +
> +		bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
> +		wlan-enable-gpios = <&tlmm 84 GPIO_ACTIVE_HIGH>;
> +
> +		vddio-supply = <&vreg_conn_pa>;
> +		vddaon-supply = <&vreg_s5a>;
> +		vddpmu-supply = <&vreg_conn_1p8>;
> +		vddpmumx-supply = <&vreg_conn_1p8>;
> +		vddpmucx-supply = <&vreg_conn_pa>;
> +		vddrfa0p95-supply = <&vreg_s5a>;
> +		vddrfa1p3-supply = <&vreg_s6a>;
> +		vddrfa1p9-supply = <&vreg_l15a>;
> +		vddpcie1p3-supply = <&vreg_s6a>;
> +		vddpcie1p9-supply = <&vreg_l15a>;
> +
> +		regulators {
> +			vreg_pmu_rfa_cmn: ldo0 {
> +				regulator-name = "vreg_pmu_rfa_cmn";
> +			};
> +
> +			vreg_pmu_aon_0p59: ldo1 {
> +				regulator-name = "vreg_pmu_aon_0p59";
> +			};
> +
> +			vreg_pmu_wlcx_0p8: ldo2 {
> +				regulator-name = "vreg_pmu_wlcx_0p8";
> +			};
> +
> +			vreg_pmu_wlmx_0p85: ldo3 {
> +				regulator-name = "vreg_pmu_wlmx_0p85";
> +			};
> +
> +			vreg_pmu_btcmx_0p85: ldo4 {
> +				regulator-name = "vreg_pmu_btcmx_0p85";
> +			};
> +
> +			vreg_pmu_rfa_0p8: ldo5 {
> +				regulator-name = "vreg_pmu_rfa_0p8";
> +			};
> +
> +			vreg_pmu_rfa_1p2: ldo6 {
> +				regulator-name = "vreg_pmu_rfa_1p2";
> +			};
> +
> +			vreg_pmu_rfa_1p7: ldo7 {
> +				regulator-name = "vreg_pmu_rfa_1p7";
> +			};
> +
> +			vreg_pmu_pcie_0p9: ldo8 {
> +				regulator-name = "vreg_pmu_pcie_0p9";
> +			};
> +
> +			vreg_pmu_pcie_1p8: ldo9 {
> +				regulator-name = "vreg_pmu_pcie_1p8";
> +			};
> +		};
> +	};
> +
> +	wifi_1p8v: regulator-wifi-1p8v {
> +		compatible = "regulator-fixed";
> +		regulator-name = "wifi_1p8v";
> +		regulator-min-microvolt = <1800000>;
> +		regulator-max-microvolt = <1800000>;
> +		gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> +		enable-active-high;
> +		pinctrl-0 = <&wifi_reg_en_pins_state>;
> +		pinctrl-names = "default";
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +
> +	wifi_3p85v: regulator-wifi-3p85v {
> +		compatible = "regulator-fixed";
> +		regulator-name = "wifi_3p85v";
> +		regulator-min-microvolt = <3850000>;
> +		regulator-max-microvolt = <3850000>;
> +		gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> +		enable-active-high;
> +		pinctrl-0 = <&wifi_reg_en_pins_state>;
> +		pinctrl-names = "default";
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +};
> +
> +&apps_rsc {
> +	regulators-0 {
> +		compatible = "qcom,pm8150-rpmh-regulators";
> +		qcom,pmic-id = "a";
> +
> +		vreg_s3a: smps3 {
> +			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>;
> +			regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
> +			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>;
> +		};
> +	};
> +};
> +
> +&gpi_dma0 {
> +	status = "okay";
> +};
> +
> +&gpi_dma1 {
> +	status = "okay";
> +};
> +
> +&i2c5 {
> +	clock-frequency = <400000>;
> +	status = "okay";
> +
> +	eeprom@57 {
> +		compatible = "atmel,24c02";
> +		reg = <0x57>;
> +		pagesize = <16>;
> +	};
> +
> +	eeprom@5f {
> +		compatible = "atmel,24mac602";
> +		reg = <0x5f>;
> +		pagesize = <16>;
> +	};
> +};
> +
> +&mdss {
> +	status = "okay";
> +};
> +
> +&mdss_dp0 {
> +	status = "okay";
> +};
> +
> +&mdss_dp0_out {
> +	link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000>;
> +	remote-endpoint = <&dp0_connector_in>;
> +};
> +
> +&mdss_dsi0 {
> +	vdda-supply = <&vreg_l11a>;
> +	status = "okay";
> +};
> +
> +&mdss_dsi0_phy {
> +	vcca-supply = <&vreg_l5a>;
> +	status = "okay";
> +};
> +
> +&pcie {
> +	perst-gpios = <&tlmm 89 GPIO_ACTIVE_LOW>;
> +	wake-gpios = <&tlmm 100 GPIO_ACTIVE_HIGH>;
> +
> +	pinctrl-0 = <&pcie_default_state>;
> +	pinctrl-names = "default";
> +
> +	status = "okay";
> +};
> +
> +&pcie_phy {
> +	vdda-phy-supply = <&vreg_l5a>;
> +	vdda-pll-supply = <&vreg_l12a>;
> +
> +	status = "okay";
> +};
> +
> +&pcie_port0 {
> +	wifi@0 {
> +		compatible = "pci17cb,1103";
> +		reg = <0x10000 0x0 0x0 0x0 0x0>;
> +
> +		qcom,calibration-variant = "QC_QCS615_Ride";
> +
> +		vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
> +		vddaon-supply = <&vreg_pmu_aon_0p59>;
> +		vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
> +		vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
> +		vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
> +		vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
> +		vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
> +		vddpcie0p9-supply = <&vreg_pmu_pcie_0p9>;
> +		vddpcie1p8-supply = <&vreg_pmu_pcie_1p8>;
> +	};
> +};
> +
> +&pm8150_gpios {
> +	usb2_en: usb2-en-state {
> +		pins = "gpio10";
> +		function = "normal";
> +		output-enable;
> +		power-source = <0>;
> +	};
> +};
> +
> +&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";
> +};
> +
> +&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";
> +};
> +
> +&spi6 {
> +	status = "okay";
> +
> +	can@0 {
> +		compatible = "microchip,mcp2515";
> +		reg = <0>;
> +		clocks = <&can_osc>;
> +		interrupts-extended = <&tlmm 87 IRQ_TYPE_LEVEL_LOW>;
> +		spi-max-frequency = <10000000>;
> +		vdd-supply = <&vreg_v3p3_can>;
> +		xceiver-supply = <&vreg_v5p0_can>;
> +	};
> +};
> +
> +&tlmm {
> +	bt_en_state: bt-en-state {
> +		pins = "gpio85";
> +		function = "gpio";
> +		bias-pull-down;
> +	};
> +
> +	pcie_default_state: pcie-default-state {
> +		clkreq-pins {
> +			pins = "gpio90";
> +			function = "pcie_clk_req";
> +			drive-strength = <2>;
> +			bias-pull-up;
> +		};
> +
> +		perst-pins {
> +			pins = "gpio89";
> +			function = "gpio";
> +			drive-strength = <2>;
> +			bias-disable;
> +		};
> +
> +		wake-pins {
> +			pins = "gpio100";
> +			function = "gpio";
> +			drive-strength = <2>;
> +			bias-pull-up;
> +		};
> +	};
> +
> +	wifi_reg_en_pins_state: wifi-reg-en-pins-state {
> +		pins = "gpio91";
> +		function = "gpio";
> +		drive-strength = <8>;
> +		output-high;
> +		bias-pull-up;
> +	};
> +
> +	wlan_en_state: wlan-en-state {
> +		pins = "gpio84";
> +		function = "gpio";
> +		drive-strength = <16>;
> +		bias-pull-down;
> +		output-low;
> +	};
> +};
> +
> +&uart0 {
> +	status = "okay";
> +};
> +
> +&uart7 {
> +	status = "okay";
> +
> +	bluetooth {
> +		compatible = "qcom,wcn6855-bt";
> +		firmware-name = "QCA6698/hpnv21", "QCA6698/hpbtfw21.tlv";
> +
> +		vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
> +		vddaon-supply = <&vreg_pmu_aon_0p59>;
> +		vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
> +		vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
> +		vddbtcmx-supply = <&vreg_pmu_btcmx_0p85>;
> +		vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
> +		vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
> +		vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
> +	};
> +};
> +
> +&usb_1 {
> +	status = "okay";
> +};
> +
> +&usb_1_dwc3 {
> +	dr_mode = "host";
> +};
> +
> +&usb_hsphy_1 {
> +	vdd-supply = <&vreg_l5a>;
> +	vdda-pll-supply = <&vreg_l12a>;
> +	vdda-phy-dpdm-supply = <&vreg_l13a>;
> +
> +	status = "okay";
> +};
> +
> +&usb_2 {
> +	status = "okay";
> +};
> +
> +&usb_2_dwc3 {
> +	dr_mode = "host";
> +};
> +
> +&usb_hsphy_2 {
> +	vdd-supply = <&vreg_l5a>;
> +	vdda-pll-supply = <&vreg_l12a>;
> +	vdda-phy-dpdm-supply = <&vreg_l13a>;
> +
> +	status = "okay";
> +};
> +
> +&usb_qmpphy {
> +	vdda-phy-supply = <&vreg_l5a>;
> +	vdda-pll-supply = <&vreg_l12a>;
> +
> +	status = "okay";
> +};
> +
> +&usb_qmpphy_2 {
> +	vdda-phy-supply = <&vreg_l11a>;
> +	vdda-pll-supply = <&vreg_l5a>;
> +
> +	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";
> +};
> +
> +&venus {
> +	status = "okay";
> +};
> diff --git a/arch/arm64/boot/dts/qcom/talos-evk.dts b/arch/arm64/boot/dts/qcom/talos-evk.dts
> new file mode 100644
> index 000000000000..af100e22beee
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/talos-evk.dts
> @@ -0,0 +1,139 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> + */
> +/dts-v1/;
> +
> +#include "talos-evk-som.dtsi"
> +
> +/ {
> +	model = "Qualcomm QCS615 IQ 615 EVK";
> +	compatible = "qcom,talos-evk", "qcom,qcs615", "qcom,sm6150";
> +	chassis-type = "embedded";
> +
> +	aliases {
> +		mmc1 = &sdhc_2;
> +	};
> +
> +	dp0-connector {
> +		compatible = "dp-connector";
> +		label = "DP0";
> +		type = "full-size";
> +
> +		hpd-gpios = <&tlmm 104 GPIO_ACTIVE_HIGH>;
> +
> +		port {
> +			dp0_connector_in: endpoint {
> +				remote-endpoint = <&mdss_dp0_out>;
> +			};
> +		};
> +	};
> +
> +	hdmi_connector: hdmi-out {
> +		compatible = "hdmi-connector";
> +		type = "d";
> +
> +		port {
> +			hdmi_con_out: endpoint {
> +			remote-endpoint = <&adv7535_out>;
> +			};
> +		};
> +	};
> +
> +	vreg_v1p8_out: regulator-v1p8-out {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg-v1p8-out";
> +		regulator-min-microvolt = <1800000>;
> +		regulator-max-microvolt = <1800000>;
> +		vin-supply = <&vreg_v5p0_out>;
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +
> +	vreg_v3p3_out: regulator-v3p3-out {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg-v3p3-out";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		vin-supply = <&vreg_v5p0_out>;
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +
> +	vreg_v5p0_out: regulator-v5p0-out {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg-v5p0-out";
> +		regulator-min-microvolt = <5000000>;
> +		regulator-max-microvolt = <5000000>;
> +		regulator-boot-on;
> +		regulator-always-on;
> +		/* Powered by system 20V rail (USBC_VBUS_IN) */
> +	};
> +};
> +
> +&i2c1 {
> +	clock-frequency = <400000>;
> +	status = "okay";
> +
> +	hdmi_bridge: bridge@3d {
> +		compatible = "adi,adv7535";
> +		reg = <0x3d>;
> +		avdd-supply = <&vreg_v1p8_out>;
> +		dvdd-supply = <&vreg_v1p8_out>;
> +		pvdd-supply = <&vreg_v1p8_out>;
> +		a2vdd-supply = <&vreg_v1p8_out>;
> +		v3p3-supply = <&vreg_v3p3_out>;
> +		interrupts-extended = <&tlmm 26 IRQ_TYPE_LEVEL_LOW>;
> +		adi,dsi-lanes = <4>;
> +
> +		ports {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +
> +			port@0 {
> +				reg = <0>;
> +
> +				adv7535_in: endpoint {
> +					remote-endpoint = <&mdss_dsi0_out>;
> +				};
> +			};
> +
> +			port@1 {
> +				reg = <1>;
> +
> +				adv7535_out: endpoint {
> +					remote-endpoint = <&hdmi_con_out>;
> +				};
> +			};
> +		};
> +	};
> +};
> +
> +&mdss_dsi0_out {
> +	remote-endpoint = <&adv7535_in>;
> +	data-lanes = <0 1 2 3>;
> +};
> +
> +&pon_pwrkey {
> +	status = "okay";
> +};
> +
> +&pon_resin {
> +	linux,code = <KEY_VOLUMEDOWN>;
> +
> +	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";
> +};
> -- 
> 2.34.1

Re: [PATCH v11 4/4] arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

[tessolveupstream]

On 12-01-2026 08:48, yuanjiey wrote:
> On Fri, Jan 09, 2026 at 03:22:20PM +0530, Sudarshan Shetty wrote:
>> Add the device tree for the QCS615-based Talos EVK platform. The
>> platform is composed of a System-on-Module following the SMARC
>> standard, and a Carrier Board.
>>
>> The Carrier Board supports several display configurations, HDMI and
>> LVDS. Both configurations use the same base hardware, with the display
>> selection controlled by a DIP switch.
>>
>> Use a DTBO file, talos-evk-lvds-auo,g133han01.dtso, which defines an
>> overlay that disables HDMI and adds LVDS. The DTs file talos-evk
>> can describe the HDMI display configurations.
>>
>> The initial device tree includes support for:
>> - CPU and memory
>> - UART
>> - GPIOs
>> - Regulators
>> - PMIC
>> - Early console
>> - AT24MAC602 EEPROM
>> - MCP2515 SPI to CAN
>> - ADV7535 DSI-to-HDMI bridge
>> - DisplayPort interface
>> - SN65DSI84ZXHR DSI-to-LVDS bridge
>> - Wi-Fi/BT
>>
>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
>> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
>> ---
>>  arch/arm64/boot/dts/qcom/Makefile             |   4 +
>>  .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
>>  arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
>>  arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
>>  4 files changed, 885 insertions(+)
>>  create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>>  create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>>  create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts
>>
>> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
>> index 76cf0115a00a..289b651ef0c5 100644
>> --- a/arch/arm64/boot/dts/qcom/Makefile
>> +++ b/arch/arm64/boot/dts/qcom/Makefile
>> @@ -324,6 +324,10 @@ dtb-$(CONFIG_ARCH_QCOM)	+= sm8650-mtp.dtb
>>  dtb-$(CONFIG_ARCH_QCOM)	+= sm8650-qrd.dtb
>>  dtb-$(CONFIG_ARCH_QCOM)	+= sm8750-mtp.dtb
>>  dtb-$(CONFIG_ARCH_QCOM)	+= sm8750-qrd.dtb
>> +dtb-$(CONFIG_ARCH_QCOM)	+= talos-evk.dtb
>> +talos-evk-lvds-auo,g133han01-dtbs	:= \
>> +	talos-evk.dtb talos-evk-lvds-auo,g133han01.dtbo
>> +dtb-$(CONFIG_ARCH_QCOM)	+= talos-evk-lvds-auo,g133han01.dtb
>>  x1e001de-devkit-el2-dtbs	:= x1e001de-devkit.dtb x1-el2.dtbo
>>  dtb-$(CONFIG_ARCH_QCOM)	+= x1e001de-devkit.dtb x1e001de-devkit-el2.dtb
>>  x1e78100-lenovo-thinkpad-t14s-el2-dtbs	:= x1e78100-lenovo-thinkpad-t14s.dtb x1-el2.dtbo
>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>> new file mode 100644
>> index 000000000000..ad058cf4cd93
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>> @@ -0,0 +1,126 @@
>> +// SPDX-License-Identifier: BSD-3-Clause
>> +/*
>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>> + */
>> +/dts-v1/;
>> +/plugin/;
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +
>> +&{/} {
>> +	backlight: backlight {
>> +		compatible = "gpio-backlight";
>> +		gpios = <&tlmm 59 GPIO_ACTIVE_HIGH>,
>> +			<&tlmm 115 GPIO_ACTIVE_HIGH>;
>> +		default-on;
>> +	};
>> +
>> +	panel-lvds {
>> +		compatible = "auo,g133han01";
>> +
>> +		ports {
>> +			#address-cells = <1>;
>> +			#size-cells = <0>;
>> +
>> +			/* LVDS A (Odd pixels) */
>> +			port@0 {
>> +				reg = <0>;
>> +				dual-lvds-odd-pixels;
>> +
>> +				lvds_panel_out_a: endpoint {
>> +					remote-endpoint = <&sn65dsi84_out_a>;
>> +				};
>> +			};
>> +
>> +			/* LVDS B (Even pixels) */
>> +			port@1 {
>> +				reg = <1>;
>> +				dual-lvds-even-pixels;
>> +
>> +				lvds_panel_out_b: endpoint {
>> +					remote-endpoint = <&sn65dsi84_out_b>;
>> +				};
>> +			};
>> +		};
>> +	};
>> +};
>> +
>> +&hdmi_connector {
>> +	status = "disabled";
>> +};
>> +
>> +&i2c1 {
>> +	clock-frequency = <400000>;
>> +
>> +	status = "okay";
>> +
>> +	hdmi_bridge: bridge@3d {
>> +		status = "disabled";
>> +	};
>> +
>> +	lvds_bridge: bridge@2c {
> power-supply, if be optional, need to modify dt-binding file.
> otherwise we will meet binding check error.

Okay, got it. I will make necessary changes in the next 
patchset so that the binding check passes.
> 
>> +		compatible = "ti,sn65dsi84";
>> +		reg = <0x2c>;
>> +		enable-gpios = <&tlmm 42 GPIO_ACTIVE_HIGH>;
>> +		ti,dsi-lanes = <4>;
>> +		ti,lvds-format = "jeida-24";
>> +		ti,lvds-bpp = <24>;
> 
> ti,dsi-lanes, ti,lvds-format, ti,lvds-bpp, do we need these property?
> I don't see these property be used from  kernel 6.6 to kernel 6.19. 
>

Will drop these ti,dsi-lanes, ti,lvds-format, and ti,lvds-bpp 
properties in the next patchset.
 
> 
> Thanks,
> Yuanjie
> 
> 
>> +		ports {
>> +			#address-cells = <1>;
>> +			#size-cells = <0>;
>> +
>> +			port@0 {
>> +				reg = <0>;
>> +
>> +				sn65dsi84_in: endpoint {
>> +					data-lanes = <1 2 3 4>;
>> +					remote-endpoint = <&mdss_dsi0_out>;
>> +				};
>> +			};
>> +
>> +			port@2 {
>> +				reg = <2>;
>> +
>> +				sn65dsi84_out_a: endpoint {
>> +					data-lanes = <1 2 3 4>;
>> +					remote-endpoint = <&lvds_panel_out_a>;
>> +				};
>> +			};
>> +
>> +			port@3 {
>> +				reg = <3>;
>> +
>> +				sn65dsi84_out_b: endpoint {
>> +					data-lanes = <1 2 3 4>;
>> +					remote-endpoint = <&lvds_panel_out_b>;
>> +				};
>> +			};
>> +		};
>> +	};
>> +};
>> +
>> +&mdss_dsi0 {
>> +	vdda-supply = <&vreg_l11a>;
>> +
>> +	status = "okay";
>> +};
>> +
>> +&mdss_dsi0_out {
>> +	remote-endpoint = <&sn65dsi84_in>;
>> +	data-lanes = <0 1 2 3>;
>> +};
>> +
>> +&tlmm {
>> +	lcd_bklt_en: lcd-bklt-en-state {
>> +		pins = "gpio115";
>> +		function = "gpio";
>> +		bias-disable;
>> +	};
>> +
>> +	lcd_bklt_pwm: lcd-bklt-pwm-state {
>> +		pins = "gpio59";
>> +		function = "gpio";
>> +		bias-disable;
>> +	};
>> +};
>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>> new file mode 100644
>> index 000000000000..95ed335bcb08
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>> @@ -0,0 +1,616 @@
>> +// SPDX-License-Identifier: BSD-3-Clause
>> +/*
>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>> + */
>> +/dts-v1/;
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
>> +#include "talos.dtsi"
>> +#include "pm8150.dtsi"
>> +/ {
>> +	aliases {
>> +		mmc0 = &sdhc_1;
>> +		serial0 = &uart0;
>> +		serial1 = &uart7;
>> +	};
>> +
>> +	chosen {
>> +		stdout-path = "serial0:115200n8";
>> +	};
>> +
>> +	clocks {
>> +		can_osc: can-oscillator {
>> +			compatible = "fixed-clock";
>> +			clock-frequency = <20000000>;
>> +			#clock-cells = <0>;
>> +		};
>> +
>> +		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>;
>> +		};
>> +	};
>> +
>> +	regulator-usb2-vbus {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "USB2_VBUS";
>> +		gpio = <&pm8150_gpios 10 GPIO_ACTIVE_HIGH>;
>> +		pinctrl-0 = <&usb2_en>;
>> +		pinctrl-names = "default";
>> +		enable-active-high;
>> +		regulator-always-on;
>> +	};
>> +
>> +	vreg_conn_1p8: regulator-conn-1p8 {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vreg_conn_1p8";
>> +		startup-delay-us = <4000>;
>> +		enable-active-high;
>> +		gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
>> +	};
>> +
>> +	vreg_conn_pa: regulator-conn-pa {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vreg_conn_pa";
>> +		startup-delay-us = <4000>;
>> +		enable-active-high;
>> +		gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
>> +	};
>> +
>> +	vreg_v3p3_can: regulator-v3p3-can {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vreg-v3p3-can";
>> +		regulator-min-microvolt = <3300000>;
>> +		regulator-max-microvolt = <3300000>;
>> +		regulator-boot-on;
>> +		regulator-always-on;
>> +	};
>> +
>> +	vreg_v5p0_can: regulator-v5p0-can {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vreg-v5p0-can";
>> +		regulator-min-microvolt = <5000000>;
>> +		regulator-max-microvolt = <5000000>;
>> +		regulator-boot-on;
>> +		regulator-always-on;
>> +	};
>> +
>> +	wcn6855-pmu {
>> +		compatible = "qcom,wcn6855-pmu";
>> +
>> +		pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
>> +		pinctrl-names = "default";
>> +
>> +		bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
>> +		wlan-enable-gpios = <&tlmm 84 GPIO_ACTIVE_HIGH>;
>> +
>> +		vddio-supply = <&vreg_conn_pa>;
>> +		vddaon-supply = <&vreg_s5a>;
>> +		vddpmu-supply = <&vreg_conn_1p8>;
>> +		vddpmumx-supply = <&vreg_conn_1p8>;
>> +		vddpmucx-supply = <&vreg_conn_pa>;
>> +		vddrfa0p95-supply = <&vreg_s5a>;
>> +		vddrfa1p3-supply = <&vreg_s6a>;
>> +		vddrfa1p9-supply = <&vreg_l15a>;
>> +		vddpcie1p3-supply = <&vreg_s6a>;
>> +		vddpcie1p9-supply = <&vreg_l15a>;
>> +
>> +		regulators {
>> +			vreg_pmu_rfa_cmn: ldo0 {
>> +				regulator-name = "vreg_pmu_rfa_cmn";
>> +			};
>> +
>> +			vreg_pmu_aon_0p59: ldo1 {
>> +				regulator-name = "vreg_pmu_aon_0p59";
>> +			};
>> +
>> +			vreg_pmu_wlcx_0p8: ldo2 {
>> +				regulator-name = "vreg_pmu_wlcx_0p8";
>> +			};
>> +
>> +			vreg_pmu_wlmx_0p85: ldo3 {
>> +				regulator-name = "vreg_pmu_wlmx_0p85";
>> +			};
>> +
>> +			vreg_pmu_btcmx_0p85: ldo4 {
>> +				regulator-name = "vreg_pmu_btcmx_0p85";
>> +			};
>> +
>> +			vreg_pmu_rfa_0p8: ldo5 {
>> +				regulator-name = "vreg_pmu_rfa_0p8";
>> +			};
>> +
>> +			vreg_pmu_rfa_1p2: ldo6 {
>> +				regulator-name = "vreg_pmu_rfa_1p2";
>> +			};
>> +
>> +			vreg_pmu_rfa_1p7: ldo7 {
>> +				regulator-name = "vreg_pmu_rfa_1p7";
>> +			};
>> +
>> +			vreg_pmu_pcie_0p9: ldo8 {
>> +				regulator-name = "vreg_pmu_pcie_0p9";
>> +			};
>> +
>> +			vreg_pmu_pcie_1p8: ldo9 {
>> +				regulator-name = "vreg_pmu_pcie_1p8";
>> +			};
>> +		};
>> +	};
>> +
>> +	wifi_1p8v: regulator-wifi-1p8v {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "wifi_1p8v";
>> +		regulator-min-microvolt = <1800000>;
>> +		regulator-max-microvolt = <1800000>;
>> +		gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
>> +		enable-active-high;
>> +		pinctrl-0 = <&wifi_reg_en_pins_state>;
>> +		pinctrl-names = "default";
>> +		regulator-boot-on;
>> +		regulator-always-on;
>> +	};
>> +
>> +	wifi_3p85v: regulator-wifi-3p85v {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "wifi_3p85v";
>> +		regulator-min-microvolt = <3850000>;
>> +		regulator-max-microvolt = <3850000>;
>> +		gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
>> +		enable-active-high;
>> +		pinctrl-0 = <&wifi_reg_en_pins_state>;
>> +		pinctrl-names = "default";
>> +		regulator-boot-on;
>> +		regulator-always-on;
>> +	};
>> +};
>> +
>> +&apps_rsc {
>> +	regulators-0 {
>> +		compatible = "qcom,pm8150-rpmh-regulators";
>> +		qcom,pmic-id = "a";
>> +
>> +		vreg_s3a: smps3 {
>> +			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>;
>> +			regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
>> +			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>;
>> +		};
>> +	};
>> +};
>> +
>> +&gpi_dma0 {
>> +	status = "okay";
>> +};
>> +
>> +&gpi_dma1 {
>> +	status = "okay";
>> +};
>> +
>> +&i2c5 {
>> +	clock-frequency = <400000>;
>> +	status = "okay";
>> +
>> +	eeprom@57 {
>> +		compatible = "atmel,24c02";
>> +		reg = <0x57>;
>> +		pagesize = <16>;
>> +	};
>> +
>> +	eeprom@5f {
>> +		compatible = "atmel,24mac602";
>> +		reg = <0x5f>;
>> +		pagesize = <16>;
>> +	};
>> +};
>> +
>> +&mdss {
>> +	status = "okay";
>> +};
>> +
>> +&mdss_dp0 {
>> +	status = "okay";
>> +};
>> +
>> +&mdss_dp0_out {
>> +	link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000>;
>> +	remote-endpoint = <&dp0_connector_in>;
>> +};
>> +
>> +&mdss_dsi0 {
>> +	vdda-supply = <&vreg_l11a>;
>> +	status = "okay";
>> +};
>> +
>> +&mdss_dsi0_phy {
>> +	vcca-supply = <&vreg_l5a>;
>> +	status = "okay";
>> +};
>> +
>> +&pcie {
>> +	perst-gpios = <&tlmm 89 GPIO_ACTIVE_LOW>;
>> +	wake-gpios = <&tlmm 100 GPIO_ACTIVE_HIGH>;
>> +
>> +	pinctrl-0 = <&pcie_default_state>;
>> +	pinctrl-names = "default";
>> +
>> +	status = "okay";
>> +};
>> +
>> +&pcie_phy {
>> +	vdda-phy-supply = <&vreg_l5a>;
>> +	vdda-pll-supply = <&vreg_l12a>;
>> +
>> +	status = "okay";
>> +};
>> +
>> +&pcie_port0 {
>> +	wifi@0 {
>> +		compatible = "pci17cb,1103";
>> +		reg = <0x10000 0x0 0x0 0x0 0x0>;
>> +
>> +		qcom,calibration-variant = "QC_QCS615_Ride";
>> +
>> +		vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
>> +		vddaon-supply = <&vreg_pmu_aon_0p59>;
>> +		vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
>> +		vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
>> +		vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
>> +		vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
>> +		vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
>> +		vddpcie0p9-supply = <&vreg_pmu_pcie_0p9>;
>> +		vddpcie1p8-supply = <&vreg_pmu_pcie_1p8>;
>> +	};
>> +};
>> +
>> +&pm8150_gpios {
>> +	usb2_en: usb2-en-state {
>> +		pins = "gpio10";
>> +		function = "normal";
>> +		output-enable;
>> +		power-source = <0>;
>> +	};
>> +};
>> +
>> +&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";
>> +};
>> +
>> +&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";
>> +};
>> +
>> +&spi6 {
>> +	status = "okay";
>> +
>> +	can@0 {
>> +		compatible = "microchip,mcp2515";
>> +		reg = <0>;
>> +		clocks = <&can_osc>;
>> +		interrupts-extended = <&tlmm 87 IRQ_TYPE_LEVEL_LOW>;
>> +		spi-max-frequency = <10000000>;
>> +		vdd-supply = <&vreg_v3p3_can>;
>> +		xceiver-supply = <&vreg_v5p0_can>;
>> +	};
>> +};
>> +
>> +&tlmm {
>> +	bt_en_state: bt-en-state {
>> +		pins = "gpio85";
>> +		function = "gpio";
>> +		bias-pull-down;
>> +	};
>> +
>> +	pcie_default_state: pcie-default-state {
>> +		clkreq-pins {
>> +			pins = "gpio90";
>> +			function = "pcie_clk_req";
>> +			drive-strength = <2>;
>> +			bias-pull-up;
>> +		};
>> +
>> +		perst-pins {
>> +			pins = "gpio89";
>> +			function = "gpio";
>> +			drive-strength = <2>;
>> +			bias-disable;
>> +		};
>> +
>> +		wake-pins {
>> +			pins = "gpio100";
>> +			function = "gpio";
>> +			drive-strength = <2>;
>> +			bias-pull-up;
>> +		};
>> +	};
>> +
>> +	wifi_reg_en_pins_state: wifi-reg-en-pins-state {
>> +		pins = "gpio91";
>> +		function = "gpio";
>> +		drive-strength = <8>;
>> +		output-high;
>> +		bias-pull-up;
>> +	};
>> +
>> +	wlan_en_state: wlan-en-state {
>> +		pins = "gpio84";
>> +		function = "gpio";
>> +		drive-strength = <16>;
>> +		bias-pull-down;
>> +		output-low;
>> +	};
>> +};
>> +
>> +&uart0 {
>> +	status = "okay";
>> +};
>> +
>> +&uart7 {
>> +	status = "okay";
>> +
>> +	bluetooth {
>> +		compatible = "qcom,wcn6855-bt";
>> +		firmware-name = "QCA6698/hpnv21", "QCA6698/hpbtfw21.tlv";
>> +
>> +		vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
>> +		vddaon-supply = <&vreg_pmu_aon_0p59>;
>> +		vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
>> +		vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
>> +		vddbtcmx-supply = <&vreg_pmu_btcmx_0p85>;
>> +		vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
>> +		vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
>> +		vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
>> +	};
>> +};
>> +
>> +&usb_1 {
>> +	status = "okay";
>> +};
>> +
>> +&usb_1_dwc3 {
>> +	dr_mode = "host";
>> +};
>> +
>> +&usb_hsphy_1 {
>> +	vdd-supply = <&vreg_l5a>;
>> +	vdda-pll-supply = <&vreg_l12a>;
>> +	vdda-phy-dpdm-supply = <&vreg_l13a>;
>> +
>> +	status = "okay";
>> +};
>> +
>> +&usb_2 {
>> +	status = "okay";
>> +};
>> +
>> +&usb_2_dwc3 {
>> +	dr_mode = "host";
>> +};
>> +
>> +&usb_hsphy_2 {
>> +	vdd-supply = <&vreg_l5a>;
>> +	vdda-pll-supply = <&vreg_l12a>;
>> +	vdda-phy-dpdm-supply = <&vreg_l13a>;
>> +
>> +	status = "okay";
>> +};
>> +
>> +&usb_qmpphy {
>> +	vdda-phy-supply = <&vreg_l5a>;
>> +	vdda-pll-supply = <&vreg_l12a>;
>> +
>> +	status = "okay";
>> +};
>> +
>> +&usb_qmpphy_2 {
>> +	vdda-phy-supply = <&vreg_l11a>;
>> +	vdda-pll-supply = <&vreg_l5a>;
>> +
>> +	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";
>> +};
>> +
>> +&venus {
>> +	status = "okay";
>> +};
>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk.dts b/arch/arm64/boot/dts/qcom/talos-evk.dts
>> new file mode 100644
>> index 000000000000..af100e22beee
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/talos-evk.dts
>> @@ -0,0 +1,139 @@
>> +// SPDX-License-Identifier: BSD-3-Clause
>> +/*
>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>> + */
>> +/dts-v1/;
>> +
>> +#include "talos-evk-som.dtsi"
>> +
>> +/ {
>> +	model = "Qualcomm QCS615 IQ 615 EVK";
>> +	compatible = "qcom,talos-evk", "qcom,qcs615", "qcom,sm6150";
>> +	chassis-type = "embedded";
>> +
>> +	aliases {
>> +		mmc1 = &sdhc_2;
>> +	};
>> +
>> +	dp0-connector {
>> +		compatible = "dp-connector";
>> +		label = "DP0";
>> +		type = "full-size";
>> +
>> +		hpd-gpios = <&tlmm 104 GPIO_ACTIVE_HIGH>;
>> +
>> +		port {
>> +			dp0_connector_in: endpoint {
>> +				remote-endpoint = <&mdss_dp0_out>;
>> +			};
>> +		};
>> +	};
>> +
>> +	hdmi_connector: hdmi-out {
>> +		compatible = "hdmi-connector";
>> +		type = "d";
>> +
>> +		port {
>> +			hdmi_con_out: endpoint {
>> +			remote-endpoint = <&adv7535_out>;
>> +			};
>> +		};
>> +	};
>> +
>> +	vreg_v1p8_out: regulator-v1p8-out {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vreg-v1p8-out";
>> +		regulator-min-microvolt = <1800000>;
>> +		regulator-max-microvolt = <1800000>;
>> +		vin-supply = <&vreg_v5p0_out>;
>> +		regulator-boot-on;
>> +		regulator-always-on;
>> +	};
>> +
>> +	vreg_v3p3_out: regulator-v3p3-out {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vreg-v3p3-out";
>> +		regulator-min-microvolt = <3300000>;
>> +		regulator-max-microvolt = <3300000>;
>> +		vin-supply = <&vreg_v5p0_out>;
>> +		regulator-boot-on;
>> +		regulator-always-on;
>> +	};
>> +
>> +	vreg_v5p0_out: regulator-v5p0-out {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vreg-v5p0-out";
>> +		regulator-min-microvolt = <5000000>;
>> +		regulator-max-microvolt = <5000000>;
>> +		regulator-boot-on;
>> +		regulator-always-on;
>> +		/* Powered by system 20V rail (USBC_VBUS_IN) */
>> +	};
>> +};
>> +
>> +&i2c1 {
>> +	clock-frequency = <400000>;
>> +	status = "okay";
>> +
>> +	hdmi_bridge: bridge@3d {
>> +		compatible = "adi,adv7535";
>> +		reg = <0x3d>;
>> +		avdd-supply = <&vreg_v1p8_out>;
>> +		dvdd-supply = <&vreg_v1p8_out>;
>> +		pvdd-supply = <&vreg_v1p8_out>;
>> +		a2vdd-supply = <&vreg_v1p8_out>;
>> +		v3p3-supply = <&vreg_v3p3_out>;
>> +		interrupts-extended = <&tlmm 26 IRQ_TYPE_LEVEL_LOW>;
>> +		adi,dsi-lanes = <4>;
>> +
>> +		ports {
>> +			#address-cells = <1>;
>> +			#size-cells = <0>;
>> +
>> +			port@0 {
>> +				reg = <0>;
>> +
>> +				adv7535_in: endpoint {
>> +					remote-endpoint = <&mdss_dsi0_out>;
>> +				};
>> +			};
>> +
>> +			port@1 {
>> +				reg = <1>;
>> +
>> +				adv7535_out: endpoint {
>> +					remote-endpoint = <&hdmi_con_out>;
>> +				};
>> +			};
>> +		};
>> +	};
>> +};
>> +
>> +&mdss_dsi0_out {
>> +	remote-endpoint = <&adv7535_in>;
>> +	data-lanes = <0 1 2 3>;
>> +};
>> +
>> +&pon_pwrkey {
>> +	status = "okay";
>> +};
>> +
>> +&pon_resin {
>> +	linux,code = <KEY_VOLUMEDOWN>;
>> +
>> +	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";
>> +};
>> -- 
>> 2.34.1

Re: [PATCH v11 4/4] arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

[Qian Zhang]

On 1/9/2026 5:52 PM, Sudarshan Shetty wrote:
> Add the device tree for the QCS615-based Talos EVK platform. The
> platform is composed of a System-on-Module following the SMARC
> standard, and a Carrier Board.
> 
> The Carrier Board supports several display configurations, HDMI and
> LVDS. Both configurations use the same base hardware, with the display
> selection controlled by a DIP switch.
> 
> Use a DTBO file, talos-evk-lvds-auo,g133han01.dtso, which defines an
> overlay that disables HDMI and adds LVDS. The DTs file talos-evk
> can describe the HDMI display configurations.
> 
> The initial device tree includes support for:
> - CPU and memory
> - UART
> - GPIOs
> - Regulators
> - PMIC
> - Early console
> - AT24MAC602 EEPROM
> - MCP2515 SPI to CAN
> - ADV7535 DSI-to-HDMI bridge
> - DisplayPort interface
> - SN65DSI84ZXHR DSI-to-LVDS bridge
> - Wi-Fi/BT
> 
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
> ---
>   arch/arm64/boot/dts/qcom/Makefile             |   4 +
>   .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
>   arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
>   arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
>   4 files changed, 885 insertions(+)
>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts
> 
> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> index 76cf0115a00a..289b651ef0c5 100644
> --- a/arch/arm64/boot/dts/qcom/Makefile
> +++ b/arch/arm64/boot/dts/qcom/Makefile
> @@ -324,6 +324,10 @@ dtb-$(CONFIG_ARCH_QCOM)	+= sm8650-mtp.dtb
>   dtb-$(CONFIG_ARCH_QCOM)	+= sm8650-qrd.dtb
>   dtb-$(CONFIG_ARCH_QCOM)	+= sm8750-mtp.dtb
>   dtb-$(CONFIG_ARCH_QCOM)	+= sm8750-qrd.dtb
> +dtb-$(CONFIG_ARCH_QCOM)	+= talos-evk.dtb
> +talos-evk-lvds-auo,g133han01-dtbs	:= \
> +	talos-evk.dtb talos-evk-lvds-auo,g133han01.dtbo
> +dtb-$(CONFIG_ARCH_QCOM)	+= talos-evk-lvds-auo,g133han01.dtb
>   x1e001de-devkit-el2-dtbs	:= x1e001de-devkit.dtb x1-el2.dtbo
>   dtb-$(CONFIG_ARCH_QCOM)	+= x1e001de-devkit.dtb x1e001de-devkit-el2.dtb
>   x1e78100-lenovo-thinkpad-t14s-el2-dtbs	:= x1e78100-lenovo-thinkpad-t14s.dtb x1-el2.dtbo
> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> new file mode 100644
> index 000000000000..ad058cf4cd93
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> @@ -0,0 +1,126 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> + */
> +/dts-v1/;
> +/plugin/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +
> +&{/} {
> +	backlight: backlight {
> +		compatible = "gpio-backlight";
> +		gpios = <&tlmm 59 GPIO_ACTIVE_HIGH>,
> +			<&tlmm 115 GPIO_ACTIVE_HIGH>;
> +		default-on;
> +	};
> +
> +	panel-lvds {
> +		compatible = "auo,g133han01";
> +
> +		ports {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +
> +			/* LVDS A (Odd pixels) */
> +			port@0 {
> +				reg = <0>;
> +				dual-lvds-odd-pixels;
> +
> +				lvds_panel_out_a: endpoint {
> +					remote-endpoint = <&sn65dsi84_out_a>;
> +				};
> +			};
> +
> +			/* LVDS B (Even pixels) */
> +			port@1 {
> +				reg = <1>;
> +				dual-lvds-even-pixels;
> +
> +				lvds_panel_out_b: endpoint {
> +					remote-endpoint = <&sn65dsi84_out_b>;
> +				};
> +			};
> +		};
> +	};
> +};
> +
> +&hdmi_connector {
> +	status = "disabled";
> +};
> +
> +&i2c1 {
> +	clock-frequency = <400000>;
> +
> +	status = "okay";
> +
> +	hdmi_bridge: bridge@3d {
> +		status = "disabled";
> +	};
> +
> +	lvds_bridge: bridge@2c {
> +		compatible = "ti,sn65dsi84";
> +		reg = <0x2c>;
> +		enable-gpios = <&tlmm 42 GPIO_ACTIVE_HIGH>;
> +		ti,dsi-lanes = <4>;
> +		ti,lvds-format = "jeida-24";
> +		ti,lvds-bpp = <24>;
> +
> +		ports {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +
> +			port@0 {
> +				reg = <0>;
> +
> +				sn65dsi84_in: endpoint {
> +					data-lanes = <1 2 3 4>;
> +					remote-endpoint = <&mdss_dsi0_out>;
> +				};
> +			};
> +
> +			port@2 {
> +				reg = <2>;
> +
> +				sn65dsi84_out_a: endpoint {
> +					data-lanes = <1 2 3 4>;
> +					remote-endpoint = <&lvds_panel_out_a>;
> +				};
> +			};
> +
> +			port@3 {
> +				reg = <3>;
> +
> +				sn65dsi84_out_b: endpoint {
> +					data-lanes = <1 2 3 4>;
> +					remote-endpoint = <&lvds_panel_out_b>;
> +				};
> +			};
> +		};
> +	};
> +};
> +
> +&mdss_dsi0 {
> +	vdda-supply = <&vreg_l11a>;
> +
> +	status = "okay";
> +};
> +
> +&mdss_dsi0_out {
> +	remote-endpoint = <&sn65dsi84_in>;
> +	data-lanes = <0 1 2 3>;
> +};
> +
> +&tlmm {
> +	lcd_bklt_en: lcd-bklt-en-state {
> +		pins = "gpio115";
> +		function = "gpio";
> +		bias-disable;
> +	};
> +
> +	lcd_bklt_pwm: lcd-bklt-pwm-state {
> +		pins = "gpio59";
> +		function = "gpio";
> +		bias-disable;
> +	};
> +};
> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> new file mode 100644
> index 000000000000..95ed335bcb08
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> @@ -0,0 +1,616 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> + */
> +/dts-v1/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> +#include "talos.dtsi"
> +#include "pm8150.dtsi"
> +/ {
> +	aliases {
> +		mmc0 = &sdhc_1;
> +		serial0 = &uart0;
> +		serial1 = &uart7;
> +	};
> +
> +	chosen {
> +		stdout-path = "serial0:115200n8";
> +	};
> +
> +	clocks {
> +		can_osc: can-oscillator {
> +			compatible = "fixed-clock";
> +			clock-frequency = <20000000>;
> +			#clock-cells = <0>;
> +		};
> +
> +		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>;
> +		};
> +	};
> +
> +	regulator-usb2-vbus {
> +		compatible = "regulator-fixed";
> +		regulator-name = "USB2_VBUS";
> +		gpio = <&pm8150_gpios 10 GPIO_ACTIVE_HIGH>;
> +		pinctrl-0 = <&usb2_en>;
> +		pinctrl-names = "default";
> +		enable-active-high;
> +		regulator-always-on;
> +	};
> +
> +	vreg_conn_1p8: regulator-conn-1p8 {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg_conn_1p8";
> +		startup-delay-us = <4000>;
> +		enable-active-high;
> +		gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
> +	};
> +
> +	vreg_conn_pa: regulator-conn-pa {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg_conn_pa";
> +		startup-delay-us = <4000>;
> +		enable-active-high;
> +		gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
> +	};
> +
> +	vreg_v3p3_can: regulator-v3p3-can {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg-v3p3-can";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +
> +	vreg_v5p0_can: regulator-v5p0-can {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg-v5p0-can";
> +		regulator-min-microvolt = <5000000>;
> +		regulator-max-microvolt = <5000000>;
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +
> +	wcn6855-pmu {
> +		compatible = "qcom,wcn6855-pmu";
> +
> +		pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
> +		pinctrl-names = "default";
> +
> +		bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
> +		wlan-enable-gpios = <&tlmm 84 GPIO_ACTIVE_HIGH>;
> +
> +		vddio-supply = <&vreg_conn_pa>;
> +		vddaon-supply = <&vreg_s5a>;
> +		vddpmu-supply = <&vreg_conn_1p8>;
> +		vddpmumx-supply = <&vreg_conn_1p8>;
> +		vddpmucx-supply = <&vreg_conn_pa>;
> +		vddrfa0p95-supply = <&vreg_s5a>;
> +		vddrfa1p3-supply = <&vreg_s6a>;
> +		vddrfa1p9-supply = <&vreg_l15a>;
> +		vddpcie1p3-supply = <&vreg_s6a>;
> +		vddpcie1p9-supply = <&vreg_l15a>;
> +
> +		regulators {
> +			vreg_pmu_rfa_cmn: ldo0 {
> +				regulator-name = "vreg_pmu_rfa_cmn";
> +			};
> +
> +			vreg_pmu_aon_0p59: ldo1 {
> +				regulator-name = "vreg_pmu_aon_0p59";
> +			};
> +
> +			vreg_pmu_wlcx_0p8: ldo2 {
> +				regulator-name = "vreg_pmu_wlcx_0p8";
> +			};
> +
> +			vreg_pmu_wlmx_0p85: ldo3 {
> +				regulator-name = "vreg_pmu_wlmx_0p85";
> +			};
> +
> +			vreg_pmu_btcmx_0p85: ldo4 {
> +				regulator-name = "vreg_pmu_btcmx_0p85";
> +			};
> +
> +			vreg_pmu_rfa_0p8: ldo5 {
> +				regulator-name = "vreg_pmu_rfa_0p8";
> +			};
> +
> +			vreg_pmu_rfa_1p2: ldo6 {
> +				regulator-name = "vreg_pmu_rfa_1p2";
> +			};
> +
> +			vreg_pmu_rfa_1p7: ldo7 {
> +				regulator-name = "vreg_pmu_rfa_1p7";
> +			};
> +
> +			vreg_pmu_pcie_0p9: ldo8 {
> +				regulator-name = "vreg_pmu_pcie_0p9";
> +			};
> +
> +			vreg_pmu_pcie_1p8: ldo9 {
> +				regulator-name = "vreg_pmu_pcie_1p8";
> +			};
> +		};
> +	};
> +
> +	wifi_1p8v: regulator-wifi-1p8v {
> +		compatible = "regulator-fixed";
> +		regulator-name = "wifi_1p8v";
> +		regulator-min-microvolt = <1800000>;
> +		regulator-max-microvolt = <1800000>;
> +		gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
Please check this pin number
> +		enable-active-high;
> +		pinctrl-0 = <&wifi_reg_en_pins_state>;
> +		pinctrl-names = "default";
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +
> +	wifi_3p85v: regulator-wifi-3p85v {
> +		compatible = "regulator-fixed";
> +		regulator-name = "wifi_3p85v";
> +		regulator-min-microvolt = <3850000>;
> +		regulator-max-microvolt = <3850000>;
> +		gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
Please check this pin number
> +		enable-active-high;
> +		pinctrl-0 = <&wifi_reg_en_pins_state>;
> +		pinctrl-names = "default";
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +};

Are these two node necessary?

> +
> +&apps_rsc {
> +	regulators-0 {
> +		compatible = "qcom,pm8150-rpmh-regulators";
> +		qcom,pmic-id = "a";
> +
> +		vreg_s3a: smps3 {
> +			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>;
> +			regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
> +			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>;
> +		};
> +	};
> +};
> +
> +&gpi_dma0 {
> +	status = "okay";
> +};
> +
> +&gpi_dma1 {
> +	status = "okay";
> +};
> +
> +&i2c5 {
> +	clock-frequency = <400000>;
> +	status = "okay";
> +
> +	eeprom@57 {
> +		compatible = "atmel,24c02";
> +		reg = <0x57>;
> +		pagesize = <16>;
> +	};
> +
> +	eeprom@5f {
> +		compatible = "atmel,24mac602";
> +		reg = <0x5f>;
> +		pagesize = <16>;
> +	};
> +};
> +
> +&mdss {
> +	status = "okay";
> +};
> +
> +&mdss_dp0 {
> +	status = "okay";
> +};
> +
> +&mdss_dp0_out {
> +	link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000>;
> +	remote-endpoint = <&dp0_connector_in>;
> +};
> +
> +&mdss_dsi0 {
> +	vdda-supply = <&vreg_l11a>;
> +	status = "okay";
> +};
> +
> +&mdss_dsi0_phy {
> +	vcca-supply = <&vreg_l5a>;
> +	status = "okay";
> +};
> +
> +&pcie {
> +	perst-gpios = <&tlmm 89 GPIO_ACTIVE_LOW>;
> +	wake-gpios = <&tlmm 100 GPIO_ACTIVE_HIGH>;
> +
> +	pinctrl-0 = <&pcie_default_state>;
> +	pinctrl-names = "default";
> +
> +	status = "okay";
> +};
> +
> +&pcie_phy {
> +	vdda-phy-supply = <&vreg_l5a>;
> +	vdda-pll-supply = <&vreg_l12a>;
> +
> +	status = "okay";
> +};
> +
> +&pcie_port0 {
> +	wifi@0 {
> +		compatible = "pci17cb,1103";

This style of writing is not very readable.

> +		reg = <0x10000 0x0 0x0 0x0 0x0>;
> +
> +		qcom,calibration-variant = "QC_QCS615_Ride";
> +
> +		vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
> +		vddaon-supply = <&vreg_pmu_aon_0p59>;
> +		vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
> +		vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
> +		vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
> +		vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
> +		vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
> +		vddpcie0p9-supply = <&vreg_pmu_pcie_0p9>;
> +		vddpcie1p8-supply = <&vreg_pmu_pcie_1p8>;
> +	};
> +};
> +
> +&pm8150_gpios {
> +	usb2_en: usb2-en-state {
> +		pins = "gpio10";
> +		function = "normal";
> +		output-enable;
> +		power-source = <0>;
> +	};
> +};
> +
> +&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";
> +};
> +
> +&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";
> +};
> +
> +&spi6 {
> +	status = "okay";
> +
> +	can@0 {
> +		compatible = "microchip,mcp2515";
> +		reg = <0>;
> +		clocks = <&can_osc>;
> +		interrupts-extended = <&tlmm 87 IRQ_TYPE_LEVEL_LOW>;
> +		spi-max-frequency = <10000000>;
> +		vdd-supply = <&vreg_v3p3_can>;
> +		xceiver-supply = <&vreg_v5p0_can>;
> +	};
> +};
> +
> +&tlmm {
> +	bt_en_state: bt-en-state {
> +		pins = "gpio85";
> +		function = "gpio";
> +		bias-pull-down;
> +	};
> +
> +	pcie_default_state: pcie-default-state {
> +		clkreq-pins {
> +			pins = "gpio90";
> +			function = "pcie_clk_req";
> +			drive-strength = <2>;
> +			bias-pull-up;
> +		};
> +
> +		perst-pins {
> +			pins = "gpio89";
> +			function = "gpio";
> +			drive-strength = <2>;
> +			bias-disable;
> +		};
> +
> +		wake-pins {
> +			pins = "gpio100";
> +			function = "gpio";
> +			drive-strength = <2>;
> +			bias-pull-up;
> +		};
> +	};
> +
> +	wifi_reg_en_pins_state: wifi-reg-en-pins-state {
> +		pins = "gpio91";
> +		function = "gpio";
> +		drive-strength = <8>;
> +		output-high;
> +		bias-pull-up;
> +	};
> +
> +	wlan_en_state: wlan-en-state {
> +		pins = "gpio84";
> +		function = "gpio";
> +		drive-strength = <16>;
> +		bias-pull-down;
> +		output-low;
> +	};
> +};
> +
> +&uart0 {
> +	status = "okay";
> +};
> +
> +&uart7 {
> +	status = "okay";
> +
> +	bluetooth {
> +		compatible = "qcom,wcn6855-bt";
> +		firmware-name = "QCA6698/hpnv21", "QCA6698/hpbtfw21.tlv";
> +
> +		vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
> +		vddaon-supply = <&vreg_pmu_aon_0p59>;
> +		vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
> +		vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
> +		vddbtcmx-supply = <&vreg_pmu_btcmx_0p85>;
> +		vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
> +		vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
> +		vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
> +	};
> +};
> +
> +&usb_1 {
> +	status = "okay";
> +};
> +
> +&usb_1_dwc3 {
> +	dr_mode = "host";
> +};
> +
> +&usb_hsphy_1 {
> +	vdd-supply = <&vreg_l5a>;
> +	vdda-pll-supply = <&vreg_l12a>;
> +	vdda-phy-dpdm-supply = <&vreg_l13a>;
> +
> +	status = "okay";
> +};
> +
> +&usb_2 {
> +	status = "okay";
> +};
> +
> +&usb_2_dwc3 {
> +	dr_mode = "host";
> +};
> +
> +&usb_hsphy_2 {
> +	vdd-supply = <&vreg_l5a>;
> +	vdda-pll-supply = <&vreg_l12a>;
> +	vdda-phy-dpdm-supply = <&vreg_l13a>;
> +
> +	status = "okay";
> +};
> +
> +&usb_qmpphy {
> +	vdda-phy-supply = <&vreg_l5a>;
> +	vdda-pll-supply = <&vreg_l12a>;
> +
> +	status = "okay";
> +};
> +
> +&usb_qmpphy_2 {
> +	vdda-phy-supply = <&vreg_l11a>;
> +	vdda-pll-supply = <&vreg_l5a>;
> +
> +	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";
> +};
> +
> +&venus {
> +	status = "okay";
> +};
> diff --git a/arch/arm64/boot/dts/qcom/talos-evk.dts b/arch/arm64/boot/dts/qcom/talos-evk.dts
> new file mode 100644
> index 000000000000..af100e22beee
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/talos-evk.dts
> @@ -0,0 +1,139 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> + */
> +/dts-v1/;
> +
> +#include "talos-evk-som.dtsi"
> +
> +/ {
> +	model = "Qualcomm QCS615 IQ 615 EVK";
> +	compatible = "qcom,talos-evk", "qcom,qcs615", "qcom,sm6150";
> +	chassis-type = "embedded";
> +
> +	aliases {
> +		mmc1 = &sdhc_2;
> +	};
> +
> +	dp0-connector {
> +		compatible = "dp-connector";
> +		label = "DP0";
> +		type = "full-size";
> +
> +		hpd-gpios = <&tlmm 104 GPIO_ACTIVE_HIGH>;
> +
> +		port {
> +			dp0_connector_in: endpoint {
> +				remote-endpoint = <&mdss_dp0_out>;
> +			};
> +		};
> +	};
> +
> +	hdmi_connector: hdmi-out {
> +		compatible = "hdmi-connector";
> +		type = "d";
> +
> +		port {
> +			hdmi_con_out: endpoint {
> +			remote-endpoint = <&adv7535_out>;
> +			};
> +		};
> +	};
> +
> +	vreg_v1p8_out: regulator-v1p8-out {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg-v1p8-out";
> +		regulator-min-microvolt = <1800000>;
> +		regulator-max-microvolt = <1800000>;
> +		vin-supply = <&vreg_v5p0_out>;
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +
> +	vreg_v3p3_out: regulator-v3p3-out {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg-v3p3-out";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		vin-supply = <&vreg_v5p0_out>;
> +		regulator-boot-on;
> +		regulator-always-on;
> +	};
> +
> +	vreg_v5p0_out: regulator-v5p0-out {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vreg-v5p0-out";
> +		regulator-min-microvolt = <5000000>;
> +		regulator-max-microvolt = <5000000>;
> +		regulator-boot-on;
> +		regulator-always-on;
> +		/* Powered by system 20V rail (USBC_VBUS_IN) */
> +	};
> +};
> +
> +&i2c1 {
> +	clock-frequency = <400000>;
> +	status = "okay";
> +
> +	hdmi_bridge: bridge@3d {
> +		compatible = "adi,adv7535";
> +		reg = <0x3d>;
> +		avdd-supply = <&vreg_v1p8_out>;
> +		dvdd-supply = <&vreg_v1p8_out>;
> +		pvdd-supply = <&vreg_v1p8_out>;
> +		a2vdd-supply = <&vreg_v1p8_out>;
> +		v3p3-supply = <&vreg_v3p3_out>;
> +		interrupts-extended = <&tlmm 26 IRQ_TYPE_LEVEL_LOW>;
> +		adi,dsi-lanes = <4>;
> +
> +		ports {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +
> +			port@0 {
> +				reg = <0>;
> +
> +				adv7535_in: endpoint {
> +					remote-endpoint = <&mdss_dsi0_out>;
> +				};
> +			};
> +
> +			port@1 {
> +				reg = <1>;
> +
> +				adv7535_out: endpoint {
> +					remote-endpoint = <&hdmi_con_out>;
> +				};
> +			};
> +		};
> +	};
> +};
> +
> +&mdss_dsi0_out {
> +	remote-endpoint = <&adv7535_in>;
> +	data-lanes = <0 1 2 3>;
> +};
> +
> +&pon_pwrkey {
> +	status = "okay";
> +};
> +
> +&pon_resin {
> +	linux,code = <KEY_VOLUMEDOWN>;
> +
> +	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";
> +};

Re: [PATCH v11 4/4] arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

[tessolveupstream]

On 15-01-2026 18:47, Qian Zhang wrote:
> 
> 
> On 1/9/2026 5:52 PM, Sudarshan Shetty wrote:
>> Add the device tree for the QCS615-based Talos EVK platform. The
>> platform is composed of a System-on-Module following the SMARC
>> standard, and a Carrier Board.
>>
>> The Carrier Board supports several display configurations, HDMI and
>> LVDS. Both configurations use the same base hardware, with the display
>> selection controlled by a DIP switch.
>>
>> Use a DTBO file, talos-evk-lvds-auo,g133han01.dtso, which defines an
>> overlay that disables HDMI and adds LVDS. The DTs file talos-evk
>> can describe the HDMI display configurations.
>>
>> The initial device tree includes support for:
>> - CPU and memory
>> - UART
>> - GPIOs
>> - Regulators
>> - PMIC
>> - Early console
>> - AT24MAC602 EEPROM
>> - MCP2515 SPI to CAN
>> - ADV7535 DSI-to-HDMI bridge
>> - DisplayPort interface
>> - SN65DSI84ZXHR DSI-to-LVDS bridge
>> - Wi-Fi/BT
>>
>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
>> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
>> ---
>>   arch/arm64/boot/dts/qcom/Makefile             |   4 +
>>   .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
>>   arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
>>   arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
>>   4 files changed, 885 insertions(+)
>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts
>>
>> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
>> index 76cf0115a00a..289b651ef0c5 100644
>> --- a/arch/arm64/boot/dts/qcom/Makefile
>> +++ b/arch/arm64/boot/dts/qcom/Makefile
>> @@ -324,6 +324,10 @@ dtb-$(CONFIG_ARCH_QCOM)    += sm8650-mtp.dtb
>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8650-qrd.dtb
>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-mtp.dtb
>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-qrd.dtb
>> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk.dtb
>> +talos-evk-lvds-auo,g133han01-dtbs    := \
>> +    talos-evk.dtb talos-evk-lvds-auo,g133han01.dtbo
>> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk-lvds-auo,g133han01.dtb
>>   x1e001de-devkit-el2-dtbs    := x1e001de-devkit.dtb x1-el2.dtbo
>>   dtb-$(CONFIG_ARCH_QCOM)    += x1e001de-devkit.dtb x1e001de-devkit-el2.dtb
>>   x1e78100-lenovo-thinkpad-t14s-el2-dtbs    := x1e78100-lenovo-thinkpad-t14s.dtb x1-el2.dtbo
>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>> new file mode 100644
>> index 000000000000..ad058cf4cd93
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>> @@ -0,0 +1,126 @@
>> +// SPDX-License-Identifier: BSD-3-Clause
>> +/*
>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>> + */
>> +/dts-v1/;
>> +/plugin/;
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +
>> +&{/} {
>> +    backlight: backlight {
>> +        compatible = "gpio-backlight";
>> +        gpios = <&tlmm 59 GPIO_ACTIVE_HIGH>,
>> +            <&tlmm 115 GPIO_ACTIVE_HIGH>;
>> +        default-on;
>> +    };
>> +
>> +    panel-lvds {
>> +        compatible = "auo,g133han01";
>> +
>> +        ports {
>> +            #address-cells = <1>;
>> +            #size-cells = <0>;
>> +
>> +            /* LVDS A (Odd pixels) */
>> +            port@0 {
>> +                reg = <0>;
>> +                dual-lvds-odd-pixels;
>> +
>> +                lvds_panel_out_a: endpoint {
>> +                    remote-endpoint = <&sn65dsi84_out_a>;
>> +                };
>> +            };
>> +
>> +            /* LVDS B (Even pixels) */
>> +            port@1 {
>> +                reg = <1>;
>> +                dual-lvds-even-pixels;
>> +
>> +                lvds_panel_out_b: endpoint {
>> +                    remote-endpoint = <&sn65dsi84_out_b>;
>> +                };
>> +            };
>> +        };
>> +    };
>> +};
>> +
>> +&hdmi_connector {
>> +    status = "disabled";
>> +};
>> +
>> +&i2c1 {
>> +    clock-frequency = <400000>;
>> +
>> +    status = "okay";
>> +
>> +    hdmi_bridge: bridge@3d {
>> +        status = "disabled";
>> +    };
>> +
>> +    lvds_bridge: bridge@2c {
>> +        compatible = "ti,sn65dsi84";
>> +        reg = <0x2c>;
>> +        enable-gpios = <&tlmm 42 GPIO_ACTIVE_HIGH>;
>> +        ti,dsi-lanes = <4>;
>> +        ti,lvds-format = "jeida-24";
>> +        ti,lvds-bpp = <24>;
>> +
>> +        ports {
>> +            #address-cells = <1>;
>> +            #size-cells = <0>;
>> +
>> +            port@0 {
>> +                reg = <0>;
>> +
>> +                sn65dsi84_in: endpoint {
>> +                    data-lanes = <1 2 3 4>;
>> +                    remote-endpoint = <&mdss_dsi0_out>;
>> +                };
>> +            };
>> +
>> +            port@2 {
>> +                reg = <2>;
>> +
>> +                sn65dsi84_out_a: endpoint {
>> +                    data-lanes = <1 2 3 4>;
>> +                    remote-endpoint = <&lvds_panel_out_a>;
>> +                };
>> +            };
>> +
>> +            port@3 {
>> +                reg = <3>;
>> +
>> +                sn65dsi84_out_b: endpoint {
>> +                    data-lanes = <1 2 3 4>;
>> +                    remote-endpoint = <&lvds_panel_out_b>;
>> +                };
>> +            };
>> +        };
>> +    };
>> +};
>> +
>> +&mdss_dsi0 {
>> +    vdda-supply = <&vreg_l11a>;
>> +
>> +    status = "okay";
>> +};
>> +
>> +&mdss_dsi0_out {
>> +    remote-endpoint = <&sn65dsi84_in>;
>> +    data-lanes = <0 1 2 3>;
>> +};
>> +
>> +&tlmm {
>> +    lcd_bklt_en: lcd-bklt-en-state {
>> +        pins = "gpio115";
>> +        function = "gpio";
>> +        bias-disable;
>> +    };
>> +
>> +    lcd_bklt_pwm: lcd-bklt-pwm-state {
>> +        pins = "gpio59";
>> +        function = "gpio";
>> +        bias-disable;
>> +    };
>> +};
>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>> new file mode 100644
>> index 000000000000..95ed335bcb08
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>> @@ -0,0 +1,616 @@
>> +// SPDX-License-Identifier: BSD-3-Clause
>> +/*
>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>> + */
>> +/dts-v1/;
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
>> +#include "talos.dtsi"
>> +#include "pm8150.dtsi"
>> +/ {
>> +    aliases {
>> +        mmc0 = &sdhc_1;
>> +        serial0 = &uart0;
>> +        serial1 = &uart7;
>> +    };
>> +
>> +    chosen {
>> +        stdout-path = "serial0:115200n8";
>> +    };
>> +
>> +    clocks {
>> +        can_osc: can-oscillator {
>> +            compatible = "fixed-clock";
>> +            clock-frequency = <20000000>;
>> +            #clock-cells = <0>;
>> +        };
>> +
>> +        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>;
>> +        };
>> +    };
>> +
>> +    regulator-usb2-vbus {
>> +        compatible = "regulator-fixed";
>> +        regulator-name = "USB2_VBUS";
>> +        gpio = <&pm8150_gpios 10 GPIO_ACTIVE_HIGH>;
>> +        pinctrl-0 = <&usb2_en>;
>> +        pinctrl-names = "default";
>> +        enable-active-high;
>> +        regulator-always-on;
>> +    };
>> +
>> +    vreg_conn_1p8: regulator-conn-1p8 {
>> +        compatible = "regulator-fixed";
>> +        regulator-name = "vreg_conn_1p8";
>> +        startup-delay-us = <4000>;
>> +        enable-active-high;
>> +        gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
>> +    };
>> +
>> +    vreg_conn_pa: regulator-conn-pa {
>> +        compatible = "regulator-fixed";
>> +        regulator-name = "vreg_conn_pa";
>> +        startup-delay-us = <4000>;
>> +        enable-active-high;
>> +        gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
>> +    };
>> +
>> +    vreg_v3p3_can: regulator-v3p3-can {
>> +        compatible = "regulator-fixed";
>> +        regulator-name = "vreg-v3p3-can";
>> +        regulator-min-microvolt = <3300000>;
>> +        regulator-max-microvolt = <3300000>;
>> +        regulator-boot-on;
>> +        regulator-always-on;
>> +    };
>> +
>> +    vreg_v5p0_can: regulator-v5p0-can {
>> +        compatible = "regulator-fixed";
>> +        regulator-name = "vreg-v5p0-can";
>> +        regulator-min-microvolt = <5000000>;
>> +        regulator-max-microvolt = <5000000>;
>> +        regulator-boot-on;
>> +        regulator-always-on;
>> +    };
>> +
>> +    wcn6855-pmu {
>> +        compatible = "qcom,wcn6855-pmu";
>> +
>> +        pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
>> +        pinctrl-names = "default";
>> +
>> +        bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
>> +        wlan-enable-gpios = <&tlmm 84 GPIO_ACTIVE_HIGH>;
>> +
>> +        vddio-supply = <&vreg_conn_pa>;
>> +        vddaon-supply = <&vreg_s5a>;
>> +        vddpmu-supply = <&vreg_conn_1p8>;
>> +        vddpmumx-supply = <&vreg_conn_1p8>;
>> +        vddpmucx-supply = <&vreg_conn_pa>;
>> +        vddrfa0p95-supply = <&vreg_s5a>;
>> +        vddrfa1p3-supply = <&vreg_s6a>;
>> +        vddrfa1p9-supply = <&vreg_l15a>;
>> +        vddpcie1p3-supply = <&vreg_s6a>;
>> +        vddpcie1p9-supply = <&vreg_l15a>;
>> +
>> +        regulators {
>> +            vreg_pmu_rfa_cmn: ldo0 {
>> +                regulator-name = "vreg_pmu_rfa_cmn";
>> +            };
>> +
>> +            vreg_pmu_aon_0p59: ldo1 {
>> +                regulator-name = "vreg_pmu_aon_0p59";
>> +            };
>> +
>> +            vreg_pmu_wlcx_0p8: ldo2 {
>> +                regulator-name = "vreg_pmu_wlcx_0p8";
>> +            };
>> +
>> +            vreg_pmu_wlmx_0p85: ldo3 {
>> +                regulator-name = "vreg_pmu_wlmx_0p85";
>> +            };
>> +
>> +            vreg_pmu_btcmx_0p85: ldo4 {
>> +                regulator-name = "vreg_pmu_btcmx_0p85";
>> +            };
>> +
>> +            vreg_pmu_rfa_0p8: ldo5 {
>> +                regulator-name = "vreg_pmu_rfa_0p8";
>> +            };
>> +
>> +            vreg_pmu_rfa_1p2: ldo6 {
>> +                regulator-name = "vreg_pmu_rfa_1p2";
>> +            };
>> +
>> +            vreg_pmu_rfa_1p7: ldo7 {
>> +                regulator-name = "vreg_pmu_rfa_1p7";
>> +            };
>> +
>> +            vreg_pmu_pcie_0p9: ldo8 {
>> +                regulator-name = "vreg_pmu_pcie_0p9";
>> +            };
>> +
>> +            vreg_pmu_pcie_1p8: ldo9 {
>> +                regulator-name = "vreg_pmu_pcie_1p8";
>> +            };
>> +        };
>> +    };
>> +
>> +    wifi_1p8v: regulator-wifi-1p8v {
>> +        compatible = "regulator-fixed";
>> +        regulator-name = "wifi_1p8v";
>> +        regulator-min-microvolt = <1800000>;
>> +        regulator-max-microvolt = <1800000>;
>> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> Please check this pin number
>> +        enable-active-high;
>> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
>> +        pinctrl-names = "default";
>> +        regulator-boot-on;
>> +        regulator-always-on;
>> +    };
>> +
>> +    wifi_3p85v: regulator-wifi-3p85v {
>> +        compatible = "regulator-fixed";
>> +        regulator-name = "wifi_3p85v";
>> +        regulator-min-microvolt = <3850000>;
>> +        regulator-max-microvolt = <3850000>;
>> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> Please check this pin number
>> +        enable-active-high;
>> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
>> +        pinctrl-names = "default";
>> +        regulator-boot-on;
>> +        regulator-always-on;
>> +    };
>> +};
> 
> Are these two node necessary?
>

On this board, GPIO91 is wired as a common enable for both WiFi 
power rails: WiFi 1.8V and WiFi 3.85V.
I currently modeled them as two regulator-fixed nodes because these 
are two distinct rails.
Would you prefer modelling a single regulator node that controls the 
shared GPIO as below:

wifi_en: regulator-wifi-en {
    compatible = "regulator-fixed";
    regulator-name = "wifi_en";
    gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
    enable-active-high;
    pinctrl-0 = <&wifi_reg_en_pins_state>;
    pinctrl-names = "default";
    regulator-boot-on;
    regulator-always-on;
};
 
>> +
>> +&apps_rsc {
>> +    regulators-0 {
>> +        compatible = "qcom,pm8150-rpmh-regulators";
>> +        qcom,pmic-id = "a";
>> +
>> +        vreg_s3a: smps3 {
>> +            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>;
>> +            regulator-initial-mode = <RPMH_REGULATOR_MODE_LPM>;
>> +            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>;
>> +        };
>> +    };
>> +};
>> +
>> +&gpi_dma0 {
>> +    status = "okay";
>> +};
>> +
>> +&gpi_dma1 {
>> +    status = "okay";
>> +};
>> +
>> +&i2c5 {
>> +    clock-frequency = <400000>;
>> +    status = "okay";
>> +
>> +    eeprom@57 {
>> +        compatible = "atmel,24c02";
>> +        reg = <0x57>;
>> +        pagesize = <16>;
>> +    };
>> +
>> +    eeprom@5f {
>> +        compatible = "atmel,24mac602";
>> +        reg = <0x5f>;
>> +        pagesize = <16>;
>> +    };
>> +};
>> +
>> +&mdss {
>> +    status = "okay";
>> +};
>> +
>> +&mdss_dp0 {
>> +    status = "okay";
>> +};
>> +
>> +&mdss_dp0_out {
>> +    link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000>;
>> +    remote-endpoint = <&dp0_connector_in>;
>> +};
>> +
>> +&mdss_dsi0 {
>> +    vdda-supply = <&vreg_l11a>;
>> +    status = "okay";
>> +};
>> +
>> +&mdss_dsi0_phy {
>> +    vcca-supply = <&vreg_l5a>;
>> +    status = "okay";
>> +};
>> +
>> +&pcie {
>> +    perst-gpios = <&tlmm 89 GPIO_ACTIVE_LOW>;
>> +    wake-gpios = <&tlmm 100 GPIO_ACTIVE_HIGH>;
>> +
>> +    pinctrl-0 = <&pcie_default_state>;
>> +    pinctrl-names = "default";
>> +
>> +    status = "okay";
>> +};
>> +
>> +&pcie_phy {
>> +    vdda-phy-supply = <&vreg_l5a>;
>> +    vdda-pll-supply = <&vreg_l12a>;
>> +
>> +    status = "okay";
>> +};
>> +
>> +&pcie_port0 {
>> +    wifi@0 {
>> +        compatible = "pci17cb,1103";
> 
> This style of writing is not very readable.
> 
>> +        reg = <0x10000 0x0 0x0 0x0 0x0>;
>> +
>> +        qcom,calibration-variant = "QC_QCS615_Ride";
>> +
>> +        vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
>> +        vddaon-supply = <&vreg_pmu_aon_0p59>;
>> +        vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
>> +        vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
>> +        vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
>> +        vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
>> +        vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
>> +        vddpcie0p9-supply = <&vreg_pmu_pcie_0p9>;
>> +        vddpcie1p8-supply = <&vreg_pmu_pcie_1p8>;
>> +    };
>> +};
>> +
>> +&pm8150_gpios {
>> +    usb2_en: usb2-en-state {
>> +        pins = "gpio10";
>> +        function = "normal";
>> +        output-enable;
>> +        power-source = <0>;
>> +    };
>> +};
>> +
>> +&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";
>> +};
>> +
>> +&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";
>> +};
>> +
>> +&spi6 {
>> +    status = "okay";
>> +
>> +    can@0 {
>> +        compatible = "microchip,mcp2515";
>> +        reg = <0>;
>> +        clocks = <&can_osc>;
>> +        interrupts-extended = <&tlmm 87 IRQ_TYPE_LEVEL_LOW>;
>> +        spi-max-frequency = <10000000>;
>> +        vdd-supply = <&vreg_v3p3_can>;
>> +        xceiver-supply = <&vreg_v5p0_can>;
>> +    };
>> +};
>> +
>> +&tlmm {
>> +    bt_en_state: bt-en-state {
>> +        pins = "gpio85";
>> +        function = "gpio";
>> +        bias-pull-down;
>> +    };
>> +
>> +    pcie_default_state: pcie-default-state {
>> +        clkreq-pins {
>> +            pins = "gpio90";
>> +            function = "pcie_clk_req";
>> +            drive-strength = <2>;
>> +            bias-pull-up;
>> +        };
>> +
>> +        perst-pins {
>> +            pins = "gpio89";
>> +            function = "gpio";
>> +            drive-strength = <2>;
>> +            bias-disable;
>> +        };
>> +
>> +        wake-pins {
>> +            pins = "gpio100";
>> +            function = "gpio";
>> +            drive-strength = <2>;
>> +            bias-pull-up;
>> +        };
>> +    };
>> +
>> +    wifi_reg_en_pins_state: wifi-reg-en-pins-state {
>> +        pins = "gpio91";
>> +        function = "gpio";
>> +        drive-strength = <8>;
>> +        output-high;
>> +        bias-pull-up;
>> +    };
>> +
>> +    wlan_en_state: wlan-en-state {
>> +        pins = "gpio84";
>> +        function = "gpio";
>> +        drive-strength = <16>;
>> +        bias-pull-down;
>> +        output-low;
>> +    };
>> +};
>> +
>> +&uart0 {
>> +    status = "okay";
>> +};
>> +
>> +&uart7 {
>> +    status = "okay";
>> +
>> +    bluetooth {
>> +        compatible = "qcom,wcn6855-bt";
>> +        firmware-name = "QCA6698/hpnv21", "QCA6698/hpbtfw21.tlv";
>> +
>> +        vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
>> +        vddaon-supply = <&vreg_pmu_aon_0p59>;
>> +        vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
>> +        vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
>> +        vddbtcmx-supply = <&vreg_pmu_btcmx_0p85>;
>> +        vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
>> +        vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
>> +        vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
>> +    };
>> +};
>> +
>> +&usb_1 {
>> +    status = "okay";
>> +};
>> +
>> +&usb_1_dwc3 {
>> +    dr_mode = "host";
>> +};
>> +
>> +&usb_hsphy_1 {
>> +    vdd-supply = <&vreg_l5a>;
>> +    vdda-pll-supply = <&vreg_l12a>;
>> +    vdda-phy-dpdm-supply = <&vreg_l13a>;
>> +
>> +    status = "okay";
>> +};
>> +
>> +&usb_2 {
>> +    status = "okay";
>> +};
>> +
>> +&usb_2_dwc3 {
>> +    dr_mode = "host";
>> +};
>> +
>> +&usb_hsphy_2 {
>> +    vdd-supply = <&vreg_l5a>;
>> +    vdda-pll-supply = <&vreg_l12a>;
>> +    vdda-phy-dpdm-supply = <&vreg_l13a>;
>> +
>> +    status = "okay";
>> +};
>> +
>> +&usb_qmpphy {
>> +    vdda-phy-supply = <&vreg_l5a>;
>> +    vdda-pll-supply = <&vreg_l12a>;
>> +
>> +    status = "okay";
>> +};
>> +
>> +&usb_qmpphy_2 {
>> +    vdda-phy-supply = <&vreg_l11a>;
>> +    vdda-pll-supply = <&vreg_l5a>;
>> +
>> +    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";
>> +};
>> +
>> +&venus {
>> +    status = "okay";
>> +};
>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk.dts b/arch/arm64/boot/dts/qcom/talos-evk.dts
>> new file mode 100644
>> index 000000000000..af100e22beee
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/qcom/talos-evk.dts
>> @@ -0,0 +1,139 @@
>> +// SPDX-License-Identifier: BSD-3-Clause
>> +/*
>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>> + */
>> +/dts-v1/;
>> +
>> +#include "talos-evk-som.dtsi"
>> +
>> +/ {
>> +    model = "Qualcomm QCS615 IQ 615 EVK";
>> +    compatible = "qcom,talos-evk", "qcom,qcs615", "qcom,sm6150";
>> +    chassis-type = "embedded";
>> +
>> +    aliases {
>> +        mmc1 = &sdhc_2;
>> +    };
>> +
>> +    dp0-connector {
>> +        compatible = "dp-connector";
>> +        label = "DP0";
>> +        type = "full-size";
>> +
>> +        hpd-gpios = <&tlmm 104 GPIO_ACTIVE_HIGH>;
>> +
>> +        port {
>> +            dp0_connector_in: endpoint {
>> +                remote-endpoint = <&mdss_dp0_out>;
>> +            };
>> +        };
>> +    };
>> +
>> +    hdmi_connector: hdmi-out {
>> +        compatible = "hdmi-connector";
>> +        type = "d";
>> +
>> +        port {
>> +            hdmi_con_out: endpoint {
>> +            remote-endpoint = <&adv7535_out>;
>> +            };
>> +        };
>> +    };
>> +
>> +    vreg_v1p8_out: regulator-v1p8-out {
>> +        compatible = "regulator-fixed";
>> +        regulator-name = "vreg-v1p8-out";
>> +        regulator-min-microvolt = <1800000>;
>> +        regulator-max-microvolt = <1800000>;
>> +        vin-supply = <&vreg_v5p0_out>;
>> +        regulator-boot-on;
>> +        regulator-always-on;
>> +    };
>> +
>> +    vreg_v3p3_out: regulator-v3p3-out {
>> +        compatible = "regulator-fixed";
>> +        regulator-name = "vreg-v3p3-out";
>> +        regulator-min-microvolt = <3300000>;
>> +        regulator-max-microvolt = <3300000>;
>> +        vin-supply = <&vreg_v5p0_out>;
>> +        regulator-boot-on;
>> +        regulator-always-on;
>> +    };
>> +
>> +    vreg_v5p0_out: regulator-v5p0-out {
>> +        compatible = "regulator-fixed";
>> +        regulator-name = "vreg-v5p0-out";
>> +        regulator-min-microvolt = <5000000>;
>> +        regulator-max-microvolt = <5000000>;
>> +        regulator-boot-on;
>> +        regulator-always-on;
>> +        /* Powered by system 20V rail (USBC_VBUS_IN) */
>> +    };
>> +};
>> +
>> +&i2c1 {
>> +    clock-frequency = <400000>;
>> +    status = "okay";
>> +
>> +    hdmi_bridge: bridge@3d {
>> +        compatible = "adi,adv7535";
>> +        reg = <0x3d>;
>> +        avdd-supply = <&vreg_v1p8_out>;
>> +        dvdd-supply = <&vreg_v1p8_out>;
>> +        pvdd-supply = <&vreg_v1p8_out>;
>> +        a2vdd-supply = <&vreg_v1p8_out>;
>> +        v3p3-supply = <&vreg_v3p3_out>;
>> +        interrupts-extended = <&tlmm 26 IRQ_TYPE_LEVEL_LOW>;
>> +        adi,dsi-lanes = <4>;
>> +
>> +        ports {
>> +            #address-cells = <1>;
>> +            #size-cells = <0>;
>> +
>> +            port@0 {
>> +                reg = <0>;
>> +
>> +                adv7535_in: endpoint {
>> +                    remote-endpoint = <&mdss_dsi0_out>;
>> +                };
>> +            };
>> +
>> +            port@1 {
>> +                reg = <1>;
>> +
>> +                adv7535_out: endpoint {
>> +                    remote-endpoint = <&hdmi_con_out>;
>> +                };
>> +            };
>> +        };
>> +    };
>> +};
>> +
>> +&mdss_dsi0_out {
>> +    remote-endpoint = <&adv7535_in>;
>> +    data-lanes = <0 1 2 3>;
>> +};
>> +
>> +&pon_pwrkey {
>> +    status = "okay";
>> +};
>> +
>> +&pon_resin {
>> +    linux,code = <KEY_VOLUMEDOWN>;
>> +
>> +    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";
>> +};
> 

Re: [PATCH v11 4/4] arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

[Dmitry Baryshkov]

On Fri, Jan 16, 2026 at 12:28:22PM +0530, tessolveupstream@gmail.com wrote:
> 
> 
> On 15-01-2026 18:47, Qian Zhang wrote:
> > 
> > 
> > On 1/9/2026 5:52 PM, Sudarshan Shetty wrote:
> >> Add the device tree for the QCS615-based Talos EVK platform. The
> >> platform is composed of a System-on-Module following the SMARC
> >> standard, and a Carrier Board.
> >>
> >> The Carrier Board supports several display configurations, HDMI and
> >> LVDS. Both configurations use the same base hardware, with the display
> >> selection controlled by a DIP switch.
> >>
> >> Use a DTBO file, talos-evk-lvds-auo,g133han01.dtso, which defines an
> >> overlay that disables HDMI and adds LVDS. The DTs file talos-evk
> >> can describe the HDMI display configurations.
> >>
> >> The initial device tree includes support for:
> >> - CPU and memory
> >> - UART
> >> - GPIOs
> >> - Regulators
> >> - PMIC
> >> - Early console
> >> - AT24MAC602 EEPROM
> >> - MCP2515 SPI to CAN
> >> - ADV7535 DSI-to-HDMI bridge
> >> - DisplayPort interface
> >> - SN65DSI84ZXHR DSI-to-LVDS bridge
> >> - Wi-Fi/BT
> >>
> >> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> >> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
> >> ---
> >>   arch/arm64/boot/dts/qcom/Makefile             |   4 +
> >>   .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
> >>   arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
> >>   arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
> >>   4 files changed, 885 insertions(+)
> >>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> >>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> >>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts
> >>
> >> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> >> index 76cf0115a00a..289b651ef0c5 100644
> >> --- a/arch/arm64/boot/dts/qcom/Makefile
> >> +++ b/arch/arm64/boot/dts/qcom/Makefile
> >> @@ -324,6 +324,10 @@ dtb-$(CONFIG_ARCH_QCOM)    += sm8650-mtp.dtb
> >>   dtb-$(CONFIG_ARCH_QCOM)    += sm8650-qrd.dtb
> >>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-mtp.dtb
> >>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-qrd.dtb
> >> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk.dtb
> >> +talos-evk-lvds-auo,g133han01-dtbs    := \
> >> +    talos-evk.dtb talos-evk-lvds-auo,g133han01.dtbo
> >> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk-lvds-auo,g133han01.dtb
> >>   x1e001de-devkit-el2-dtbs    := x1e001de-devkit.dtb x1-el2.dtbo
> >>   dtb-$(CONFIG_ARCH_QCOM)    += x1e001de-devkit.dtb x1e001de-devkit-el2.dtb
> >>   x1e78100-lenovo-thinkpad-t14s-el2-dtbs    := x1e78100-lenovo-thinkpad-t14s.dtb x1-el2.dtbo
> >> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> >> new file mode 100644
> >> index 000000000000..ad058cf4cd93
> >> --- /dev/null
> >> +++ b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> >> @@ -0,0 +1,126 @@
> >> +// SPDX-License-Identifier: BSD-3-Clause
> >> +/*
> >> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> >> + */
> >> +/dts-v1/;
> >> +/plugin/;
> >> +
> >> +#include <dt-bindings/gpio/gpio.h>
> >> +
> >> +&{/} {
> >> +    backlight: backlight {
> >> +        compatible = "gpio-backlight";
> >> +        gpios = <&tlmm 59 GPIO_ACTIVE_HIGH>,
> >> +            <&tlmm 115 GPIO_ACTIVE_HIGH>;
> >> +        default-on;
> >> +    };
> >> +
> >> +    panel-lvds {
> >> +        compatible = "auo,g133han01";
> >> +
> >> +        ports {
> >> +            #address-cells = <1>;
> >> +            #size-cells = <0>;
> >> +
> >> +            /* LVDS A (Odd pixels) */
> >> +            port@0 {
> >> +                reg = <0>;
> >> +                dual-lvds-odd-pixels;
> >> +
> >> +                lvds_panel_out_a: endpoint {
> >> +                    remote-endpoint = <&sn65dsi84_out_a>;
> >> +                };
> >> +            };
> >> +
> >> +            /* LVDS B (Even pixels) */
> >> +            port@1 {
> >> +                reg = <1>;
> >> +                dual-lvds-even-pixels;
> >> +
> >> +                lvds_panel_out_b: endpoint {
> >> +                    remote-endpoint = <&sn65dsi84_out_b>;
> >> +                };
> >> +            };
> >> +        };
> >> +    };
> >> +};
> >> +
> >> +&hdmi_connector {
> >> +    status = "disabled";
> >> +};
> >> +
> >> +&i2c1 {
> >> +    clock-frequency = <400000>;
> >> +
> >> +    status = "okay";
> >> +
> >> +    hdmi_bridge: bridge@3d {
> >> +        status = "disabled";
> >> +    };
> >> +
> >> +    lvds_bridge: bridge@2c {
> >> +        compatible = "ti,sn65dsi84";
> >> +        reg = <0x2c>;
> >> +        enable-gpios = <&tlmm 42 GPIO_ACTIVE_HIGH>;
> >> +        ti,dsi-lanes = <4>;
> >> +        ti,lvds-format = "jeida-24";
> >> +        ti,lvds-bpp = <24>;
> >> +
> >> +        ports {
> >> +            #address-cells = <1>;
> >> +            #size-cells = <0>;
> >> +
> >> +            port@0 {
> >> +                reg = <0>;
> >> +
> >> +                sn65dsi84_in: endpoint {
> >> +                    data-lanes = <1 2 3 4>;
> >> +                    remote-endpoint = <&mdss_dsi0_out>;
> >> +                };
> >> +            };
> >> +
> >> +            port@2 {
> >> +                reg = <2>;
> >> +
> >> +                sn65dsi84_out_a: endpoint {
> >> +                    data-lanes = <1 2 3 4>;
> >> +                    remote-endpoint = <&lvds_panel_out_a>;
> >> +                };
> >> +            };
> >> +
> >> +            port@3 {
> >> +                reg = <3>;
> >> +
> >> +                sn65dsi84_out_b: endpoint {
> >> +                    data-lanes = <1 2 3 4>;
> >> +                    remote-endpoint = <&lvds_panel_out_b>;
> >> +                };
> >> +            };
> >> +        };
> >> +    };
> >> +};
> >> +
> >> +&mdss_dsi0 {
> >> +    vdda-supply = <&vreg_l11a>;
> >> +
> >> +    status = "okay";
> >> +};
> >> +
> >> +&mdss_dsi0_out {
> >> +    remote-endpoint = <&sn65dsi84_in>;
> >> +    data-lanes = <0 1 2 3>;
> >> +};
> >> +
> >> +&tlmm {
> >> +    lcd_bklt_en: lcd-bklt-en-state {
> >> +        pins = "gpio115";
> >> +        function = "gpio";
> >> +        bias-disable;
> >> +    };
> >> +
> >> +    lcd_bklt_pwm: lcd-bklt-pwm-state {
> >> +        pins = "gpio59";
> >> +        function = "gpio";
> >> +        bias-disable;
> >> +    };
> >> +};
> >> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> >> new file mode 100644
> >> index 000000000000..95ed335bcb08
> >> --- /dev/null
> >> +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> >> @@ -0,0 +1,616 @@
> >> +// SPDX-License-Identifier: BSD-3-Clause
> >> +/*
> >> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> >> + */
> >> +/dts-v1/;
> >> +
> >> +#include <dt-bindings/gpio/gpio.h>
> >> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> >> +#include "talos.dtsi"
> >> +#include "pm8150.dtsi"
> >> +/ {
> >> +    aliases {
> >> +        mmc0 = &sdhc_1;
> >> +        serial0 = &uart0;
> >> +        serial1 = &uart7;
> >> +    };
> >> +
> >> +    chosen {
> >> +        stdout-path = "serial0:115200n8";
> >> +    };
> >> +
> >> +    clocks {
> >> +        can_osc: can-oscillator {
> >> +            compatible = "fixed-clock";
> >> +            clock-frequency = <20000000>;
> >> +            #clock-cells = <0>;
> >> +        };
> >> +
> >> +        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>;
> >> +        };
> >> +    };
> >> +
> >> +    regulator-usb2-vbus {
> >> +        compatible = "regulator-fixed";
> >> +        regulator-name = "USB2_VBUS";
> >> +        gpio = <&pm8150_gpios 10 GPIO_ACTIVE_HIGH>;
> >> +        pinctrl-0 = <&usb2_en>;
> >> +        pinctrl-names = "default";
> >> +        enable-active-high;
> >> +        regulator-always-on;
> >> +    };
> >> +
> >> +    vreg_conn_1p8: regulator-conn-1p8 {
> >> +        compatible = "regulator-fixed";
> >> +        regulator-name = "vreg_conn_1p8";
> >> +        startup-delay-us = <4000>;
> >> +        enable-active-high;
> >> +        gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
> >> +    };
> >> +
> >> +    vreg_conn_pa: regulator-conn-pa {
> >> +        compatible = "regulator-fixed";
> >> +        regulator-name = "vreg_conn_pa";
> >> +        startup-delay-us = <4000>;
> >> +        enable-active-high;
> >> +        gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
> >> +    };
> >> +
> >> +    vreg_v3p3_can: regulator-v3p3-can {
> >> +        compatible = "regulator-fixed";
> >> +        regulator-name = "vreg-v3p3-can";
> >> +        regulator-min-microvolt = <3300000>;
> >> +        regulator-max-microvolt = <3300000>;
> >> +        regulator-boot-on;
> >> +        regulator-always-on;
> >> +    };
> >> +
> >> +    vreg_v5p0_can: regulator-v5p0-can {
> >> +        compatible = "regulator-fixed";
> >> +        regulator-name = "vreg-v5p0-can";
> >> +        regulator-min-microvolt = <5000000>;
> >> +        regulator-max-microvolt = <5000000>;
> >> +        regulator-boot-on;
> >> +        regulator-always-on;
> >> +    };
> >> +
> >> +    wcn6855-pmu {
> >> +        compatible = "qcom,wcn6855-pmu";
> >> +
> >> +        pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
> >> +        pinctrl-names = "default";
> >> +
> >> +        bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
> >> +        wlan-enable-gpios = <&tlmm 84 GPIO_ACTIVE_HIGH>;
> >> +
> >> +        vddio-supply = <&vreg_conn_pa>;
> >> +        vddaon-supply = <&vreg_s5a>;
> >> +        vddpmu-supply = <&vreg_conn_1p8>;
> >> +        vddpmumx-supply = <&vreg_conn_1p8>;
> >> +        vddpmucx-supply = <&vreg_conn_pa>;
> >> +        vddrfa0p95-supply = <&vreg_s5a>;
> >> +        vddrfa1p3-supply = <&vreg_s6a>;
> >> +        vddrfa1p9-supply = <&vreg_l15a>;
> >> +        vddpcie1p3-supply = <&vreg_s6a>;
> >> +        vddpcie1p9-supply = <&vreg_l15a>;
> >> +
> >> +        regulators {
> >> +            vreg_pmu_rfa_cmn: ldo0 {
> >> +                regulator-name = "vreg_pmu_rfa_cmn";
> >> +            };
> >> +
> >> +            vreg_pmu_aon_0p59: ldo1 {
> >> +                regulator-name = "vreg_pmu_aon_0p59";
> >> +            };
> >> +
> >> +            vreg_pmu_wlcx_0p8: ldo2 {
> >> +                regulator-name = "vreg_pmu_wlcx_0p8";
> >> +            };
> >> +
> >> +            vreg_pmu_wlmx_0p85: ldo3 {
> >> +                regulator-name = "vreg_pmu_wlmx_0p85";
> >> +            };
> >> +
> >> +            vreg_pmu_btcmx_0p85: ldo4 {
> >> +                regulator-name = "vreg_pmu_btcmx_0p85";
> >> +            };
> >> +
> >> +            vreg_pmu_rfa_0p8: ldo5 {
> >> +                regulator-name = "vreg_pmu_rfa_0p8";
> >> +            };
> >> +
> >> +            vreg_pmu_rfa_1p2: ldo6 {
> >> +                regulator-name = "vreg_pmu_rfa_1p2";
> >> +            };
> >> +
> >> +            vreg_pmu_rfa_1p7: ldo7 {
> >> +                regulator-name = "vreg_pmu_rfa_1p7";
> >> +            };
> >> +
> >> +            vreg_pmu_pcie_0p9: ldo8 {
> >> +                regulator-name = "vreg_pmu_pcie_0p9";
> >> +            };
> >> +
> >> +            vreg_pmu_pcie_1p8: ldo9 {
> >> +                regulator-name = "vreg_pmu_pcie_1p8";
> >> +            };
> >> +        };
> >> +    };
> >> +
> >> +    wifi_1p8v: regulator-wifi-1p8v {
> >> +        compatible = "regulator-fixed";
> >> +        regulator-name = "wifi_1p8v";
> >> +        regulator-min-microvolt = <1800000>;
> >> +        regulator-max-microvolt = <1800000>;
> >> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> > Please check this pin number
> >> +        enable-active-high;
> >> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
> >> +        pinctrl-names = "default";
> >> +        regulator-boot-on;
> >> +        regulator-always-on;
> >> +    };
> >> +
> >> +    wifi_3p85v: regulator-wifi-3p85v {
> >> +        compatible = "regulator-fixed";
> >> +        regulator-name = "wifi_3p85v";
> >> +        regulator-min-microvolt = <3850000>;
> >> +        regulator-max-microvolt = <3850000>;
> >> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> > Please check this pin number
> >> +        enable-active-high;
> >> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
> >> +        pinctrl-names = "default";
> >> +        regulator-boot-on;
> >> +        regulator-always-on;
> >> +    };
> >> +};
> > 
> > Are these two node necessary?
> >
> 
> On this board, GPIO91 is wired as a common enable for both WiFi 
> power rails: WiFi 1.8V and WiFi 3.85V.
> I currently modeled them as two regulator-fixed nodes because these 
> are two distinct rails.
> Would you prefer modelling a single regulator node that controls the 
> shared GPIO as below:
> 
> wifi_en: regulator-wifi-en {
>     compatible = "regulator-fixed";
>     regulator-name = "wifi_en";
>     gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
>     enable-active-high;
>     pinctrl-0 = <&wifi_reg_en_pins_state>;
>     pinctrl-names = "default";
>     regulator-boot-on;
>     regulator-always-on;
> };

What is the voltage of this regulator? What does it represent? What
should be represented in the DT?

BTW: what is powered on by those regulators? I don't see them being
wired to the PMU.

-- 
With best wishes
Dmitry

Re: [PATCH v11 4/4] arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

[tessolveupstream]

On 16-01-2026 13:26, Dmitry Baryshkov wrote:
> On Fri, Jan 16, 2026 at 12:28:22PM +0530, tessolveupstream@gmail.com wrote:
>>
>>
>> On 15-01-2026 18:47, Qian Zhang wrote:
>>>
>>>
>>> On 1/9/2026 5:52 PM, Sudarshan Shetty wrote:
>>>> Add the device tree for the QCS615-based Talos EVK platform. The
>>>> platform is composed of a System-on-Module following the SMARC
>>>> standard, and a Carrier Board.
>>>>
>>>> The Carrier Board supports several display configurations, HDMI and
>>>> LVDS. Both configurations use the same base hardware, with the display
>>>> selection controlled by a DIP switch.
>>>>
>>>> Use a DTBO file, talos-evk-lvds-auo,g133han01.dtso, which defines an
>>>> overlay that disables HDMI and adds LVDS. The DTs file talos-evk
>>>> can describe the HDMI display configurations.
>>>>
>>>> The initial device tree includes support for:
>>>> - CPU and memory
>>>> - UART
>>>> - GPIOs
>>>> - Regulators
>>>> - PMIC
>>>> - Early console
>>>> - AT24MAC602 EEPROM
>>>> - MCP2515 SPI to CAN
>>>> - ADV7535 DSI-to-HDMI bridge
>>>> - DisplayPort interface
>>>> - SN65DSI84ZXHR DSI-to-LVDS bridge
>>>> - Wi-Fi/BT
>>>>
>>>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
>>>> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
>>>> ---
>>>>   arch/arm64/boot/dts/qcom/Makefile             |   4 +
>>>>   .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
>>>>   arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
>>>>   arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
>>>>   4 files changed, 885 insertions(+)
>>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts
>>>>
>>>> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
>>>> index 76cf0115a00a..289b651ef0c5 100644
>>>> --- a/arch/arm64/boot/dts/qcom/Makefile
>>>> +++ b/arch/arm64/boot/dts/qcom/Makefile
>>>> @@ -324,6 +324,10 @@ dtb-$(CONFIG_ARCH_QCOM)    += sm8650-mtp.dtb
>>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8650-qrd.dtb
>>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-mtp.dtb
>>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-qrd.dtb
>>>> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk.dtb
>>>> +talos-evk-lvds-auo,g133han01-dtbs    := \
>>>> +    talos-evk.dtb talos-evk-lvds-auo,g133han01.dtbo
>>>> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk-lvds-auo,g133han01.dtb
>>>>   x1e001de-devkit-el2-dtbs    := x1e001de-devkit.dtb x1-el2.dtbo
>>>>   dtb-$(CONFIG_ARCH_QCOM)    += x1e001de-devkit.dtb x1e001de-devkit-el2.dtb
>>>>   x1e78100-lenovo-thinkpad-t14s-el2-dtbs    := x1e78100-lenovo-thinkpad-t14s.dtb x1-el2.dtbo
>>>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>>>> new file mode 100644
>>>> index 000000000000..ad058cf4cd93
>>>> --- /dev/null
>>>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>>>> @@ -0,0 +1,126 @@
>>>> +// SPDX-License-Identifier: BSD-3-Clause
>>>> +/*
>>>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>>>> + */
>>>> +/dts-v1/;
>>>> +/plugin/;
>>>> +
>>>> +#include <dt-bindings/gpio/gpio.h>
>>>> +
>>>> +&{/} {
>>>> +    backlight: backlight {
>>>> +        compatible = "gpio-backlight";
>>>> +        gpios = <&tlmm 59 GPIO_ACTIVE_HIGH>,
>>>> +            <&tlmm 115 GPIO_ACTIVE_HIGH>;
>>>> +        default-on;
>>>> +    };
>>>> +
>>>> +    panel-lvds {
>>>> +        compatible = "auo,g133han01";
>>>> +
>>>> +        ports {
>>>> +            #address-cells = <1>;
>>>> +            #size-cells = <0>;
>>>> +
>>>> +            /* LVDS A (Odd pixels) */
>>>> +            port@0 {
>>>> +                reg = <0>;
>>>> +                dual-lvds-odd-pixels;
>>>> +
>>>> +                lvds_panel_out_a: endpoint {
>>>> +                    remote-endpoint = <&sn65dsi84_out_a>;
>>>> +                };
>>>> +            };
>>>> +
>>>> +            /* LVDS B (Even pixels) */
>>>> +            port@1 {
>>>> +                reg = <1>;
>>>> +                dual-lvds-even-pixels;
>>>> +
>>>> +                lvds_panel_out_b: endpoint {
>>>> +                    remote-endpoint = <&sn65dsi84_out_b>;
>>>> +                };
>>>> +            };
>>>> +        };
>>>> +    };
>>>> +};
>>>> +
>>>> +&hdmi_connector {
>>>> +    status = "disabled";
>>>> +};
>>>> +
>>>> +&i2c1 {
>>>> +    clock-frequency = <400000>;
>>>> +
>>>> +    status = "okay";
>>>> +
>>>> +    hdmi_bridge: bridge@3d {
>>>> +        status = "disabled";
>>>> +    };
>>>> +
>>>> +    lvds_bridge: bridge@2c {
>>>> +        compatible = "ti,sn65dsi84";
>>>> +        reg = <0x2c>;
>>>> +        enable-gpios = <&tlmm 42 GPIO_ACTIVE_HIGH>;
>>>> +        ti,dsi-lanes = <4>;
>>>> +        ti,lvds-format = "jeida-24";
>>>> +        ti,lvds-bpp = <24>;
>>>> +
>>>> +        ports {
>>>> +            #address-cells = <1>;
>>>> +            #size-cells = <0>;
>>>> +
>>>> +            port@0 {
>>>> +                reg = <0>;
>>>> +
>>>> +                sn65dsi84_in: endpoint {
>>>> +                    data-lanes = <1 2 3 4>;
>>>> +                    remote-endpoint = <&mdss_dsi0_out>;
>>>> +                };
>>>> +            };
>>>> +
>>>> +            port@2 {
>>>> +                reg = <2>;
>>>> +
>>>> +                sn65dsi84_out_a: endpoint {
>>>> +                    data-lanes = <1 2 3 4>;
>>>> +                    remote-endpoint = <&lvds_panel_out_a>;
>>>> +                };
>>>> +            };
>>>> +
>>>> +            port@3 {
>>>> +                reg = <3>;
>>>> +
>>>> +                sn65dsi84_out_b: endpoint {
>>>> +                    data-lanes = <1 2 3 4>;
>>>> +                    remote-endpoint = <&lvds_panel_out_b>;
>>>> +                };
>>>> +            };
>>>> +        };
>>>> +    };
>>>> +};
>>>> +
>>>> +&mdss_dsi0 {
>>>> +    vdda-supply = <&vreg_l11a>;
>>>> +
>>>> +    status = "okay";
>>>> +};
>>>> +
>>>> +&mdss_dsi0_out {
>>>> +    remote-endpoint = <&sn65dsi84_in>;
>>>> +    data-lanes = <0 1 2 3>;
>>>> +};
>>>> +
>>>> +&tlmm {
>>>> +    lcd_bklt_en: lcd-bklt-en-state {
>>>> +        pins = "gpio115";
>>>> +        function = "gpio";
>>>> +        bias-disable;
>>>> +    };
>>>> +
>>>> +    lcd_bklt_pwm: lcd-bklt-pwm-state {
>>>> +        pins = "gpio59";
>>>> +        function = "gpio";
>>>> +        bias-disable;
>>>> +    };
>>>> +};
>>>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>>>> new file mode 100644
>>>> index 000000000000..95ed335bcb08
>>>> --- /dev/null
>>>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>>>> @@ -0,0 +1,616 @@
>>>> +// SPDX-License-Identifier: BSD-3-Clause
>>>> +/*
>>>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>>>> + */
>>>> +/dts-v1/;
>>>> +
>>>> +#include <dt-bindings/gpio/gpio.h>
>>>> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
>>>> +#include "talos.dtsi"
>>>> +#include "pm8150.dtsi"
>>>> +/ {
>>>> +    aliases {
>>>> +        mmc0 = &sdhc_1;
>>>> +        serial0 = &uart0;
>>>> +        serial1 = &uart7;
>>>> +    };
>>>> +
>>>> +    chosen {
>>>> +        stdout-path = "serial0:115200n8";
>>>> +    };
>>>> +
>>>> +    clocks {
>>>> +        can_osc: can-oscillator {
>>>> +            compatible = "fixed-clock";
>>>> +            clock-frequency = <20000000>;
>>>> +            #clock-cells = <0>;
>>>> +        };
>>>> +
>>>> +        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>;
>>>> +        };
>>>> +    };
>>>> +
>>>> +    regulator-usb2-vbus {
>>>> +        compatible = "regulator-fixed";
>>>> +        regulator-name = "USB2_VBUS";
>>>> +        gpio = <&pm8150_gpios 10 GPIO_ACTIVE_HIGH>;
>>>> +        pinctrl-0 = <&usb2_en>;
>>>> +        pinctrl-names = "default";
>>>> +        enable-active-high;
>>>> +        regulator-always-on;
>>>> +    };
>>>> +
>>>> +    vreg_conn_1p8: regulator-conn-1p8 {
>>>> +        compatible = "regulator-fixed";
>>>> +        regulator-name = "vreg_conn_1p8";
>>>> +        startup-delay-us = <4000>;
>>>> +        enable-active-high;
>>>> +        gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
>>>> +    };
>>>> +
>>>> +    vreg_conn_pa: regulator-conn-pa {
>>>> +        compatible = "regulator-fixed";
>>>> +        regulator-name = "vreg_conn_pa";
>>>> +        startup-delay-us = <4000>;
>>>> +        enable-active-high;
>>>> +        gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
>>>> +    };
>>>> +
>>>> +    vreg_v3p3_can: regulator-v3p3-can {
>>>> +        compatible = "regulator-fixed";
>>>> +        regulator-name = "vreg-v3p3-can";
>>>> +        regulator-min-microvolt = <3300000>;
>>>> +        regulator-max-microvolt = <3300000>;
>>>> +        regulator-boot-on;
>>>> +        regulator-always-on;
>>>> +    };
>>>> +
>>>> +    vreg_v5p0_can: regulator-v5p0-can {
>>>> +        compatible = "regulator-fixed";
>>>> +        regulator-name = "vreg-v5p0-can";
>>>> +        regulator-min-microvolt = <5000000>;
>>>> +        regulator-max-microvolt = <5000000>;
>>>> +        regulator-boot-on;
>>>> +        regulator-always-on;
>>>> +    };
>>>> +
>>>> +    wcn6855-pmu {
>>>> +        compatible = "qcom,wcn6855-pmu";
>>>> +
>>>> +        pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
>>>> +        pinctrl-names = "default";
>>>> +
>>>> +        bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
>>>> +        wlan-enable-gpios = <&tlmm 84 GPIO_ACTIVE_HIGH>;
>>>> +
>>>> +        vddio-supply = <&vreg_conn_pa>;
>>>> +        vddaon-supply = <&vreg_s5a>;
>>>> +        vddpmu-supply = <&vreg_conn_1p8>;
>>>> +        vddpmumx-supply = <&vreg_conn_1p8>;
>>>> +        vddpmucx-supply = <&vreg_conn_pa>;
>>>> +        vddrfa0p95-supply = <&vreg_s5a>;
>>>> +        vddrfa1p3-supply = <&vreg_s6a>;
>>>> +        vddrfa1p9-supply = <&vreg_l15a>;
>>>> +        vddpcie1p3-supply = <&vreg_s6a>;
>>>> +        vddpcie1p9-supply = <&vreg_l15a>;
>>>> +
>>>> +        regulators {
>>>> +            vreg_pmu_rfa_cmn: ldo0 {
>>>> +                regulator-name = "vreg_pmu_rfa_cmn";
>>>> +            };
>>>> +
>>>> +            vreg_pmu_aon_0p59: ldo1 {
>>>> +                regulator-name = "vreg_pmu_aon_0p59";
>>>> +            };
>>>> +
>>>> +            vreg_pmu_wlcx_0p8: ldo2 {
>>>> +                regulator-name = "vreg_pmu_wlcx_0p8";
>>>> +            };
>>>> +
>>>> +            vreg_pmu_wlmx_0p85: ldo3 {
>>>> +                regulator-name = "vreg_pmu_wlmx_0p85";
>>>> +            };
>>>> +
>>>> +            vreg_pmu_btcmx_0p85: ldo4 {
>>>> +                regulator-name = "vreg_pmu_btcmx_0p85";
>>>> +            };
>>>> +
>>>> +            vreg_pmu_rfa_0p8: ldo5 {
>>>> +                regulator-name = "vreg_pmu_rfa_0p8";
>>>> +            };
>>>> +
>>>> +            vreg_pmu_rfa_1p2: ldo6 {
>>>> +                regulator-name = "vreg_pmu_rfa_1p2";
>>>> +            };
>>>> +
>>>> +            vreg_pmu_rfa_1p7: ldo7 {
>>>> +                regulator-name = "vreg_pmu_rfa_1p7";
>>>> +            };
>>>> +
>>>> +            vreg_pmu_pcie_0p9: ldo8 {
>>>> +                regulator-name = "vreg_pmu_pcie_0p9";
>>>> +            };
>>>> +
>>>> +            vreg_pmu_pcie_1p8: ldo9 {
>>>> +                regulator-name = "vreg_pmu_pcie_1p8";
>>>> +            };
>>>> +        };
>>>> +    };
>>>> +
>>>> +    wifi_1p8v: regulator-wifi-1p8v {
>>>> +        compatible = "regulator-fixed";
>>>> +        regulator-name = "wifi_1p8v";
>>>> +        regulator-min-microvolt = <1800000>;
>>>> +        regulator-max-microvolt = <1800000>;
>>>> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
>>> Please check this pin number
>>>> +        enable-active-high;
>>>> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
>>>> +        pinctrl-names = "default";
>>>> +        regulator-boot-on;
>>>> +        regulator-always-on;
>>>> +    };
>>>> +
>>>> +    wifi_3p85v: regulator-wifi-3p85v {
>>>> +        compatible = "regulator-fixed";
>>>> +        regulator-name = "wifi_3p85v";
>>>> +        regulator-min-microvolt = <3850000>;
>>>> +        regulator-max-microvolt = <3850000>;
>>>> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
>>> Please check this pin number
>>>> +        enable-active-high;
>>>> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
>>>> +        pinctrl-names = "default";
>>>> +        regulator-boot-on;
>>>> +        regulator-always-on;
>>>> +    };
>>>> +};
>>>
>>> Are these two node necessary?
>>>
>>
>> On this board, GPIO91 is wired as a common enable for both WiFi 
>> power rails: WiFi 1.8V and WiFi 3.85V.
>> I currently modeled them as two regulator-fixed nodes because these 
>> are two distinct rails.
>> Would you prefer modelling a single regulator node that controls the 
>> shared GPIO as below:
>>
>> wifi_en: regulator-wifi-en {
>>     compatible = "regulator-fixed";
>>     regulator-name = "wifi_en";
>>     gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
>>     enable-active-high;
>>     pinctrl-0 = <&wifi_reg_en_pins_state>;
>>     pinctrl-names = "default";
>>     regulator-boot-on;
>>     regulator-always-on;
>> };
> 
> What is the voltage of this regulator? What does it represent? What
> should be represented in the DT?
> 
> BTW: what is powered on by those regulators? I don't see them being
> wired to the PMU.
> 

There are two separate TPS62130RGTR regulator ICs on the board:
One provides 1.8V and Another provides 3.85V.

Both regulators use the same enable signal (SOC_GPIO_91_S4A_1P8 - 
GPIO91), which connects to their EN pins. 
When GPIO91 goes high, both regulators switch on at the same 
time.

I described them as two separate regulator-fixed nodes since they 
are different chips with different voltages.

As an alternative, I suggested representing them as a single node 
(based on the review comment from Qian Zhang) if you only want to 
show the shared GPIO enable control instead of the individual 
voltage rails.

In AF68E module, 3.85V regulator powers VDD_PA_5G and 1.8V regulator
powers VDD_PA_2G.

Re: [PATCH v11 4/4] arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

[Dmitry Baryshkov]

On Fri, Jan 16, 2026 at 04:57:42PM +0530, tessolveupstream@gmail.com wrote:
> 
> 
> On 16-01-2026 13:26, Dmitry Baryshkov wrote:
> > On Fri, Jan 16, 2026 at 12:28:22PM +0530, tessolveupstream@gmail.com wrote:
> >>
> >>
> >> On 15-01-2026 18:47, Qian Zhang wrote:
> >>>
> >>>
> >>> On 1/9/2026 5:52 PM, Sudarshan Shetty wrote:
> >>>> Add the device tree for the QCS615-based Talos EVK platform. The
> >>>> platform is composed of a System-on-Module following the SMARC
> >>>> standard, and a Carrier Board.
> >>>>
> >>>> The Carrier Board supports several display configurations, HDMI and
> >>>> LVDS. Both configurations use the same base hardware, with the display
> >>>> selection controlled by a DIP switch.
> >>>>
> >>>> Use a DTBO file, talos-evk-lvds-auo,g133han01.dtso, which defines an
> >>>> overlay that disables HDMI and adds LVDS. The DTs file talos-evk
> >>>> can describe the HDMI display configurations.
> >>>>
> >>>> The initial device tree includes support for:
> >>>> - CPU and memory
> >>>> - UART
> >>>> - GPIOs
> >>>> - Regulators
> >>>> - PMIC
> >>>> - Early console
> >>>> - AT24MAC602 EEPROM
> >>>> - MCP2515 SPI to CAN
> >>>> - ADV7535 DSI-to-HDMI bridge
> >>>> - DisplayPort interface
> >>>> - SN65DSI84ZXHR DSI-to-LVDS bridge
> >>>> - Wi-Fi/BT
> >>>>
> >>>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> >>>> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
> >>>> ---
> >>>>   arch/arm64/boot/dts/qcom/Makefile             |   4 +
> >>>>   .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
> >>>>   arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
> >>>>   arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
> >>>>   4 files changed, 885 insertions(+)
> >>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> >>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> >>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts
> >>>>
> >>>> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> >>>> index 76cf0115a00a..289b651ef0c5 100644
> >>>> --- a/arch/arm64/boot/dts/qcom/Makefile
> >>>> +++ b/arch/arm64/boot/dts/qcom/Makefile
> >>>> @@ -324,6 +324,10 @@ dtb-$(CONFIG_ARCH_QCOM)    += sm8650-mtp.dtb
> >>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8650-qrd.dtb
> >>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-mtp.dtb
> >>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-qrd.dtb
> >>>> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk.dtb
> >>>> +talos-evk-lvds-auo,g133han01-dtbs    := \
> >>>> +    talos-evk.dtb talos-evk-lvds-auo,g133han01.dtbo
> >>>> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk-lvds-auo,g133han01.dtb
> >>>>   x1e001de-devkit-el2-dtbs    := x1e001de-devkit.dtb x1-el2.dtbo
> >>>>   dtb-$(CONFIG_ARCH_QCOM)    += x1e001de-devkit.dtb x1e001de-devkit-el2.dtb
> >>>>   x1e78100-lenovo-thinkpad-t14s-el2-dtbs    := x1e78100-lenovo-thinkpad-t14s.dtb x1-el2.dtbo
> >>>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> >>>> new file mode 100644
> >>>> index 000000000000..ad058cf4cd93
> >>>> --- /dev/null
> >>>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> >>>> @@ -0,0 +1,126 @@
> >>>> +// SPDX-License-Identifier: BSD-3-Clause
> >>>> +/*
> >>>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> >>>> + */
> >>>> +/dts-v1/;
> >>>> +/plugin/;
> >>>> +
> >>>> +#include <dt-bindings/gpio/gpio.h>
> >>>> +
> >>>> +&{/} {
> >>>> +    backlight: backlight {
> >>>> +        compatible = "gpio-backlight";
> >>>> +        gpios = <&tlmm 59 GPIO_ACTIVE_HIGH>,
> >>>> +            <&tlmm 115 GPIO_ACTIVE_HIGH>;
> >>>> +        default-on;
> >>>> +    };
> >>>> +
> >>>> +    panel-lvds {
> >>>> +        compatible = "auo,g133han01";
> >>>> +
> >>>> +        ports {
> >>>> +            #address-cells = <1>;
> >>>> +            #size-cells = <0>;
> >>>> +
> >>>> +            /* LVDS A (Odd pixels) */
> >>>> +            port@0 {
> >>>> +                reg = <0>;
> >>>> +                dual-lvds-odd-pixels;
> >>>> +
> >>>> +                lvds_panel_out_a: endpoint {
> >>>> +                    remote-endpoint = <&sn65dsi84_out_a>;
> >>>> +                };
> >>>> +            };
> >>>> +
> >>>> +            /* LVDS B (Even pixels) */
> >>>> +            port@1 {
> >>>> +                reg = <1>;
> >>>> +                dual-lvds-even-pixels;
> >>>> +
> >>>> +                lvds_panel_out_b: endpoint {
> >>>> +                    remote-endpoint = <&sn65dsi84_out_b>;
> >>>> +                };
> >>>> +            };
> >>>> +        };
> >>>> +    };
> >>>> +};
> >>>> +
> >>>> +&hdmi_connector {
> >>>> +    status = "disabled";
> >>>> +};
> >>>> +
> >>>> +&i2c1 {
> >>>> +    clock-frequency = <400000>;
> >>>> +
> >>>> +    status = "okay";
> >>>> +
> >>>> +    hdmi_bridge: bridge@3d {
> >>>> +        status = "disabled";
> >>>> +    };
> >>>> +
> >>>> +    lvds_bridge: bridge@2c {
> >>>> +        compatible = "ti,sn65dsi84";
> >>>> +        reg = <0x2c>;
> >>>> +        enable-gpios = <&tlmm 42 GPIO_ACTIVE_HIGH>;
> >>>> +        ti,dsi-lanes = <4>;
> >>>> +        ti,lvds-format = "jeida-24";
> >>>> +        ti,lvds-bpp = <24>;
> >>>> +
> >>>> +        ports {
> >>>> +            #address-cells = <1>;
> >>>> +            #size-cells = <0>;
> >>>> +
> >>>> +            port@0 {
> >>>> +                reg = <0>;
> >>>> +
> >>>> +                sn65dsi84_in: endpoint {
> >>>> +                    data-lanes = <1 2 3 4>;
> >>>> +                    remote-endpoint = <&mdss_dsi0_out>;
> >>>> +                };
> >>>> +            };
> >>>> +
> >>>> +            port@2 {
> >>>> +                reg = <2>;
> >>>> +
> >>>> +                sn65dsi84_out_a: endpoint {
> >>>> +                    data-lanes = <1 2 3 4>;
> >>>> +                    remote-endpoint = <&lvds_panel_out_a>;
> >>>> +                };
> >>>> +            };
> >>>> +
> >>>> +            port@3 {
> >>>> +                reg = <3>;
> >>>> +
> >>>> +                sn65dsi84_out_b: endpoint {
> >>>> +                    data-lanes = <1 2 3 4>;
> >>>> +                    remote-endpoint = <&lvds_panel_out_b>;
> >>>> +                };
> >>>> +            };
> >>>> +        };
> >>>> +    };
> >>>> +};
> >>>> +
> >>>> +&mdss_dsi0 {
> >>>> +    vdda-supply = <&vreg_l11a>;
> >>>> +
> >>>> +    status = "okay";
> >>>> +};
> >>>> +
> >>>> +&mdss_dsi0_out {
> >>>> +    remote-endpoint = <&sn65dsi84_in>;
> >>>> +    data-lanes = <0 1 2 3>;
> >>>> +};
> >>>> +
> >>>> +&tlmm {
> >>>> +    lcd_bklt_en: lcd-bklt-en-state {
> >>>> +        pins = "gpio115";
> >>>> +        function = "gpio";
> >>>> +        bias-disable;
> >>>> +    };
> >>>> +
> >>>> +    lcd_bklt_pwm: lcd-bklt-pwm-state {
> >>>> +        pins = "gpio59";
> >>>> +        function = "gpio";
> >>>> +        bias-disable;
> >>>> +    };
> >>>> +};
> >>>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> >>>> new file mode 100644
> >>>> index 000000000000..95ed335bcb08
> >>>> --- /dev/null
> >>>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> >>>> @@ -0,0 +1,616 @@
> >>>> +// SPDX-License-Identifier: BSD-3-Clause
> >>>> +/*
> >>>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> >>>> + */
> >>>> +/dts-v1/;
> >>>> +
> >>>> +#include <dt-bindings/gpio/gpio.h>
> >>>> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> >>>> +#include "talos.dtsi"
> >>>> +#include "pm8150.dtsi"
> >>>> +/ {
> >>>> +    aliases {
> >>>> +        mmc0 = &sdhc_1;
> >>>> +        serial0 = &uart0;
> >>>> +        serial1 = &uart7;
> >>>> +    };
> >>>> +
> >>>> +    chosen {
> >>>> +        stdout-path = "serial0:115200n8";
> >>>> +    };
> >>>> +
> >>>> +    clocks {
> >>>> +        can_osc: can-oscillator {
> >>>> +            compatible = "fixed-clock";
> >>>> +            clock-frequency = <20000000>;
> >>>> +            #clock-cells = <0>;
> >>>> +        };
> >>>> +
> >>>> +        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>;
> >>>> +        };
> >>>> +    };
> >>>> +
> >>>> +    regulator-usb2-vbus {
> >>>> +        compatible = "regulator-fixed";
> >>>> +        regulator-name = "USB2_VBUS";
> >>>> +        gpio = <&pm8150_gpios 10 GPIO_ACTIVE_HIGH>;
> >>>> +        pinctrl-0 = <&usb2_en>;
> >>>> +        pinctrl-names = "default";
> >>>> +        enable-active-high;
> >>>> +        regulator-always-on;
> >>>> +    };
> >>>> +
> >>>> +    vreg_conn_1p8: regulator-conn-1p8 {
> >>>> +        compatible = "regulator-fixed";
> >>>> +        regulator-name = "vreg_conn_1p8";
> >>>> +        startup-delay-us = <4000>;
> >>>> +        enable-active-high;
> >>>> +        gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
> >>>> +    };
> >>>> +
> >>>> +    vreg_conn_pa: regulator-conn-pa {
> >>>> +        compatible = "regulator-fixed";
> >>>> +        regulator-name = "vreg_conn_pa";
> >>>> +        startup-delay-us = <4000>;
> >>>> +        enable-active-high;
> >>>> +        gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
> >>>> +    };
> >>>> +
> >>>> +    vreg_v3p3_can: regulator-v3p3-can {
> >>>> +        compatible = "regulator-fixed";
> >>>> +        regulator-name = "vreg-v3p3-can";
> >>>> +        regulator-min-microvolt = <3300000>;
> >>>> +        regulator-max-microvolt = <3300000>;
> >>>> +        regulator-boot-on;
> >>>> +        regulator-always-on;
> >>>> +    };
> >>>> +
> >>>> +    vreg_v5p0_can: regulator-v5p0-can {
> >>>> +        compatible = "regulator-fixed";
> >>>> +        regulator-name = "vreg-v5p0-can";
> >>>> +        regulator-min-microvolt = <5000000>;
> >>>> +        regulator-max-microvolt = <5000000>;
> >>>> +        regulator-boot-on;
> >>>> +        regulator-always-on;
> >>>> +    };
> >>>> +
> >>>> +    wcn6855-pmu {
> >>>> +        compatible = "qcom,wcn6855-pmu";
> >>>> +
> >>>> +        pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
> >>>> +        pinctrl-names = "default";
> >>>> +
> >>>> +        bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
> >>>> +        wlan-enable-gpios = <&tlmm 84 GPIO_ACTIVE_HIGH>;
> >>>> +
> >>>> +        vddio-supply = <&vreg_conn_pa>;
> >>>> +        vddaon-supply = <&vreg_s5a>;
> >>>> +        vddpmu-supply = <&vreg_conn_1p8>;
> >>>> +        vddpmumx-supply = <&vreg_conn_1p8>;
> >>>> +        vddpmucx-supply = <&vreg_conn_pa>;
> >>>> +        vddrfa0p95-supply = <&vreg_s5a>;
> >>>> +        vddrfa1p3-supply = <&vreg_s6a>;
> >>>> +        vddrfa1p9-supply = <&vreg_l15a>;
> >>>> +        vddpcie1p3-supply = <&vreg_s6a>;
> >>>> +        vddpcie1p9-supply = <&vreg_l15a>;
> >>>> +
> >>>> +        regulators {
> >>>> +            vreg_pmu_rfa_cmn: ldo0 {
> >>>> +                regulator-name = "vreg_pmu_rfa_cmn";
> >>>> +            };
> >>>> +
> >>>> +            vreg_pmu_aon_0p59: ldo1 {
> >>>> +                regulator-name = "vreg_pmu_aon_0p59";
> >>>> +            };
> >>>> +
> >>>> +            vreg_pmu_wlcx_0p8: ldo2 {
> >>>> +                regulator-name = "vreg_pmu_wlcx_0p8";
> >>>> +            };
> >>>> +
> >>>> +            vreg_pmu_wlmx_0p85: ldo3 {
> >>>> +                regulator-name = "vreg_pmu_wlmx_0p85";
> >>>> +            };
> >>>> +
> >>>> +            vreg_pmu_btcmx_0p85: ldo4 {
> >>>> +                regulator-name = "vreg_pmu_btcmx_0p85";
> >>>> +            };
> >>>> +
> >>>> +            vreg_pmu_rfa_0p8: ldo5 {
> >>>> +                regulator-name = "vreg_pmu_rfa_0p8";
> >>>> +            };
> >>>> +
> >>>> +            vreg_pmu_rfa_1p2: ldo6 {
> >>>> +                regulator-name = "vreg_pmu_rfa_1p2";
> >>>> +            };
> >>>> +
> >>>> +            vreg_pmu_rfa_1p7: ldo7 {
> >>>> +                regulator-name = "vreg_pmu_rfa_1p7";
> >>>> +            };
> >>>> +
> >>>> +            vreg_pmu_pcie_0p9: ldo8 {
> >>>> +                regulator-name = "vreg_pmu_pcie_0p9";
> >>>> +            };
> >>>> +
> >>>> +            vreg_pmu_pcie_1p8: ldo9 {
> >>>> +                regulator-name = "vreg_pmu_pcie_1p8";
> >>>> +            };
> >>>> +        };
> >>>> +    };
> >>>> +
> >>>> +    wifi_1p8v: regulator-wifi-1p8v {
> >>>> +        compatible = "regulator-fixed";
> >>>> +        regulator-name = "wifi_1p8v";
> >>>> +        regulator-min-microvolt = <1800000>;
> >>>> +        regulator-max-microvolt = <1800000>;
> >>>> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> >>> Please check this pin number
> >>>> +        enable-active-high;
> >>>> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
> >>>> +        pinctrl-names = "default";
> >>>> +        regulator-boot-on;
> >>>> +        regulator-always-on;
> >>>> +    };
> >>>> +
> >>>> +    wifi_3p85v: regulator-wifi-3p85v {
> >>>> +        compatible = "regulator-fixed";
> >>>> +        regulator-name = "wifi_3p85v";
> >>>> +        regulator-min-microvolt = <3850000>;
> >>>> +        regulator-max-microvolt = <3850000>;
> >>>> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> >>> Please check this pin number
> >>>> +        enable-active-high;
> >>>> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
> >>>> +        pinctrl-names = "default";
> >>>> +        regulator-boot-on;
> >>>> +        regulator-always-on;
> >>>> +    };
> >>>> +};
> >>>
> >>> Are these two node necessary?
> >>>
> >>
> >> On this board, GPIO91 is wired as a common enable for both WiFi 
> >> power rails: WiFi 1.8V and WiFi 3.85V.
> >> I currently modeled them as two regulator-fixed nodes because these 
> >> are two distinct rails.
> >> Would you prefer modelling a single regulator node that controls the 
> >> shared GPIO as below:
> >>
> >> wifi_en: regulator-wifi-en {
> >>     compatible = "regulator-fixed";
> >>     regulator-name = "wifi_en";
> >>     gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> >>     enable-active-high;
> >>     pinctrl-0 = <&wifi_reg_en_pins_state>;
> >>     pinctrl-names = "default";
> >>     regulator-boot-on;
> >>     regulator-always-on;
> >> };
> > 
> > What is the voltage of this regulator? What does it represent? What
> > should be represented in the DT?
> > 
> > BTW: what is powered on by those regulators? I don't see them being
> > wired to the PMU.
> > 
> 
> There are two separate TPS62130RGTR regulator ICs on the board:
> One provides 1.8V and Another provides 3.85V.
> 
> Both regulators use the same enable signal (SOC_GPIO_91_S4A_1P8 - 
> GPIO91), which connects to their EN pins. 
> When GPIO91 goes high, both regulators switch on at the same 
> time.

And gpiolib-shared.c should be able to support that.

> 
> I described them as two separate regulator-fixed nodes since they 
> are different chips with different voltages.

okay.

> 
> As an alternative, I suggested representing them as a single node 
> (based on the review comment from Qian Zhang) if you only want to 
> show the shared GPIO enable control instead of the individual 
> voltage rails.

Why do you need an alternative?

> 
> In AF68E module, 3.85V regulator powers VDD_PA_5G and 1.8V regulator
> powers VDD_PA_2G.

Do they power up the WCN6855 itself or some external PA present on the
module? In the former case, those regulators should be consumed by the
PMU, in the latter case, by the WiFi node. Anyway, they should not be
left as always-on / boot-on.

-- 
With best wishes
Dmitry

Re: [PATCH v11 4/4] arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

[tessolveupstream]

On 16-01-2026 17:19, Dmitry Baryshkov wrote:
> On Fri, Jan 16, 2026 at 04:57:42PM +0530, tessolveupstream@gmail.com wrote:
>>
>>
>> On 16-01-2026 13:26, Dmitry Baryshkov wrote:
>>> On Fri, Jan 16, 2026 at 12:28:22PM +0530, tessolveupstream@gmail.com wrote:
>>>>
>>>>
>>>> On 15-01-2026 18:47, Qian Zhang wrote:
>>>>>
>>>>>
>>>>> On 1/9/2026 5:52 PM, Sudarshan Shetty wrote:
>>>>>> Add the device tree for the QCS615-based Talos EVK platform. The
>>>>>> platform is composed of a System-on-Module following the SMARC
>>>>>> standard, and a Carrier Board.
>>>>>>
>>>>>> The Carrier Board supports several display configurations, HDMI and
>>>>>> LVDS. Both configurations use the same base hardware, with the display
>>>>>> selection controlled by a DIP switch.
>>>>>>
>>>>>> Use a DTBO file, talos-evk-lvds-auo,g133han01.dtso, which defines an
>>>>>> overlay that disables HDMI and adds LVDS. The DTs file talos-evk
>>>>>> can describe the HDMI display configurations.
>>>>>>
>>>>>> The initial device tree includes support for:
>>>>>> - CPU and memory
>>>>>> - UART
>>>>>> - GPIOs
>>>>>> - Regulators
>>>>>> - PMIC
>>>>>> - Early console
>>>>>> - AT24MAC602 EEPROM
>>>>>> - MCP2515 SPI to CAN
>>>>>> - ADV7535 DSI-to-HDMI bridge
>>>>>> - DisplayPort interface
>>>>>> - SN65DSI84ZXHR DSI-to-LVDS bridge
>>>>>> - Wi-Fi/BT
>>>>>>
>>>>>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
>>>>>> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
>>>>>> ---
>>>>>>   arch/arm64/boot/dts/qcom/Makefile             |   4 +
>>>>>>   .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
>>>>>>   arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
>>>>>>   arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
>>>>>>   4 files changed, 885 insertions(+)
>>>>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>>>>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>>>>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts
>>>>>>
>>>>>> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
>>>>>> index 76cf0115a00a..289b651ef0c5 100644
>>>>>> --- a/arch/arm64/boot/dts/qcom/Makefile
>>>>>> +++ b/arch/arm64/boot/dts/qcom/Makefile
>>>>>> @@ -324,6 +324,10 @@ dtb-$(CONFIG_ARCH_QCOM)    += sm8650-mtp.dtb
>>>>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8650-qrd.dtb
>>>>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-mtp.dtb
>>>>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-qrd.dtb
>>>>>> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk.dtb
>>>>>> +talos-evk-lvds-auo,g133han01-dtbs    := \
>>>>>> +    talos-evk.dtb talos-evk-lvds-auo,g133han01.dtbo
>>>>>> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk-lvds-auo,g133han01.dtb
>>>>>>   x1e001de-devkit-el2-dtbs    := x1e001de-devkit.dtb x1-el2.dtbo
>>>>>>   dtb-$(CONFIG_ARCH_QCOM)    += x1e001de-devkit.dtb x1e001de-devkit-el2.dtb
>>>>>>   x1e78100-lenovo-thinkpad-t14s-el2-dtbs    := x1e78100-lenovo-thinkpad-t14s.dtb x1-el2.dtbo
>>>>>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>>>>>> new file mode 100644
>>>>>> index 000000000000..ad058cf4cd93
>>>>>> --- /dev/null
>>>>>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
>>>>>> @@ -0,0 +1,126 @@
>>>>>> +// SPDX-License-Identifier: BSD-3-Clause
>>>>>> +/*
>>>>>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>>>>>> + */
>>>>>> +/dts-v1/;
>>>>>> +/plugin/;
>>>>>> +
>>>>>> +#include <dt-bindings/gpio/gpio.h>
>>>>>> +
>>>>>> +&{/} {
>>>>>> +    backlight: backlight {
>>>>>> +        compatible = "gpio-backlight";
>>>>>> +        gpios = <&tlmm 59 GPIO_ACTIVE_HIGH>,
>>>>>> +            <&tlmm 115 GPIO_ACTIVE_HIGH>;
>>>>>> +        default-on;
>>>>>> +    };
>>>>>> +
>>>>>> +    panel-lvds {
>>>>>> +        compatible = "auo,g133han01";
>>>>>> +
>>>>>> +        ports {
>>>>>> +            #address-cells = <1>;
>>>>>> +            #size-cells = <0>;
>>>>>> +
>>>>>> +            /* LVDS A (Odd pixels) */
>>>>>> +            port@0 {
>>>>>> +                reg = <0>;
>>>>>> +                dual-lvds-odd-pixels;
>>>>>> +
>>>>>> +                lvds_panel_out_a: endpoint {
>>>>>> +                    remote-endpoint = <&sn65dsi84_out_a>;
>>>>>> +                };
>>>>>> +            };
>>>>>> +
>>>>>> +            /* LVDS B (Even pixels) */
>>>>>> +            port@1 {
>>>>>> +                reg = <1>;
>>>>>> +                dual-lvds-even-pixels;
>>>>>> +
>>>>>> +                lvds_panel_out_b: endpoint {
>>>>>> +                    remote-endpoint = <&sn65dsi84_out_b>;
>>>>>> +                };
>>>>>> +            };
>>>>>> +        };
>>>>>> +    };
>>>>>> +};
>>>>>> +
>>>>>> +&hdmi_connector {
>>>>>> +    status = "disabled";
>>>>>> +};
>>>>>> +
>>>>>> +&i2c1 {
>>>>>> +    clock-frequency = <400000>;
>>>>>> +
>>>>>> +    status = "okay";
>>>>>> +
>>>>>> +    hdmi_bridge: bridge@3d {
>>>>>> +        status = "disabled";
>>>>>> +    };
>>>>>> +
>>>>>> +    lvds_bridge: bridge@2c {
>>>>>> +        compatible = "ti,sn65dsi84";
>>>>>> +        reg = <0x2c>;
>>>>>> +        enable-gpios = <&tlmm 42 GPIO_ACTIVE_HIGH>;
>>>>>> +        ti,dsi-lanes = <4>;
>>>>>> +        ti,lvds-format = "jeida-24";
>>>>>> +        ti,lvds-bpp = <24>;
>>>>>> +
>>>>>> +        ports {
>>>>>> +            #address-cells = <1>;
>>>>>> +            #size-cells = <0>;
>>>>>> +
>>>>>> +            port@0 {
>>>>>> +                reg = <0>;
>>>>>> +
>>>>>> +                sn65dsi84_in: endpoint {
>>>>>> +                    data-lanes = <1 2 3 4>;
>>>>>> +                    remote-endpoint = <&mdss_dsi0_out>;
>>>>>> +                };
>>>>>> +            };
>>>>>> +
>>>>>> +            port@2 {
>>>>>> +                reg = <2>;
>>>>>> +
>>>>>> +                sn65dsi84_out_a: endpoint {
>>>>>> +                    data-lanes = <1 2 3 4>;
>>>>>> +                    remote-endpoint = <&lvds_panel_out_a>;
>>>>>> +                };
>>>>>> +            };
>>>>>> +
>>>>>> +            port@3 {
>>>>>> +                reg = <3>;
>>>>>> +
>>>>>> +                sn65dsi84_out_b: endpoint {
>>>>>> +                    data-lanes = <1 2 3 4>;
>>>>>> +                    remote-endpoint = <&lvds_panel_out_b>;
>>>>>> +                };
>>>>>> +            };
>>>>>> +        };
>>>>>> +    };
>>>>>> +};
>>>>>> +
>>>>>> +&mdss_dsi0 {
>>>>>> +    vdda-supply = <&vreg_l11a>;
>>>>>> +
>>>>>> +    status = "okay";
>>>>>> +};
>>>>>> +
>>>>>> +&mdss_dsi0_out {
>>>>>> +    remote-endpoint = <&sn65dsi84_in>;
>>>>>> +    data-lanes = <0 1 2 3>;
>>>>>> +};
>>>>>> +
>>>>>> +&tlmm {
>>>>>> +    lcd_bklt_en: lcd-bklt-en-state {
>>>>>> +        pins = "gpio115";
>>>>>> +        function = "gpio";
>>>>>> +        bias-disable;
>>>>>> +    };
>>>>>> +
>>>>>> +    lcd_bklt_pwm: lcd-bklt-pwm-state {
>>>>>> +        pins = "gpio59";
>>>>>> +        function = "gpio";
>>>>>> +        bias-disable;
>>>>>> +    };
>>>>>> +};
>>>>>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>>>>>> new file mode 100644
>>>>>> index 000000000000..95ed335bcb08
>>>>>> --- /dev/null
>>>>>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
>>>>>> @@ -0,0 +1,616 @@
>>>>>> +// SPDX-License-Identifier: BSD-3-Clause
>>>>>> +/*
>>>>>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
>>>>>> + */
>>>>>> +/dts-v1/;
>>>>>> +
>>>>>> +#include <dt-bindings/gpio/gpio.h>
>>>>>> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
>>>>>> +#include "talos.dtsi"
>>>>>> +#include "pm8150.dtsi"
>>>>>> +/ {
>>>>>> +    aliases {
>>>>>> +        mmc0 = &sdhc_1;
>>>>>> +        serial0 = &uart0;
>>>>>> +        serial1 = &uart7;
>>>>>> +    };
>>>>>> +
>>>>>> +    chosen {
>>>>>> +        stdout-path = "serial0:115200n8";
>>>>>> +    };
>>>>>> +
>>>>>> +    clocks {
>>>>>> +        can_osc: can-oscillator {
>>>>>> +            compatible = "fixed-clock";
>>>>>> +            clock-frequency = <20000000>;
>>>>>> +            #clock-cells = <0>;
>>>>>> +        };
>>>>>> +
>>>>>> +        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>;
>>>>>> +        };
>>>>>> +    };
>>>>>> +
>>>>>> +    regulator-usb2-vbus {
>>>>>> +        compatible = "regulator-fixed";
>>>>>> +        regulator-name = "USB2_VBUS";
>>>>>> +        gpio = <&pm8150_gpios 10 GPIO_ACTIVE_HIGH>;
>>>>>> +        pinctrl-0 = <&usb2_en>;
>>>>>> +        pinctrl-names = "default";
>>>>>> +        enable-active-high;
>>>>>> +        regulator-always-on;
>>>>>> +    };
>>>>>> +
>>>>>> +    vreg_conn_1p8: regulator-conn-1p8 {
>>>>>> +        compatible = "regulator-fixed";
>>>>>> +        regulator-name = "vreg_conn_1p8";
>>>>>> +        startup-delay-us = <4000>;
>>>>>> +        enable-active-high;
>>>>>> +        gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
>>>>>> +    };
>>>>>> +
>>>>>> +    vreg_conn_pa: regulator-conn-pa {
>>>>>> +        compatible = "regulator-fixed";
>>>>>> +        regulator-name = "vreg_conn_pa";
>>>>>> +        startup-delay-us = <4000>;
>>>>>> +        enable-active-high;
>>>>>> +        gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
>>>>>> +    };
>>>>>> +
>>>>>> +    vreg_v3p3_can: regulator-v3p3-can {
>>>>>> +        compatible = "regulator-fixed";
>>>>>> +        regulator-name = "vreg-v3p3-can";
>>>>>> +        regulator-min-microvolt = <3300000>;
>>>>>> +        regulator-max-microvolt = <3300000>;
>>>>>> +        regulator-boot-on;
>>>>>> +        regulator-always-on;
>>>>>> +    };
>>>>>> +
>>>>>> +    vreg_v5p0_can: regulator-v5p0-can {
>>>>>> +        compatible = "regulator-fixed";
>>>>>> +        regulator-name = "vreg-v5p0-can";
>>>>>> +        regulator-min-microvolt = <5000000>;
>>>>>> +        regulator-max-microvolt = <5000000>;
>>>>>> +        regulator-boot-on;
>>>>>> +        regulator-always-on;
>>>>>> +    };
>>>>>> +
>>>>>> +    wcn6855-pmu {
>>>>>> +        compatible = "qcom,wcn6855-pmu";
>>>>>> +
>>>>>> +        pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
>>>>>> +        pinctrl-names = "default";
>>>>>> +
>>>>>> +        bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
>>>>>> +        wlan-enable-gpios = <&tlmm 84 GPIO_ACTIVE_HIGH>;
>>>>>> +
>>>>>> +        vddio-supply = <&vreg_conn_pa>;
>>>>>> +        vddaon-supply = <&vreg_s5a>;
>>>>>> +        vddpmu-supply = <&vreg_conn_1p8>;
>>>>>> +        vddpmumx-supply = <&vreg_conn_1p8>;
>>>>>> +        vddpmucx-supply = <&vreg_conn_pa>;
>>>>>> +        vddrfa0p95-supply = <&vreg_s5a>;
>>>>>> +        vddrfa1p3-supply = <&vreg_s6a>;
>>>>>> +        vddrfa1p9-supply = <&vreg_l15a>;
>>>>>> +        vddpcie1p3-supply = <&vreg_s6a>;
>>>>>> +        vddpcie1p9-supply = <&vreg_l15a>;
>>>>>> +
>>>>>> +        regulators {
>>>>>> +            vreg_pmu_rfa_cmn: ldo0 {
>>>>>> +                regulator-name = "vreg_pmu_rfa_cmn";
>>>>>> +            };
>>>>>> +
>>>>>> +            vreg_pmu_aon_0p59: ldo1 {
>>>>>> +                regulator-name = "vreg_pmu_aon_0p59";
>>>>>> +            };
>>>>>> +
>>>>>> +            vreg_pmu_wlcx_0p8: ldo2 {
>>>>>> +                regulator-name = "vreg_pmu_wlcx_0p8";
>>>>>> +            };
>>>>>> +
>>>>>> +            vreg_pmu_wlmx_0p85: ldo3 {
>>>>>> +                regulator-name = "vreg_pmu_wlmx_0p85";
>>>>>> +            };
>>>>>> +
>>>>>> +            vreg_pmu_btcmx_0p85: ldo4 {
>>>>>> +                regulator-name = "vreg_pmu_btcmx_0p85";
>>>>>> +            };
>>>>>> +
>>>>>> +            vreg_pmu_rfa_0p8: ldo5 {
>>>>>> +                regulator-name = "vreg_pmu_rfa_0p8";
>>>>>> +            };
>>>>>> +
>>>>>> +            vreg_pmu_rfa_1p2: ldo6 {
>>>>>> +                regulator-name = "vreg_pmu_rfa_1p2";
>>>>>> +            };
>>>>>> +
>>>>>> +            vreg_pmu_rfa_1p7: ldo7 {
>>>>>> +                regulator-name = "vreg_pmu_rfa_1p7";
>>>>>> +            };
>>>>>> +
>>>>>> +            vreg_pmu_pcie_0p9: ldo8 {
>>>>>> +                regulator-name = "vreg_pmu_pcie_0p9";
>>>>>> +            };
>>>>>> +
>>>>>> +            vreg_pmu_pcie_1p8: ldo9 {
>>>>>> +                regulator-name = "vreg_pmu_pcie_1p8";
>>>>>> +            };
>>>>>> +        };
>>>>>> +    };
>>>>>> +
>>>>>> +    wifi_1p8v: regulator-wifi-1p8v {
>>>>>> +        compatible = "regulator-fixed";
>>>>>> +        regulator-name = "wifi_1p8v";
>>>>>> +        regulator-min-microvolt = <1800000>;
>>>>>> +        regulator-max-microvolt = <1800000>;
>>>>>> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
>>>>> Please check this pin number
>>>>>> +        enable-active-high;
>>>>>> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
>>>>>> +        pinctrl-names = "default";
>>>>>> +        regulator-boot-on;
>>>>>> +        regulator-always-on;
>>>>>> +    };
>>>>>> +
>>>>>> +    wifi_3p85v: regulator-wifi-3p85v {
>>>>>> +        compatible = "regulator-fixed";
>>>>>> +        regulator-name = "wifi_3p85v";
>>>>>> +        regulator-min-microvolt = <3850000>;
>>>>>> +        regulator-max-microvolt = <3850000>;
>>>>>> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
>>>>> Please check this pin number
>>>>>> +        enable-active-high;
>>>>>> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
>>>>>> +        pinctrl-names = "default";
>>>>>> +        regulator-boot-on;
>>>>>> +        regulator-always-on;
>>>>>> +    };
>>>>>> +};
>>>>>
>>>>> Are these two node necessary?
>>>>>
>>>>
>>>> On this board, GPIO91 is wired as a common enable for both WiFi 
>>>> power rails: WiFi 1.8V and WiFi 3.85V.
>>>> I currently modeled them as two regulator-fixed nodes because these 
>>>> are two distinct rails.
>>>> Would you prefer modelling a single regulator node that controls the 
>>>> shared GPIO as below:
>>>>
>>>> wifi_en: regulator-wifi-en {
>>>>     compatible = "regulator-fixed";
>>>>     regulator-name = "wifi_en";
>>>>     gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
>>>>     enable-active-high;
>>>>     pinctrl-0 = <&wifi_reg_en_pins_state>;
>>>>     pinctrl-names = "default";
>>>>     regulator-boot-on;
>>>>     regulator-always-on;
>>>> };
>>>
>>> What is the voltage of this regulator? What does it represent? What
>>> should be represented in the DT?
>>>
>>> BTW: what is powered on by those regulators? I don't see them being
>>> wired to the PMU.
>>>
>>
>> There are two separate TPS62130RGTR regulator ICs on the board:
>> One provides 1.8V and Another provides 3.85V.
>>
>> Both regulators use the same enable signal (SOC_GPIO_91_S4A_1P8 - 
>> GPIO91), which connects to their EN pins. 
>> When GPIO91 goes high, both regulators switch on at the same 
>> time.
> 
> And gpiolib-shared.c should be able to support that.
> 
>>
>> I described them as two separate regulator-fixed nodes since they 
>> are different chips with different voltages.
> 
> okay.
> 
>>
>> As an alternative, I suggested representing them as a single node 
>> (based on the review comment from Qian Zhang) if you only want to 
>> show the shared GPIO enable control instead of the individual 
>> voltage rails.
> 
> Why do you need an alternative?
> 
>>
>> In AF68E module, 3.85V regulator powers VDD_PA_5G and 1.8V regulator
>> powers VDD_PA_2G.
> 
> Do they power up the WCN6855 itself or some external PA present on the
> module? In the former case, those regulators should be consumed by the
> PMU, in the latter case, by the WiFi node. Anyway, they should not be
> left as always-on / boot-on.
> 

On this board, the GPIO-controlled regulators power the WCN6855 
WiFi module itself (not an external PA). To model this correctly, 
I updated the device tree like this:
 
a) I added a single upstream fixed regulator, wifi_vin, which 
   represents the GPIO enable line (tlmm 91). This is the only node 
   marked always-on, so the enable GPIO stays asserted.
b) The downstream rails wifi_1p8v and wifi_3p85v are now defined as 
   fixed regulators that depend on wifi_vin using vin-supply. They 
   no longer control the GPIO directly.
c) These two rails are then explicitly connected as supplies to the 
   WiFi device node (wifi@0) using:
     1. vddpaa-supply = <&wifi_3p85v>;
     2. vddpab-supply = <&wifi_1p8v>;

With this change, the regulators are not left as generic always-on/
boot-on supplies without a consumer. Instead, the WiFi device node 
consumes the rails, which matches the real hardware wiring: the GPIO 
enables power to the WCN6855 rails.

Please find the updated changes below - could you please confirm 
whether this aligns with your expectations?

diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
index 95ed335bcb08..f37159af2375 100644
--- a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
+++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
@@ -146,29 +146,41 @@ vreg_pmu_pcie_1p8: ldo9 {
                };
        };

-       wifi_1p8v: regulator-wifi-1p8v {
+       /* Downstream rails controlled by the same GPIO */
+       wifi_1p8v: regulator-1p8v {
                compatible = "regulator-fixed";
+
                regulator-name = "wifi_1p8v";
                regulator-min-microvolt = <1800000>;
                regulator-max-microvolt = <1800000>;
-               gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
-               enable-active-high;
-               pinctrl-0 = <&wifi_reg_en_pins_state>;
-               pinctrl-names = "default";
-               regulator-boot-on;
-               regulator-always-on;
+
+               vin-supply = <&wifi_vin>;
        };

-       wifi_3p85v: regulator-wifi-3p85v {
+       wifi_3p85v: regulator-3p85v {
                compatible = "regulator-fixed";
+
                regulator-name = "wifi_3p85v";
                regulator-min-microvolt = <3850000>;
                regulator-max-microvolt = <3850000>;
-               gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
+
+               vin-supply = <&wifi_vin>;
+       };
+
+       /* Upstream supply */
+       wifi_vin: regulator-wifi-en {
+               compatible = "regulator-fixed";
+
+               regulator-name = "wifi_vin";
+               regulator-min-microvolt = <3850000>;
+               regulator-max-microvolt = <3850000>;
+
                enable-active-high;
+               gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
+
                pinctrl-0 = <&wifi_reg_en_pins_state>;
                pinctrl-names = "default";
-               regulator-boot-on;
+
                regulator-always-on;
        };
 };
@@ -413,6 +425,9 @@ wifi@0 {
                vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
                vddpcie0p9-supply = <&vreg_pmu_pcie_0p9>;
                vddpcie1p8-supply = <&vreg_pmu_pcie_1p8>;
+               vddpaa-supply = <&wifi_3p85v>;
+               vddpab-supply = <&wifi_1p8v>;
+
        };
 };

Re: [PATCH v11 4/4] arm64: dts: qcom: talos-evk: Add support for QCS615 talos evk board

[Dmitry Baryshkov]

On Tue, Jan 20, 2026 at 04:33:28PM +0530, tessolveupstream@gmail.com wrote:
> 
> 
> On 16-01-2026 17:19, Dmitry Baryshkov wrote:
> > On Fri, Jan 16, 2026 at 04:57:42PM +0530, tessolveupstream@gmail.com wrote:
> >>
> >>
> >> On 16-01-2026 13:26, Dmitry Baryshkov wrote:
> >>> On Fri, Jan 16, 2026 at 12:28:22PM +0530, tessolveupstream@gmail.com wrote:
> >>>>
> >>>>
> >>>> On 15-01-2026 18:47, Qian Zhang wrote:
> >>>>>
> >>>>>
> >>>>> On 1/9/2026 5:52 PM, Sudarshan Shetty wrote:
> >>>>>> Add the device tree for the QCS615-based Talos EVK platform. The
> >>>>>> platform is composed of a System-on-Module following the SMARC
> >>>>>> standard, and a Carrier Board.
> >>>>>>
> >>>>>> The Carrier Board supports several display configurations, HDMI and
> >>>>>> LVDS. Both configurations use the same base hardware, with the display
> >>>>>> selection controlled by a DIP switch.
> >>>>>>
> >>>>>> Use a DTBO file, talos-evk-lvds-auo,g133han01.dtso, which defines an
> >>>>>> overlay that disables HDMI and adds LVDS. The DTs file talos-evk
> >>>>>> can describe the HDMI display configurations.
> >>>>>>
> >>>>>> The initial device tree includes support for:
> >>>>>> - CPU and memory
> >>>>>> - UART
> >>>>>> - GPIOs
> >>>>>> - Regulators
> >>>>>> - PMIC
> >>>>>> - Early console
> >>>>>> - AT24MAC602 EEPROM
> >>>>>> - MCP2515 SPI to CAN
> >>>>>> - ADV7535 DSI-to-HDMI bridge
> >>>>>> - DisplayPort interface
> >>>>>> - SN65DSI84ZXHR DSI-to-LVDS bridge
> >>>>>> - Wi-Fi/BT
> >>>>>>
> >>>>>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> >>>>>> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
> >>>>>> ---
> >>>>>>   arch/arm64/boot/dts/qcom/Makefile             |   4 +
> >>>>>>   .../qcom/talos-evk-lvds-auo,g133han01.dtso    | 126 ++++
> >>>>>>   arch/arm64/boot/dts/qcom/talos-evk-som.dtsi   | 616 ++++++++++++++++++
> >>>>>>   arch/arm64/boot/dts/qcom/talos-evk.dts        | 139 ++++
> >>>>>>   4 files changed, 885 insertions(+)
> >>>>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> >>>>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> >>>>>>   create mode 100644 arch/arm64/boot/dts/qcom/talos-evk.dts
> >>>>>>
> >>>>>> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> >>>>>> index 76cf0115a00a..289b651ef0c5 100644
> >>>>>> --- a/arch/arm64/boot/dts/qcom/Makefile
> >>>>>> +++ b/arch/arm64/boot/dts/qcom/Makefile
> >>>>>> @@ -324,6 +324,10 @@ dtb-$(CONFIG_ARCH_QCOM)    += sm8650-mtp.dtb
> >>>>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8650-qrd.dtb
> >>>>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-mtp.dtb
> >>>>>>   dtb-$(CONFIG_ARCH_QCOM)    += sm8750-qrd.dtb
> >>>>>> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk.dtb
> >>>>>> +talos-evk-lvds-auo,g133han01-dtbs    := \
> >>>>>> +    talos-evk.dtb talos-evk-lvds-auo,g133han01.dtbo
> >>>>>> +dtb-$(CONFIG_ARCH_QCOM)    += talos-evk-lvds-auo,g133han01.dtb
> >>>>>>   x1e001de-devkit-el2-dtbs    := x1e001de-devkit.dtb x1-el2.dtbo
> >>>>>>   dtb-$(CONFIG_ARCH_QCOM)    += x1e001de-devkit.dtb x1e001de-devkit-el2.dtb
> >>>>>>   x1e78100-lenovo-thinkpad-t14s-el2-dtbs    := x1e78100-lenovo-thinkpad-t14s.dtb x1-el2.dtbo
> >>>>>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> >>>>>> new file mode 100644
> >>>>>> index 000000000000..ad058cf4cd93
> >>>>>> --- /dev/null
> >>>>>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-lvds-auo,g133han01.dtso
> >>>>>> @@ -0,0 +1,126 @@
> >>>>>> +// SPDX-License-Identifier: BSD-3-Clause
> >>>>>> +/*
> >>>>>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> >>>>>> + */
> >>>>>> +/dts-v1/;
> >>>>>> +/plugin/;
> >>>>>> +
> >>>>>> +#include <dt-bindings/gpio/gpio.h>
> >>>>>> +
> >>>>>> +&{/} {
> >>>>>> +    backlight: backlight {
> >>>>>> +        compatible = "gpio-backlight";
> >>>>>> +        gpios = <&tlmm 59 GPIO_ACTIVE_HIGH>,
> >>>>>> +            <&tlmm 115 GPIO_ACTIVE_HIGH>;
> >>>>>> +        default-on;
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    panel-lvds {
> >>>>>> +        compatible = "auo,g133han01";
> >>>>>> +
> >>>>>> +        ports {
> >>>>>> +            #address-cells = <1>;
> >>>>>> +            #size-cells = <0>;
> >>>>>> +
> >>>>>> +            /* LVDS A (Odd pixels) */
> >>>>>> +            port@0 {
> >>>>>> +                reg = <0>;
> >>>>>> +                dual-lvds-odd-pixels;
> >>>>>> +
> >>>>>> +                lvds_panel_out_a: endpoint {
> >>>>>> +                    remote-endpoint = <&sn65dsi84_out_a>;
> >>>>>> +                };
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            /* LVDS B (Even pixels) */
> >>>>>> +            port@1 {
> >>>>>> +                reg = <1>;
> >>>>>> +                dual-lvds-even-pixels;
> >>>>>> +
> >>>>>> +                lvds_panel_out_b: endpoint {
> >>>>>> +                    remote-endpoint = <&sn65dsi84_out_b>;
> >>>>>> +                };
> >>>>>> +            };
> >>>>>> +        };
> >>>>>> +    };
> >>>>>> +};
> >>>>>> +
> >>>>>> +&hdmi_connector {
> >>>>>> +    status = "disabled";
> >>>>>> +};
> >>>>>> +
> >>>>>> +&i2c1 {
> >>>>>> +    clock-frequency = <400000>;
> >>>>>> +
> >>>>>> +    status = "okay";
> >>>>>> +
> >>>>>> +    hdmi_bridge: bridge@3d {
> >>>>>> +        status = "disabled";
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    lvds_bridge: bridge@2c {
> >>>>>> +        compatible = "ti,sn65dsi84";
> >>>>>> +        reg = <0x2c>;
> >>>>>> +        enable-gpios = <&tlmm 42 GPIO_ACTIVE_HIGH>;
> >>>>>> +        ti,dsi-lanes = <4>;
> >>>>>> +        ti,lvds-format = "jeida-24";
> >>>>>> +        ti,lvds-bpp = <24>;
> >>>>>> +
> >>>>>> +        ports {
> >>>>>> +            #address-cells = <1>;
> >>>>>> +            #size-cells = <0>;
> >>>>>> +
> >>>>>> +            port@0 {
> >>>>>> +                reg = <0>;
> >>>>>> +
> >>>>>> +                sn65dsi84_in: endpoint {
> >>>>>> +                    data-lanes = <1 2 3 4>;
> >>>>>> +                    remote-endpoint = <&mdss_dsi0_out>;
> >>>>>> +                };
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            port@2 {
> >>>>>> +                reg = <2>;
> >>>>>> +
> >>>>>> +                sn65dsi84_out_a: endpoint {
> >>>>>> +                    data-lanes = <1 2 3 4>;
> >>>>>> +                    remote-endpoint = <&lvds_panel_out_a>;
> >>>>>> +                };
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            port@3 {
> >>>>>> +                reg = <3>;
> >>>>>> +
> >>>>>> +                sn65dsi84_out_b: endpoint {
> >>>>>> +                    data-lanes = <1 2 3 4>;
> >>>>>> +                    remote-endpoint = <&lvds_panel_out_b>;
> >>>>>> +                };
> >>>>>> +            };
> >>>>>> +        };
> >>>>>> +    };
> >>>>>> +};
> >>>>>> +
> >>>>>> +&mdss_dsi0 {
> >>>>>> +    vdda-supply = <&vreg_l11a>;
> >>>>>> +
> >>>>>> +    status = "okay";
> >>>>>> +};
> >>>>>> +
> >>>>>> +&mdss_dsi0_out {
> >>>>>> +    remote-endpoint = <&sn65dsi84_in>;
> >>>>>> +    data-lanes = <0 1 2 3>;
> >>>>>> +};
> >>>>>> +
> >>>>>> +&tlmm {
> >>>>>> +    lcd_bklt_en: lcd-bklt-en-state {
> >>>>>> +        pins = "gpio115";
> >>>>>> +        function = "gpio";
> >>>>>> +        bias-disable;
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    lcd_bklt_pwm: lcd-bklt-pwm-state {
> >>>>>> +        pins = "gpio59";
> >>>>>> +        function = "gpio";
> >>>>>> +        bias-disable;
> >>>>>> +    };
> >>>>>> +};
> >>>>>> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> >>>>>> new file mode 100644
> >>>>>> index 000000000000..95ed335bcb08
> >>>>>> --- /dev/null
> >>>>>> +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> >>>>>> @@ -0,0 +1,616 @@
> >>>>>> +// SPDX-License-Identifier: BSD-3-Clause
> >>>>>> +/*
> >>>>>> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> >>>>>> + */
> >>>>>> +/dts-v1/;
> >>>>>> +
> >>>>>> +#include <dt-bindings/gpio/gpio.h>
> >>>>>> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> >>>>>> +#include "talos.dtsi"
> >>>>>> +#include "pm8150.dtsi"
> >>>>>> +/ {
> >>>>>> +    aliases {
> >>>>>> +        mmc0 = &sdhc_1;
> >>>>>> +        serial0 = &uart0;
> >>>>>> +        serial1 = &uart7;
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    chosen {
> >>>>>> +        stdout-path = "serial0:115200n8";
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    clocks {
> >>>>>> +        can_osc: can-oscillator {
> >>>>>> +            compatible = "fixed-clock";
> >>>>>> +            clock-frequency = <20000000>;
> >>>>>> +            #clock-cells = <0>;
> >>>>>> +        };
> >>>>>> +
> >>>>>> +        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>;
> >>>>>> +        };
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    regulator-usb2-vbus {
> >>>>>> +        compatible = "regulator-fixed";
> >>>>>> +        regulator-name = "USB2_VBUS";
> >>>>>> +        gpio = <&pm8150_gpios 10 GPIO_ACTIVE_HIGH>;
> >>>>>> +        pinctrl-0 = <&usb2_en>;
> >>>>>> +        pinctrl-names = "default";
> >>>>>> +        enable-active-high;
> >>>>>> +        regulator-always-on;
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    vreg_conn_1p8: regulator-conn-1p8 {
> >>>>>> +        compatible = "regulator-fixed";
> >>>>>> +        regulator-name = "vreg_conn_1p8";
> >>>>>> +        startup-delay-us = <4000>;
> >>>>>> +        enable-active-high;
> >>>>>> +        gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    vreg_conn_pa: regulator-conn-pa {
> >>>>>> +        compatible = "regulator-fixed";
> >>>>>> +        regulator-name = "vreg_conn_pa";
> >>>>>> +        startup-delay-us = <4000>;
> >>>>>> +        enable-active-high;
> >>>>>> +        gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    vreg_v3p3_can: regulator-v3p3-can {
> >>>>>> +        compatible = "regulator-fixed";
> >>>>>> +        regulator-name = "vreg-v3p3-can";
> >>>>>> +        regulator-min-microvolt = <3300000>;
> >>>>>> +        regulator-max-microvolt = <3300000>;
> >>>>>> +        regulator-boot-on;
> >>>>>> +        regulator-always-on;
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    vreg_v5p0_can: regulator-v5p0-can {
> >>>>>> +        compatible = "regulator-fixed";
> >>>>>> +        regulator-name = "vreg-v5p0-can";
> >>>>>> +        regulator-min-microvolt = <5000000>;
> >>>>>> +        regulator-max-microvolt = <5000000>;
> >>>>>> +        regulator-boot-on;
> >>>>>> +        regulator-always-on;
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    wcn6855-pmu {
> >>>>>> +        compatible = "qcom,wcn6855-pmu";
> >>>>>> +
> >>>>>> +        pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
> >>>>>> +        pinctrl-names = "default";
> >>>>>> +
> >>>>>> +        bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
> >>>>>> +        wlan-enable-gpios = <&tlmm 84 GPIO_ACTIVE_HIGH>;
> >>>>>> +
> >>>>>> +        vddio-supply = <&vreg_conn_pa>;
> >>>>>> +        vddaon-supply = <&vreg_s5a>;
> >>>>>> +        vddpmu-supply = <&vreg_conn_1p8>;
> >>>>>> +        vddpmumx-supply = <&vreg_conn_1p8>;
> >>>>>> +        vddpmucx-supply = <&vreg_conn_pa>;
> >>>>>> +        vddrfa0p95-supply = <&vreg_s5a>;
> >>>>>> +        vddrfa1p3-supply = <&vreg_s6a>;
> >>>>>> +        vddrfa1p9-supply = <&vreg_l15a>;
> >>>>>> +        vddpcie1p3-supply = <&vreg_s6a>;
> >>>>>> +        vddpcie1p9-supply = <&vreg_l15a>;
> >>>>>> +
> >>>>>> +        regulators {
> >>>>>> +            vreg_pmu_rfa_cmn: ldo0 {
> >>>>>> +                regulator-name = "vreg_pmu_rfa_cmn";
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            vreg_pmu_aon_0p59: ldo1 {
> >>>>>> +                regulator-name = "vreg_pmu_aon_0p59";
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            vreg_pmu_wlcx_0p8: ldo2 {
> >>>>>> +                regulator-name = "vreg_pmu_wlcx_0p8";
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            vreg_pmu_wlmx_0p85: ldo3 {
> >>>>>> +                regulator-name = "vreg_pmu_wlmx_0p85";
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            vreg_pmu_btcmx_0p85: ldo4 {
> >>>>>> +                regulator-name = "vreg_pmu_btcmx_0p85";
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            vreg_pmu_rfa_0p8: ldo5 {
> >>>>>> +                regulator-name = "vreg_pmu_rfa_0p8";
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            vreg_pmu_rfa_1p2: ldo6 {
> >>>>>> +                regulator-name = "vreg_pmu_rfa_1p2";
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            vreg_pmu_rfa_1p7: ldo7 {
> >>>>>> +                regulator-name = "vreg_pmu_rfa_1p7";
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            vreg_pmu_pcie_0p9: ldo8 {
> >>>>>> +                regulator-name = "vreg_pmu_pcie_0p9";
> >>>>>> +            };
> >>>>>> +
> >>>>>> +            vreg_pmu_pcie_1p8: ldo9 {
> >>>>>> +                regulator-name = "vreg_pmu_pcie_1p8";
> >>>>>> +            };
> >>>>>> +        };
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    wifi_1p8v: regulator-wifi-1p8v {
> >>>>>> +        compatible = "regulator-fixed";
> >>>>>> +        regulator-name = "wifi_1p8v";
> >>>>>> +        regulator-min-microvolt = <1800000>;
> >>>>>> +        regulator-max-microvolt = <1800000>;
> >>>>>> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> >>>>> Please check this pin number
> >>>>>> +        enable-active-high;
> >>>>>> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
> >>>>>> +        pinctrl-names = "default";
> >>>>>> +        regulator-boot-on;
> >>>>>> +        regulator-always-on;
> >>>>>> +    };
> >>>>>> +
> >>>>>> +    wifi_3p85v: regulator-wifi-3p85v {
> >>>>>> +        compatible = "regulator-fixed";
> >>>>>> +        regulator-name = "wifi_3p85v";
> >>>>>> +        regulator-min-microvolt = <3850000>;
> >>>>>> +        regulator-max-microvolt = <3850000>;
> >>>>>> +        gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> >>>>> Please check this pin number
> >>>>>> +        enable-active-high;
> >>>>>> +        pinctrl-0 = <&wifi_reg_en_pins_state>;
> >>>>>> +        pinctrl-names = "default";
> >>>>>> +        regulator-boot-on;
> >>>>>> +        regulator-always-on;
> >>>>>> +    };
> >>>>>> +};
> >>>>>
> >>>>> Are these two node necessary?
> >>>>>
> >>>>
> >>>> On this board, GPIO91 is wired as a common enable for both WiFi 
> >>>> power rails: WiFi 1.8V and WiFi 3.85V.
> >>>> I currently modeled them as two regulator-fixed nodes because these 
> >>>> are two distinct rails.
> >>>> Would you prefer modelling a single regulator node that controls the 
> >>>> shared GPIO as below:
> >>>>
> >>>> wifi_en: regulator-wifi-en {
> >>>>     compatible = "regulator-fixed";
> >>>>     regulator-name = "wifi_en";
> >>>>     gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> >>>>     enable-active-high;
> >>>>     pinctrl-0 = <&wifi_reg_en_pins_state>;
> >>>>     pinctrl-names = "default";
> >>>>     regulator-boot-on;
> >>>>     regulator-always-on;
> >>>> };
> >>>
> >>> What is the voltage of this regulator? What does it represent? What
> >>> should be represented in the DT?
> >>>
> >>> BTW: what is powered on by those regulators? I don't see them being
> >>> wired to the PMU.
> >>>
> >>
> >> There are two separate TPS62130RGTR regulator ICs on the board:
> >> One provides 1.8V and Another provides 3.85V.
> >>
> >> Both regulators use the same enable signal (SOC_GPIO_91_S4A_1P8 - 
> >> GPIO91), which connects to their EN pins. 
> >> When GPIO91 goes high, both regulators switch on at the same 
> >> time.
> > 
> > And gpiolib-shared.c should be able to support that.
> > 
> >>
> >> I described them as two separate regulator-fixed nodes since they 
> >> are different chips with different voltages.
> > 
> > okay.
> > 
> >>
> >> As an alternative, I suggested representing them as a single node 
> >> (based on the review comment from Qian Zhang) if you only want to 
> >> show the shared GPIO enable control instead of the individual 
> >> voltage rails.
> > 
> > Why do you need an alternative?
> > 
> >>
> >> In AF68E module, 3.85V regulator powers VDD_PA_5G and 1.8V regulator
> >> powers VDD_PA_2G.
> > 
> > Do they power up the WCN6855 itself or some external PA present on the
> > module? In the former case, those regulators should be consumed by the
> > PMU, in the latter case, by the WiFi node. Anyway, they should not be
> > left as always-on / boot-on.
> > 
> 
> On this board, the GPIO-controlled regulators power the WCN6855 
> WiFi module itself (not an external PA). To model this correctly, 
> I updated the device tree like this:
>  
> a) I added a single upstream fixed regulator, wifi_vin, which 
>    represents the GPIO enable line (tlmm 91). This is the only node 
>    marked always-on, so the enable GPIO stays asserted.

Why? Is there such a regulator on the device? Why is it always-on? What
if the user wants to turn WiFi off? If you have consumers, why do you
need always-on at all?

> b) The downstream rails wifi_1p8v and wifi_3p85v are now defined as 
>    fixed regulators that depend on wifi_vin using vin-supply. They 
>    no longer control the GPIO directly.

Why? Does the shared GPIO mechanism fail here? If so, how?

> c) These two rails are then explicitly connected as supplies to the 
>    WiFi device node (wifi@0) using:
>      1. vddpaa-supply = <&wifi_3p85v>;
>      2. vddpab-supply = <&wifi_1p8v>;

Are those documented? Which pin is this regulator connected to?

> 
> With this change, the regulators are not left as generic always-on/
> boot-on supplies without a consumer. Instead, the WiFi device node 
> consumes the rails, which matches the real hardware wiring: the GPIO 
> enables power to the WCN6855 rails.
> 
> Please find the updated changes below - could you please confirm 
> whether this aligns with your expectations?
> 
> diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> index 95ed335bcb08..f37159af2375 100644
> --- a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi
> @@ -146,29 +146,41 @@ vreg_pmu_pcie_1p8: ldo9 {
>                 };
>         };
> 
> -       wifi_1p8v: regulator-wifi-1p8v {
> +       /* Downstream rails controlled by the same GPIO */
> +       wifi_1p8v: regulator-1p8v {
>                 compatible = "regulator-fixed";
> +
>                 regulator-name = "wifi_1p8v";
>                 regulator-min-microvolt = <1800000>;
>                 regulator-max-microvolt = <1800000>;
> -               gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> -               enable-active-high;
> -               pinctrl-0 = <&wifi_reg_en_pins_state>;
> -               pinctrl-names = "default";
> -               regulator-boot-on;
> -               regulator-always-on;
> +
> +               vin-supply = <&wifi_vin>;
>         };
> 
> -       wifi_3p85v: regulator-wifi-3p85v {
> +       wifi_3p85v: regulator-3p85v {
>                 compatible = "regulator-fixed";
> +
>                 regulator-name = "wifi_3p85v";
>                 regulator-min-microvolt = <3850000>;
>                 regulator-max-microvolt = <3850000>;
> -               gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> +
> +               vin-supply = <&wifi_vin>;
> +       };
> +
> +       /* Upstream supply */
> +       wifi_vin: regulator-wifi-en {
> +               compatible = "regulator-fixed";
> +
> +               regulator-name = "wifi_vin";
> +               regulator-min-microvolt = <3850000>;
> +               regulator-max-microvolt = <3850000>;
> +
>                 enable-active-high;
> +               gpio = <&tlmm 91 GPIO_ACTIVE_HIGH>;
> +
>                 pinctrl-0 = <&wifi_reg_en_pins_state>;
>                 pinctrl-names = "default";
> -               regulator-boot-on;
> +
>                 regulator-always-on;
>         };
>  };
> @@ -413,6 +425,9 @@ wifi@0 {
>                 vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
>                 vddpcie0p9-supply = <&vreg_pmu_pcie_0p9>;
>                 vddpcie1p8-supply = <&vreg_pmu_pcie_1p8>;
> +               vddpaa-supply = <&wifi_3p85v>;
> +               vddpab-supply = <&wifi_1p8v>;
> +
>         };
>  };
> 
> 
> 
> 

-- 
With best wishes
Dmitry

Re: [PATCH v11 0/4] Add Qualcomm Technologies, Inc. Talos EVK SMARC support

[Rob Herring]

On Fri, Jan 09, 2026 at 03:22:16PM +0530, Sudarshan Shetty wrote:
> Hi all,

From the mail headers:
> Content-Type: text/plain; charset=y                                                                        
> Content-Transfer-Encoding: 8bit                                                                            

When git-send-email prompts you for which encoding to use, the correct 
answer is not 'y'. The default utf-8 is correct, so just hit enter. 
IIRC, newer git versions fixed this.

Rob

Re: [PATCH v11 0/4] Add Qualcomm Technologies, Inc. Talos EVK SMARC support

[tessolveupstream]

On 12-01-2026 21:01, Rob Herring wrote:
> On Fri, Jan 09, 2026 at 03:22:16PM +0530, Sudarshan Shetty wrote:
>> Hi all,
> 
> From the mail headers:
>> Content-Type: text/plain; charset=y                                                                        
>> Content-Transfer-Encoding: 8bit                                                                            
> 
> When git-send-email prompts you for which encoding to use, the correct 
> answer is not 'y'. The default utf-8 is correct, so just hit enter. 
> IIRC, newer git versions fixed this.
>

Okay, got it.
 
> Rob