From: Bjorn Andersson <andersson@kernel.org>
To: Anand Tiwari <anand.tiwari@oss.qualcomm.com>
Cc: Konrad Dybcio <konradybcio@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, venkata.valluru@oss.qualcomm.com,
vishnu.saini@oss.qualcomm.com,
Jessica Zhang <jesszhan0024@gmail.com>
Subject: Re: [PATCH 2/2] arm64: dts: qcom: purwa-iot-evk: Add eDP display overlay
Date: Thu, 6 Aug 2026 19:06:24 -0500 [thread overview]
Message-ID: <anUhYybtYglbk2kD@baldur> (raw)
In-Reply-To: <20260806-edp_overlay_for-next-v1-2-c3aebbc81f38@oss.qualcomm.com>
On Thu, Aug 06, 2026 at 10:01:21PM +0530, Anand Tiwari wrote:
> Move the eDP panel configuration and related power, backlight, and pinctrl
> nodes into a separate overlay. Keep the base DTB suitable for headless
> variants and provide a composite DTB for headed variants.
>
Why?
> Signed-off-by: Anand Tiwari <anand.tiwari@oss.qualcomm.com>
> ---
> arch/arm64/boot/dts/qcom/Makefile | 3 +
> arch/arm64/boot/dts/qcom/purwa-iot-evk-edp.dtso | 126 ++++++++++++++++++++++++
> arch/arm64/boot/dts/qcom/purwa-iot-evk.dts | 107 --------------------
> 3 files changed, 129 insertions(+), 107 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> index bb0da34983c8..fc954da58402 100644
> --- a/arch/arm64/boot/dts/qcom/Makefile
> +++ b/arch/arm64/boot/dts/qcom/Makefile
> @@ -165,6 +165,9 @@ dtb-$(CONFIG_ARCH_QCOM) += msm8998-sony-xperia-yoshino-poplar.dtb
> dtb-$(CONFIG_ARCH_QCOM) += msm8998-xiaomi-sagit.dtb
> dtb-$(CONFIG_ARCH_QCOM) += purwa-iot-evk.dtb
>
> +purwa-iot-evk-edp-dtbs := purwa-iot-evk.dtb purwa-iot-evk-edp.dtbo
> +dtb-$(CONFIG_ARCH_QCOM) += purwa-iot-evk-edp.dtb
> +
> purwa-iot-evk-el2-dtbs := purwa-iot-evk.dtb x1-el2.dtbo
>
> dtb-$(CONFIG_ARCH_QCOM) += purwa-iot-evk-el2.dtb
> diff --git a/arch/arm64/boot/dts/qcom/purwa-iot-evk-edp.dtso b/arch/arm64/boot/dts/qcom/purwa-iot-evk-edp.dtso
> new file mode 100644
> index 000000000000..759d05342627
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/purwa-iot-evk-edp.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 = "pwm-backlight";
> + pwms = <&pmk8550_pwm 0 5000000>;
> + enable-gpios = <&pmc8380_3_gpios 4 GPIO_ACTIVE_HIGH>;
> + power-supply = <&vreg_edp_bl>;
> +
> + pinctrl-0 = <&edp_bl_en>, <&edp_bl_pwm>;
> + pinctrl-names = "default";
> + };
> +
> + vreg_edp_3p3: regulator-edp-3p3 {
> + compatible = "regulator-fixed";
> +
> + regulator-name = "VREG_EDP_3P3";
> + regulator-min-microvolt = <3300000>;
> + regulator-max-microvolt = <3300000>;
> +
> + gpio = <&tlmm 70 GPIO_ACTIVE_HIGH>;
> + enable-active-high;
> +
> + pinctrl-0 = <&edp_reg_en>;
> + pinctrl-names = "default";
> +
> + regulator-boot-on;
> + };
> +
> + vreg_edp_bl: regulator-edp-bl {
> + compatible = "regulator-fixed";
> +
> + regulator-name = "VBL9";
> + regulator-min-microvolt = <3600000>;
> + regulator-max-microvolt = <3600000>;
> +
> + gpio = <&pmc8380_3_gpios 10 GPIO_ACTIVE_HIGH>;
> + enable-active-high;
> +
> + pinctrl-0 = <&edp_bl_reg_en>;
> + pinctrl-names = "default";
> +
> + regulator-boot-on;
> + };
> +};
> +
> +&mdss_dp3 {
> + /delete-property/ #sound-dai-cells;
> +
> + pinctrl-0 = <&edp0_hpd_default>;
> + pinctrl-names = "default";
> +
> + status = "okay";
> +
> + aux-bus {
> + panel {
> + compatible = "edp-panel";
> +
> + backlight = <&backlight>;
> + power-supply = <&vreg_edp_3p3>;
> +
> + port {
> + edp_panel_in: endpoint {
> + remote-endpoint = <&mdss_dp3_out>;
> + };
> + };
> + };
> + };
> +};
> +
> +&mdss_dp3_out {
> + data-lanes = <0 1 2 3>;
> + link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000 8100000000>;
> +
> + remote-endpoint = <&edp_panel_in>;
> +};
> +
> +&mdss_dp3_phy {
> + vdda-phy-supply = <&vreg_l3j_0p8>;
> + vdda-pll-supply = <&vreg_l2j_1p2>;
> +
> + status = "okay";
> +};
> +
> +&pmc8380_3_gpios {
> + edp_bl_en: edp-bl-en-state {
> + pins = "gpio4";
> + function = "normal";
> + power-source = <1>;
> + input-disable;
> + output-enable;
> + };
> +
> + edp_bl_reg_en: edp-bl-reg-en-state {
> + pins = "gpio10";
> + function = "normal";
> + };
> +};
> +
> +&pmk8550_gpios {
> + edp_bl_pwm: edp-bl-pwm-state {
> + pins = "gpio5";
> + function = "func3";
> + };
> +};
> +
> +&pmk8550_pwm {
> + status = "okay";
> +};
> +
> +&tlmm {
> + edp_reg_en: edp-reg-en-state {
> + pins = "gpio70";
> + function = "gpio";
> + drive-strength = <16>;
> + bias-disable;
> + };
> +};
> diff --git a/arch/arm64/boot/dts/qcom/purwa-iot-evk.dts b/arch/arm64/boot/dts/qcom/purwa-iot-evk.dts
> index ad503beec1d3..6dde925025dd 100644
> --- a/arch/arm64/boot/dts/qcom/purwa-iot-evk.dts
> +++ b/arch/arm64/boot/dts/qcom/purwa-iot-evk.dts
> @@ -19,16 +19,6 @@ aliases {
> serial1 = &uart14;
> };
>
> - backlight: backlight {
> - compatible = "pwm-backlight";
> - pwms = <&pmk8550_pwm 0 5000000>;
> - enable-gpios = <&pmc8380_3_gpios 4 GPIO_ACTIVE_HIGH>;
> - power-supply = <&vreg_edp_bl>;
> -
> - pinctrl-0 = <&edp_bl_en>, <&edp_bl_pwm>;
> - pinctrl-names = "default";
> - };
> -
> wcd938x: audio-codec {
> compatible = "qcom,wcd9385-codec";
>
> @@ -205,38 +195,6 @@ pmic_glink_ss2_con_sbu_in: endpoint {
> };
> };
>
> - vreg_edp_3p3: regulator-edp-3p3 {
> - compatible = "regulator-fixed";
> -
> - regulator-name = "VREG_EDP_3P3";
> - regulator-min-microvolt = <3300000>;
> - regulator-max-microvolt = <3300000>;
> -
> - gpio = <&tlmm 70 GPIO_ACTIVE_HIGH>;
> - enable-active-high;
> -
> - pinctrl-0 = <&edp_reg_en>;
> - pinctrl-names = "default";
> -
> - regulator-boot-on;
> - };
> -
> - vreg_edp_bl: regulator-edp-bl {
> - compatible = "regulator-fixed";
> -
> - regulator-name = "VBL9";
> - regulator-min-microvolt = <3600000>;
> - regulator-max-microvolt = <3600000>;
> -
> - gpio = <&pmc8380_3_gpios 10 GPIO_ACTIVE_HIGH>;
> - enable-active-high;
> -
> - pinctrl-0 = <&edp_bl_reg_en>;
> - pinctrl-names = "default";
> -
> - regulator-boot-on;
> - };
> -
> vreg_nvme: regulator-nvme {
> compatible = "regulator-fixed";
>
> @@ -962,44 +920,6 @@ &mdss_dp2_out {
> link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000 8100000000>;
> };
>
> -&mdss_dp3 {
> - /delete-property/ #sound-dai-cells;
> -
> - pinctrl-0 = <&edp0_hpd_default>;
> - pinctrl-names = "default";
> -
> - status = "okay";
> -
> - aux-bus {
> - panel {
> - compatible = "edp-panel";
> -
> - backlight = <&backlight>;
> - power-supply = <&vreg_edp_3p3>;
> -
> - port {
> - edp_panel_in: endpoint {
> - remote-endpoint = <&mdss_dp3_out>;
> - };
> - };
> - };
> - };
> -};
> -
> -&mdss_dp3_out {
> - data-lanes = <0 1 2 3>;
> - link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000 8100000000>;
> -
> - remote-endpoint = <&edp_panel_in>;
> -};
> -
> -&mdss_dp3_phy {
> - vdda-phy-supply = <&vreg_l3j_0p8>;
> - vdda-pll-supply = <&vreg_l2j_1p2>;
> -
> - status = "okay";
> -};
> -
> &pcie3_port0 {
> vpcie12v-supply = <&vreg_pcie_12v>;
> vpcie3v3-supply = <&vreg_pcie_3v3>;
> @@ -1128,19 +1048,6 @@ led@2 {
> };
>
> &pmc8380_3_gpios {
> - edp_bl_en: edp-bl-en-state {
> - pins = "gpio4";
> - function = "normal";
> - power-source = <1>;
> - input-disable;
> - output-enable;
> - };
> -
> - edp_bl_reg_en: edp-bl-reg-en-state {
> - pins = "gpio10";
> - function = "normal";
> - };
> -
> pm_sde7_aux_3p3_en: pcie-aux-3p3-default-state {
> pins = "gpio8";
> function = "normal";
> @@ -1169,13 +1076,6 @@ usb0_pwr_1p15_reg_en: usb0-pwr-1p15-reg-en-state {
> };
> };
>
> -&pmk8550_gpios {
> - edp_bl_pwm: edp-bl-pwm-state {
> - pins = "gpio5";
> - function = "func3";
> - };
> -};
> -
> &pmk8550_pwm {
> status = "okay";
> };
> @@ -1326,13 +1226,6 @@ right_tweeter: speaker@0,1 {
> };
>
> &tlmm {
> - edp_reg_en: edp-reg-en-state {
> - pins = "gpio70";
> - function = "gpio";
> - drive-strength = <16>;
> - bias-disable;
> - };
> -
> eusb3_reset_n: eusb3-reset-n-state {
> pins = "gpio6";
> function = "gpio";
>
> --
> 2.54.0
>
next prev parent reply other threads:[~2026-08-07 0:06 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-06 16:31 [PATCH 0/2] arm64: dts: qcom: Add eDP display overlays for hamoa and purwa IoT EVKs Anand Tiwari
2026-08-06 16:31 ` [PATCH 1/2] arm64: dts: qcom: hamoa-iot-evk: Add eDP display overlay Anand Tiwari
2026-08-06 16:55 ` sashiko-bot
2026-08-07 0:05 ` Bjorn Andersson
2026-08-07 10:00 ` Anand Tiwari
2026-08-07 21:01 ` Bjorn Andersson
2026-08-10 8:22 ` Anand Tiwari
2026-08-10 13:11 ` Bjorn Andersson
2026-08-10 9:04 ` Dmitry Baryshkov
2026-08-06 16:31 ` [PATCH 2/2] arm64: dts: qcom: purwa-iot-evk: " Anand Tiwari
2026-08-06 16:54 ` sashiko-bot
2026-08-07 0:06 ` Bjorn Andersson [this message]
2026-08-07 9:22 ` [PATCH 0/2] arm64: dts: qcom: Add eDP display overlays for hamoa and purwa IoT EVKs Dmitry Baryshkov
2026-08-07 10:36 ` Anand Tiwari
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=anUhYybtYglbk2kD@baldur \
--to=andersson@kernel.org \
--cc=anand.tiwari@oss.qualcomm.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jesszhan0024@gmail.com \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
--cc=venkata.valluru@oss.qualcomm.com \
--cc=vishnu.saini@oss.qualcomm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.