| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <bubac7n7qx2ww5dzzsth7dsw5vtzwfom2r42ox6n6aft474hyx@mlo3jpfozvha>
Date: Sat, 19 Jul 2025 22:36:58 -0500
From: Bjorn Andersson <andersson@...nel.org>
To: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
Cc: Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
Ulf Hansson <ulf.hansson@...aro.org>, Konrad Dybcio <konradybcio@...nel.org>,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-pm@...r.kernel.org
Subject: Re: [PATCH 5/8] arm64: dts: qcom: dts: switch to RPMPD_* indices
On Fri, Jul 18, 2025 at 07:13:43PM +0300, Dmitry Baryshkov wrote:
> Use generic RPMPD_* defines for power domain instead of using
> platform-specific defines.
>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
Reviewed-by: Bjorn Andersson <andersson@...nel.org>
Acked-by: Bjorn Andersson <andersson@...nel.org>
Regards,
Bjorn
> ---
> arch/arm64/boot/dts/qcom/msm8916.dtsi | 8 ++++----
> arch/arm64/boot/dts/qcom/msm8917.dtsi | 10 +++++-----
> arch/arm64/boot/dts/qcom/msm8976.dtsi | 4 ++--
> arch/arm64/boot/dts/qcom/msm8998.dtsi | 16 ++++++++--------
> arch/arm64/boot/dts/qcom/sdm630.dtsi | 16 ++++++++--------
> arch/arm64/boot/dts/qcom/sm6125.dtsi | 12 ++++++------
> 6 files changed, 33 insertions(+), 33 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi
> index b50c7e6e0bfcd35ab4f8b84aeabe214fd60e8d7c..ca184ee3af065df0dfb3c7106d29360a860b8b36 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi
> @@ -1943,8 +1943,8 @@ mpss: remoteproc@...0000 {
> interrupt-names = "wdog", "fatal", "ready",
> "handover", "stop-ack";
>
> - power-domains = <&rpmpd MSM8916_VDDCX>,
> - <&rpmpd MSM8916_VDDMX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>,
> + <&rpmpd RPMPD_VDDMX>;
> power-domain-names = "cx", "mx";
>
> clocks = <&gcc GCC_MSS_CFG_AHB_CLK>,
> @@ -2445,8 +2445,8 @@ wcnss: remoteproc@...4000 {
> <&wcnss_smp2p_in 3 IRQ_TYPE_EDGE_RISING>;
> interrupt-names = "wdog", "fatal", "ready", "handover", "stop-ack";
>
> - power-domains = <&rpmpd MSM8916_VDDCX>,
> - <&rpmpd MSM8916_VDDMX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>,
> + <&rpmpd RPMPD_VDDMX>;
> power-domain-names = "cx", "mx";
>
> qcom,smem-states = <&wcnss_smp2p_out 0>;
> diff --git a/arch/arm64/boot/dts/qcom/msm8917.dtsi b/arch/arm64/boot/dts/qcom/msm8917.dtsi
> index 8a642fce2e40d6d252a1cfbdfed602e6789ef09a..87dfc1e7e131cd4192ffbdd4913f934af17c5c88 100644
> --- a/arch/arm64/boot/dts/qcom/msm8917.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8917.dtsi
> @@ -1070,7 +1070,7 @@ mdss_dsi0: dsi@...4000 {
> phys = <&mdss_dsi0_phy>;
>
> operating-points-v2 = <&mdss_dsi0_opp_table>;
> - power-domains = <&rpmpd MSM8917_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> #address-cells = <1>;
> #size-cells = <0>;
> @@ -1288,7 +1288,7 @@ sdhc_1: mmc@...4900 {
> pinctrl-0 = <&sdc1_default>;
> pinctrl-1 = <&sdc1_sleep>;
> pinctrl-names = "default", "sleep";
> - power-domains = <&rpmpd MSM8917_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
> mmc-hs200-1_8v;
> mmc-hs400-1_8v;
> mmc-ddr-1_8v;
> @@ -1313,7 +1313,7 @@ sdhc_2: mmc@...4900 {
> pinctrl-0 = <&sdc2_default>;
> pinctrl-1 = <&sdc2_sleep>;
> pinctrl-names = "default", "sleep";
> - power-domains = <&rpmpd MSM8917_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
> bus-width = <4>;
> status = "disabled";
> };
> @@ -1517,8 +1517,8 @@ wcnss: remoteproc@...4000 {
> <&wcnss_smp2p_in 3 IRQ_TYPE_EDGE_RISING>;
> interrupt-names = "wdog", "fatal", "ready", "handover", "stop-ack";
>
> - power-domains = <&rpmpd MSM8917_VDDCX>,
> - <&rpmpd MSM8917_VDDMX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>,
> + <&rpmpd RPMPD_VDDMX>;
> power-domain-names = "cx", "mx";
>
> qcom,smem-states = <&wcnss_smp2p_out 0>;
> diff --git a/arch/arm64/boot/dts/qcom/msm8976.dtsi b/arch/arm64/boot/dts/qcom/msm8976.dtsi
> index f9962512f243d6c1af4931787f4602554c63bb39..80a0a09e055d79c5e56f89e5d67baf9971d87069 100644
> --- a/arch/arm64/boot/dts/qcom/msm8976.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8976.dtsi
> @@ -1558,8 +1558,8 @@ wcnss: remoteproc@...4000 {
> "handover",
> "stop-ack";
>
> - power-domains = <&rpmpd MSM8976_VDDCX>,
> - <&rpmpd MSM8976_VDDMX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>,
> + <&rpmpd RPMPD_VDDMX>;
> power-domain-names = "cx", "mx";
>
> qcom,smem-states = <&wcnss_smp2p_out 0>;
> diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi b/arch/arm64/boot/dts/qcom/msm8998.dtsi
> index 0b0a9379cb05b01ebb17cef9fe7d22d56790efd3..2d7c45fec94e868b41a2b8d7acf8175a7b02c0b7 100644
> --- a/arch/arm64/boot/dts/qcom/msm8998.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8998.dtsi
> @@ -1497,8 +1497,8 @@ remoteproc_mss: remoteproc@...0000 {
>
> qcom,halt-regs = <&tcsr_regs_1 0x3000 0x5000 0x4000>;
>
> - power-domains = <&rpmpd MSM8998_VDDCX>,
> - <&rpmpd MSM8998_VDDMX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>,
> + <&rpmpd RPMPD_VDDMX>;
> power-domain-names = "cx", "mx";
>
> status = "disabled";
> @@ -1544,7 +1544,7 @@ adreno_gpu: gpu@...0000 {
> interrupts = <GIC_SPI 300 IRQ_TYPE_LEVEL_HIGH>;
> iommus = <&adreno_smmu 0>;
> operating-points-v2 = <&gpu_opp_table>;
> - power-domains = <&rpmpd MSM8998_VDDMX>;
> + power-domains = <&rpmpd RPMPD_VDDMX>;
> status = "disabled";
>
> gpu_opp_table: opp-table {
> @@ -1680,7 +1680,7 @@ remoteproc_slpi: remoteproc@...0000 {
> qcom,smem-states = <&slpi_smp2p_out 0>;
> qcom,smem-state-names = "stop";
>
> - power-domains = <&rpmpd MSM8998_SSCCX>;
> + power-domains = <&rpmpd RPMPD_SSCCX>;
> power-domain-names = "ssc_cx";
>
> status = "disabled";
> @@ -2871,7 +2871,7 @@ mdss_mdp: display-controller@...1000 {
> assigned-clock-rates = <19200000>;
>
> operating-points-v2 = <&mdp_opp_table>;
> - power-domains = <&rpmpd MSM8998_VDDMX>;
> + power-domains = <&rpmpd RPMPD_VDDMX>;
>
> mdp_opp_table: opp-table {
> compatible = "operating-points-v2";
> @@ -2953,7 +2953,7 @@ mdss_dsi0: dsi@...4000 {
> <&mdss_dsi0_phy DSI_PIXEL_PLL_CLK>;
>
> operating-points-v2 = <&dsi_opp_table>;
> - power-domains = <&rpmpd MSM8998_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> phys = <&mdss_dsi0_phy>;
> phy-names = "dsi";
> @@ -3029,7 +3029,7 @@ mdss_dsi1: dsi@...6000 {
> <&mdss_dsi1_phy DSI_PIXEL_PLL_CLK>;
>
> operating-points-v2 = <&dsi_opp_table>;
> - power-domains = <&rpmpd MSM8998_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> phys = <&mdss_dsi1_phy>;
> phy-names = "dsi";
> @@ -3277,7 +3277,7 @@ remoteproc_adsp: remoteproc@...00000 {
> qcom,smem-states = <&adsp_smp2p_out 0>;
> qcom,smem-state-names = "stop";
>
> - power-domains = <&rpmpd MSM8998_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
> power-domain-names = "cx";
>
> status = "disabled";
> diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> index 8b1a45a4e56ed1ae02e5bb6e78ca6255d87add1c..412d7f7d6de67008c1f406ac5a73a51ff7f4f98d 100644
> --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> @@ -1058,8 +1058,8 @@ remoteproc_mss: remoteproc@...0000 {
>
> qcom,halt-regs = <&tcsr_regs_1 0x3000 0x5000 0x4000>;
>
> - power-domains = <&rpmpd SDM660_VDDCX>,
> - <&rpmpd SDM660_VDDMX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>,
> + <&rpmpd RPMPD_VDDMX>;
> power-domain-names = "cx", "mx";
>
> memory-region = <&mba_region>, <&mpss_region>, <&mdata_mem>;
> @@ -1096,7 +1096,7 @@ adreno_gpu: gpu@...0000 {
> "rbcpr",
> "core";
>
> - power-domains = <&rpmpd SDM660_VDDMX>;
> + power-domains = <&rpmpd RPMPD_VDDMX>;
> iommus = <&kgsl_smmu 0>;
>
> nvmem-cells = <&gpu_speed_bin>;
> @@ -1396,7 +1396,7 @@ sdhc_2: mmc@...4000 {
> pinctrl-names = "default", "sleep";
> pinctrl-0 = <&sdc2_state_on>;
> pinctrl-1 = <&sdc2_state_off>;
> - power-domains = <&rpmpd SDM660_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> status = "disabled";
>
> @@ -1450,7 +1450,7 @@ sdhc_1: mmc@...4000 {
> pinctrl-names = "default", "sleep";
> pinctrl-0 = <&sdc1_state_on>;
> pinctrl-1 = <&sdc1_state_off>;
> - power-domains = <&rpmpd SDM660_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> bus-width = <8>;
> non-removable;
> @@ -1612,7 +1612,7 @@ mdp: display-controller@...1000 {
> "rotator-mem";
> iommus = <&mmss_smmu 0>;
> operating-points-v2 = <&mdp_opp_table>;
> - power-domains = <&rpmpd SDM660_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> ports {
> #address-cells = <1>;
> @@ -1664,7 +1664,7 @@ mdss_dsi0: dsi@...4000 {
> reg-names = "dsi_ctrl";
>
> operating-points-v2 = <&dsi_opp_table>;
> - power-domains = <&rpmpd SDM660_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> interrupt-parent = <&mdss>;
> interrupts = <4>;
> @@ -2280,7 +2280,7 @@ adsp_pil: remoteproc@...00000 {
> clock-names = "xo";
>
> memory-region = <&adsp_region>;
> - power-domains = <&rpmpd SDM660_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
> power-domain-names = "cx";
>
> qcom,smem-states = <&adsp_smp2p_out 0>;
> diff --git a/arch/arm64/boot/dts/qcom/sm6125.dtsi b/arch/arm64/boot/dts/qcom/sm6125.dtsi
> index 8f2d65543373e70b48b4015478e21e8e74fd23c9..80c42dff5399b7c60576c2b6cf79afdbd8b9b1d4 100644
> --- a/arch/arm64/boot/dts/qcom/sm6125.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm6125.dtsi
> @@ -724,7 +724,7 @@ sdhc_1: mmc@...4000 {
> clock-names = "iface", "core", "xo";
> iommus = <&apps_smmu 0x160 0x0>;
>
> - power-domains = <&rpmpd SM6125_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> qcom,dll-config = <0x000f642c>;
> qcom,ddr-config = <0x80040873>;
> @@ -755,7 +755,7 @@ sdhc_2: mmc@...4000 {
> pinctrl-1 = <&sdc2_off_state>;
> pinctrl-names = "default", "sleep";
>
> - power-domains = <&rpmpd SM6125_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> qcom,dll-config = <0x0007642c>;
> qcom,ddr-config = <0x80040873>;
> @@ -1275,7 +1275,7 @@ mdss_mdp: display-controller@...1000 {
> assigned-clock-rates = <19200000>;
>
> operating-points-v2 = <&mdp_opp_table>;
> - power-domains = <&rpmpd SM6125_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> ports {
> #address-cells = <1>;
> @@ -1345,7 +1345,7 @@ mdss_dsi0: dsi@...4000 {
> <&mdss_dsi0_phy DSI_PIXEL_PLL_CLK>;
>
> operating-points-v2 = <&dsi_opp_table>;
> - power-domains = <&rpmpd SM6125_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> phys = <&mdss_dsi0_phy>;
> phy-names = "dsi";
> @@ -1406,7 +1406,7 @@ mdss_dsi0_phy: phy@...4400 {
> "ref";
>
> required-opps = <&rpmpd_opp_nom>;
> - power-domains = <&rpmpd SM6125_VDDMX>;
> + power-domains = <&rpmpd RPMPD_VDDMX>;
>
> status = "disabled";
> };
> @@ -1434,7 +1434,7 @@ dispcc: clock-controller@...0000 {
> "gcc_disp_gpll0_div_clk_src";
>
> required-opps = <&rpmpd_opp_ret>;
> - power-domains = <&rpmpd SM6125_VDDCX>;
> + power-domains = <&rpmpd RPMPD_VDDCX>;
>
> #clock-cells = <1>;
> #power-domain-cells = <1>;
>
> --
> 2.39.5
>
Powered by blists - more mailing lists