From: Bjorn Andersson <andersson@kernel.org>
To: Abel Vesa <abel.vesa@oss.qualcomm.com>
Cc: Konrad Dybcio <konradybcio@kernel.org>,
Abel Vesa <abelvesa@kernel.org>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 4/5] arm64: dts: qcom: eliza: Add CQS SoM UFS variant
Date: Sun, 27 Sep 2026 21:15:50 -0500 [thread overview]
Message-ID: <arnM9pny5hIM0FaD@zenbook> (raw)
In-Reply-To: <20260915-dts-qcom-eliza-add-ufs-variant-v2-4-86d72a2a8f47@oss.qualcomm.com>
On Tue, Sep 15, 2026 at 03:03:30PM +0300, Abel Vesa wrote:
> Add a new dtsi for the Eliza CQS SoM UFS variant.
> This SoM only comes with UFS installed.
I am not familiar with the Eliza lineup and unfortunately these commit
messages doesn't provide any real value. Here you're saying that you're
adding support for the UFS-variant of the SOM, in the previous commit
message you said "UFS EVK" - which would imply it's the carrier board
that is different.
Please ensure that your commit messages clearly describe what's going
on - and as the documentation states; start with the problem
description, not the word "Add".
Regards,
Bjorn
>
> Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> ---
> arch/arm64/boot/dts/qcom/eliza-cqs-som-ufs.dtsi | 421 ++++++++++++++++++++++++
> 1 file changed, 421 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/eliza-cqs-som-ufs.dtsi b/arch/arm64/boot/dts/qcom/eliza-cqs-som-ufs.dtsi
> new file mode 100644
> index 000000000000..cf1cd97065b4
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/eliza-cqs-som-ufs.dtsi
> @@ -0,0 +1,421 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> + */
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/pinctrl/qcom,pmic-gpio.h>
> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> +
> +#include "eliza.dtsi"
> +#include "pm7550ba-eliza.dtsi"
> +#include "pm7550-eliza.dtsi"
> +#include "pm8550vs-d-eliza.dtsi"
> +#include "pm8550vs-g-eliza.dtsi"
> +#include "pmr735d-eliza.dtsi"
> +#include "pmk8550-eliza.dtsi"
> +
> +/ {
> + clocks {
> + xo_board: xo-board {
> + compatible = "fixed-clock";
> + clock-frequency = <76800000>;
> + #clock-cells = <0>;
> + };
> +
> + sleep_clk: sleep-clk {
> + compatible = "fixed-clock";
> + clock-frequency = <32764>;
> + #clock-cells = <0>;
> + };
> +
> + bi_tcxo_div2: bi-tcxo-div2-clk {
> + compatible = "fixed-factor-clock";
> + #clock-cells = <0>;
> +
> + clocks = <&rpmhcc RPMH_CXO_CLK>;
> + clock-mult = <1>;
> + clock-div = <2>;
> + };
> +
> + bi_tcxo_ao_div2: bi-tcxo-ao-div2-clk {
> + compatible = "fixed-factor-clock";
> + #clock-cells = <0>;
> +
> + clocks = <&rpmhcc RPMH_CXO_CLK_A>;
> + clock-mult = <1>;
> + clock-div = <2>;
> + };
> + };
> +
> + vph_pwr: regulator-vph-pwr {
> + compatible = "regulator-fixed";
> +
> + regulator-name = "vph_pwr";
> + regulator-min-microvolt = <3700000>;
> + regulator-max-microvolt = <3700000>;
> +
> + regulator-always-on;
> + regulator-boot-on;
> + };
> +};
> +
> +&apps_rsc {
> + regulators-0 {
> + compatible = "qcom,pm7550-rpmh-regulators";
> +
> + vdd-l1-supply = <&vreg_s3b>;
> + vdd-l2-l3-supply = <&vreg_s3b>;
> + vdd-l4-l5-supply = <&vreg_s2b>;
> + vdd-l6-supply = <&vreg_s2b>;
> + vdd-l7-supply = <&vreg_s1b>;
> + vdd-l8-supply = <&vreg_s1b>;
> + vdd-l9-l10-supply = <&vreg_s1b>;
> + vdd-l11-supply = <&vreg_s1b>;
> + vdd-l12-l14-supply = <&vreg_bob>;
> + vdd-l13-l16-supply = <&vreg_bob>;
> + vdd-l15-l17-l18-l19-l20-l21-l22-l23-supply = <&vreg_bob>;
> + vdd-s1-supply = <&vph_pwr>;
> + vdd-s2-supply = <&vph_pwr>;
> + vdd-s3-supply = <&vph_pwr>;
> + vdd-s4-supply = <&vph_pwr>;
> + vdd-s5-supply = <&vph_pwr>;
> + vdd-s6-supply = <&vph_pwr>;
> +
> + vdd-bob-supply = <&vph_pwr>;
> +
> + qcom,pmic-id = "b";
> +
> + vreg_s1b: smps1 {
> + regulator-name = "vreg_s1b";
> + regulator-min-microvolt = <1850000>;
> + regulator-max-microvolt = <2040000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_s2b: smps2 {
> + regulator-name = "vreg_s2b";
> + regulator-min-microvolt = <375000>;
> + regulator-max-microvolt = <2744000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_s3b: smps3 {
> + regulator-name = "vreg_s3b";
> + regulator-min-microvolt = <375000>;
> + regulator-max-microvolt = <2744000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_s4b: smps4 {
> + regulator-name = "vreg_s4b";
> + regulator-min-microvolt = <2156000>;
> + regulator-max-microvolt = <2400000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l2b: ldo2 {
> + regulator-name = "vreg_l2b";
> + regulator-min-microvolt = <720000>;
> + regulator-max-microvolt = <950000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l3b: ldo3 {
> + regulator-name = "vreg_l3b";
> + regulator-min-microvolt = <880000>;
> + regulator-max-microvolt = <912000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l4b: ldo4 {
> + regulator-name = "vreg_l4b";
> + regulator-min-microvolt = <1200000>;
> + regulator-max-microvolt = <1200000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l6b: ldo6 {
> + regulator-name = "vreg_l6b";
> + regulator-min-microvolt = <866000>;
> + regulator-max-microvolt = <958000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l7b: ldo7 {
> + regulator-name = "vreg_l7b";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l8b: ldo8 {
> + regulator-name = "vreg_l8b";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l9b: ldo9 {
> + regulator-name = "vreg_l9b";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l10b: ldo10 {
> + regulator-name = "vreg_l10b";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l11b: ldo11 {
> + regulator-name = "vreg_l11b";
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l12b: ldo12 {
> + regulator-name = "vreg_l12b";
> + regulator-min-microvolt = <2400000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l13b: ldo13 {
> + regulator-name = "vreg_l13b";
> + regulator-min-microvolt = <2700000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l14b: ldo14 {
> + regulator-name = "vreg_l14b";
> + regulator-min-microvolt = <3300000>;
> + regulator-max-microvolt = <3304000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l15b: ldo15 {
> + regulator-name = "vreg_l15b";
> + regulator-min-microvolt = <3300000>;
> + regulator-max-microvolt = <3304000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l16b: ldo16 {
> + regulator-name = "vreg_l16b";
> + regulator-min-microvolt = <3008000>;
> + regulator-max-microvolt = <3008000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l17b: ldo17 {
> + regulator-name = "vreg_l17b";
> + regulator-min-microvolt = <3104000>;
> + regulator-max-microvolt = <3104000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l18b: ldo18 {
> + regulator-name = "vreg_l18b";
> + regulator-min-microvolt = <2800000>;
> + regulator-max-microvolt = <2800000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l19b: ldo19 {
> + regulator-name = "vreg_l19b";
> + regulator-min-microvolt = <3000000>;
> + regulator-max-microvolt = <3000000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l20b: ldo20 {
> + regulator-name = "vreg_l20b";
> + regulator-min-microvolt = <1620000>;
> + regulator-max-microvolt = <3544000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l21b: ldo21 {
> + regulator-name = "vreg_l21b";
> + regulator-min-microvolt = <1620000>;
> + regulator-max-microvolt = <3544000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l22b: ldo22 {
> + regulator-name = "vreg_l22b";
> + regulator-min-microvolt = <3200000>;
> + regulator-max-microvolt = <3200000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l23b: ldo23 {
> + regulator-name = "vreg_l23b";
> + regulator-min-microvolt = <1650000>;
> + regulator-max-microvolt = <3544000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_bob: bob {
> + regulator-name = "vreg_bob";
> + regulator-min-microvolt = <3008000>;
> + regulator-max-microvolt = <3960000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> + };
> +
> + regulators-1 {
> + compatible = "qcom,pm8550vs-rpmh-regulators";
> +
> + vdd-l1-supply = <&vreg_s1b>;
> +
> + qcom,pmic-id = "d";
> +
> + vreg_l1d: ldo1 {
> + regulator-name = "vreg_l1d";
> + regulator-min-microvolt = <1140000>;
> + regulator-max-microvolt = <1260000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> + };
> +
> + regulators-2 {
> + compatible = "qcom,pm8550vs-rpmh-regulators";
> +
> + vdd-l1-supply = <&vreg_s2b>;
> + vdd-l3-supply = <&vreg_s2b>;
> +
> + qcom,pmic-id = "g";
> +
> + vreg_l1g: ldo1 {
> + regulator-name = "vreg_l1g";
> + regulator-min-microvolt = <1150000>;
> + regulator-max-microvolt = <1260000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l3g: ldo3 {
> + regulator-name = "vreg_l3g";
> + regulator-min-microvolt = <1150000>;
> + regulator-max-microvolt = <1260000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + };
> +
> + regulators-3 {
> + compatible = "qcom,pmr735d-rpmh-regulators";
> +
> + vdd-l1-l2-l5-supply = <&vreg_s3b>;
> + vdd-l3-l4-supply = <&vreg_s2b>;
> + vdd-l6-supply = <&vreg_s1b>;
> + vdd-l7-supply = <&vreg_s3b>;
> +
> + qcom,pmic-id = "k";
> +
> + vreg_l1k: ldo1 {
> + regulator-name = "vreg_l1k";
> + regulator-min-microvolt = <488000>;
> + regulator-max-microvolt = <912000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l2k: ldo2 {
> + regulator-name = "vreg_l2k";
> + regulator-min-microvolt = <920000>;
> + regulator-max-microvolt = <969000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l3k: ldo3 {
> + regulator-name = "vreg_l3k";
> + regulator-min-microvolt = <1080000>;
> + regulator-max-microvolt = <1350000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l4k: ldo4 {
> + regulator-name = "vreg_l4k";
> + regulator-min-microvolt = <960000>;
> + regulator-max-microvolt = <1980000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l5k: ldo5 {
> + regulator-name = "vreg_l5k";
> + regulator-min-microvolt = <866000>;
> + regulator-max-microvolt = <931000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l6k: ldo6 {
> + regulator-name = "vreg_l6k";
> + regulator-min-microvolt = <1100000>;
> + regulator-max-microvolt = <2000000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> +
> + vreg_l7k: ldo7 {
> + regulator-name = "vreg_l7k";
> + regulator-min-microvolt = <720000>;
> + regulator-max-microvolt = <958000>;
> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> + };
> + };
> +};
> +
> +&pm7550ba_eusb2_repeater {
> + vdd18-supply = <&vreg_l7b>;
> + vdd3-supply = <&vreg_l17b>;
> +};
> +
> +&gpu {
> + status = "okay";
> +};
> +
> +&gpu_zap_shader {
> + firmware-name = "qcom/eliza/gen71700_zap.mbn";
> +};
> +
> +&remoteproc_adsp {
> + firmware-name = "qcom/eliza/adsp.mbn",
> + "qcom/eliza/adsp_dtb.mbn";
> +
> + status = "okay";
> +};
> +
> +&remoteproc_cdsp {
> + firmware-name = "qcom/eliza/cdsp.mbn",
> + "qcom/eliza/cdsp_dtb.mbn";
> +
> + status = "okay";
> +};
> +
> +&tlmm {
> + gpio-reserved-ranges = <20 4>, /* NFC SPI */
> + <111 2>, /* WCN UART1 */
> + <118 1>; /* NFC Secure I/O */
> +};
> +
> +&ufs_mem_hc {
> + reset-gpios = <&tlmm 185 GPIO_ACTIVE_LOW>;
> +
> + vcc-supply = <&vreg_l12b>;
> + vcc-max-microamp = <1300000>;
> + vccq-supply = <&vreg_l1d>;
> + vccq-max-microamp = <1200000>;
> +
> + status = "okay";
> +};
> +
> +&ufs_mem_phy {
> + vdda-phy-supply = <&vreg_l6b>;
> + vdda-pll-supply = <&vreg_l4b>;
> +
> + status = "okay";
> +};
>
> --
> 2.54.0
>
next prev parent reply other threads:[~2026-09-28 2:15 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-15 12:03 [PATCH v2 0/5] arm64: dts: qcom: eliza-cqs-evk: Add storage subtypes and rework generic parts Abel Vesa
2026-09-15 12:03 ` [PATCH v2 1/5] dt-bindings: arm: qcom: Add more compatible for Eliza EVKs and SoMs Abel Vesa
2026-09-15 12:03 ` [PATCH v2 2/5] arm64: dts: qcom: eliza-cqs-evk: Move sound card support to Eliza EVK generic Abel Vesa
2026-09-23 12:52 ` Konrad Dybcio
2026-09-24 11:02 ` Abel Vesa
2026-09-15 12:03 ` [PATCH v2 3/5] arm64: dts: qcom: eliza: Rename existing EVK to suggest eMMC Abel Vesa
2026-09-15 12:03 ` [PATCH v2 4/5] arm64: dts: qcom: eliza: Add CQS SoM UFS variant Abel Vesa
2026-09-28 2:15 ` Bjorn Andersson [this message]
2026-09-15 12:03 ` [PATCH v2 5/5] arm64: dts: qcom: eliza: Add CQS EVK " Abel Vesa
2026-09-23 12:59 ` [PATCH v2 0/5] arm64: dts: qcom: eliza-cqs-evk: Add storage subtypes and rework generic parts Konrad Dybcio
2026-09-24 11:00 ` Abel Vesa
2026-09-28 2:18 ` Bjorn Andersson
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=arnM9pny5hIM0FaD@zenbook \
--to=andersson@kernel.org \
--cc=abel.vesa@oss.qualcomm.com \
--cc=abelvesa@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=krzk@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®