[PATCH 0/4] MSM8917/MSM837: Add missing MDSS reset

[PATCH 0/4] MSM8917/MSM837: Add missing MDSS reset

[Barnabás Czémán]

Add missing MDSS reset can be found on MSM8917 and MSM8937.

Signed-off-by: Barnabás Czémán <barnabas.czeman@mainlining.org>
---
Barnabás Czémán (4):
      dt-bindings: clock: gcc-msm8917: Add missing MDSS reset
      clk: qcom: gcc-msm8917: Add missing MDSS reset
      arm64: dts: qcom: msm8917: add reset for display subsystem
      arm64: dts: qcom: msm8937: add reset for display subsystem

 arch/arm64/boot/dts/qcom/msm8917.dtsi        | 2 +-
 arch/arm64/boot/dts/qcom/msm8937.dtsi        | 1 +
 drivers/clk/qcom/gcc-msm8917.c               | 1 +
 include/dt-bindings/clock/qcom,gcc-msm8917.h | 1 +
 4 files changed, 4 insertions(+), 1 deletion(-)
---
base-commit: 0f2995693867bfb26197b117cd55624ddc57582f
change-id: 20251116-mdss-resets-msm8917-msm8937-1b89f25a24d5

Best regards,
-- 
Barnabás Czémán <barnabas.czeman@mainlining.org>

[PATCH 1/4] dt-bindings: clock: gcc-msm8917: Add missing MDSS reset

[Barnabás Czémán]

Add definition for MDSS reset can be found on MSM8917 and MSM8937.

Signed-off-by: Barnabás Czémán <barnabas.czeman@mainlining.org>
---
 include/dt-bindings/clock/qcom,gcc-msm8917.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/dt-bindings/clock/qcom,gcc-msm8917.h b/include/dt-bindings/clock/qcom,gcc-msm8917.h
index 4e3897b3669d..4265460bfb30 100644
--- a/include/dt-bindings/clock/qcom,gcc-msm8917.h
+++ b/include/dt-bindings/clock/qcom,gcc-msm8917.h
@@ -194,6 +194,7 @@
 #define GCC_QUSB2_PHY_BCR			2
 #define GCC_USB_HS_BCR				3
 #define GCC_USB2_HS_PHY_ONLY_BCR		4
+#define GCC_MDSS_BCR				5
 
 /* GDSCs */
 #define CPP_GDSC				0

-- 
2.51.2

[PATCH 2/4] clk: qcom: gcc-msm8917: Add missing MDSS reset

[Barnabás Czémán]

From: Barnabás Czémán <trabarni@gmail.com>

Add missing MDSS reset can be found on MSM8917 and MSM8937.

Signed-off-by: Barnabás Czémán <barnabas.czeman@mainlining.org>
---
 drivers/clk/qcom/gcc-msm8917.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/qcom/gcc-msm8917.c b/drivers/clk/qcom/gcc-msm8917.c
index 0a1aa623cd49..248fa0fd86f5 100644
--- a/drivers/clk/qcom/gcc-msm8917.c
+++ b/drivers/clk/qcom/gcc-msm8917.c
@@ -3770,6 +3770,7 @@ static const struct qcom_reset_map gcc_msm8917_resets[] = {
 	[GCC_QUSB2_PHY_BCR]		= { 0x4103c },
 	[GCC_USB_HS_BCR]		= { 0x41000 },
 	[GCC_USB2_HS_PHY_ONLY_BCR]	= { 0x41034 },
+	[GCC_MDSS_BCR]			= { 0x4d074 },
 };
 
 static const struct regmap_config gcc_msm8917_regmap_config = {

-- 
2.51.2

[PATCH 3/4] arm64: dts: qcom: msm8917: add reset for display subsystem

[Barnabás Czémán]

Add reset for display subsystem, make sure it gets
properly reset.

Signed-off-by: Barnabás Czémán <barnabas.czeman@mainlining.org>
---
 arch/arm64/boot/dts/qcom/msm8917.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/msm8917.dtsi b/arch/arm64/boot/dts/qcom/msm8917.dtsi
index 8a642fce2e40..6251189b8e6d 100644
--- a/arch/arm64/boot/dts/qcom/msm8917.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8917.dtsi
@@ -996,7 +996,7 @@ mdss: display-subsystem@1a00000 {
 			clock-names = "iface",
 				      "bus",
 				      "vsync";
-
+			resets = <&gcc GCC_MDSS_BCR>;
 			interrupts = <GIC_SPI 72 IRQ_TYPE_LEVEL_HIGH>;
 
 			interrupt-controller;

-- 
2.51.2

[PATCH 4/4] arm64: dts: qcom: msm8937: add reset for display subsystem

[Barnabás Czémán]

Add reset for display subsystem, make sure it gets
properly reset.

Signed-off-by: Barnabás Czémán <barnabas.czeman@mainlining.org>
---
 arch/arm64/boot/dts/qcom/msm8937.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/qcom/msm8937.dtsi b/arch/arm64/boot/dts/qcom/msm8937.dtsi
index b93621080989..c94eb14e31bd 100644
--- a/arch/arm64/boot/dts/qcom/msm8937.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8937.dtsi
@@ -1044,6 +1044,7 @@ mdss: display-subsystem@1a00000 {
 			clock-names = "iface",
 				      "bus",
 				      "vsync";
+			resets = <&gcc GCC_MDSS_BCR>;
 			interrupts = <GIC_SPI 72 IRQ_TYPE_LEVEL_HIGH>;
 
 			interrupt-controller;

-- 
2.51.2

Re: [PATCH 2/4] clk: qcom: gcc-msm8917: Add missing MDSS reset

[barnabas.czeman]

On 2025-11-16 23:10, Barnabás Czémán wrote:
> From: Barnabás Czémán <trabarni@gmail.com>
This is my old mail, should I send a v2 to fix it?
> 
> Add missing MDSS reset can be found on MSM8917 and MSM8937.
> 
> Signed-off-by: Barnabás Czémán <barnabas.czeman@mainlining.org>
> ---
>  drivers/clk/qcom/gcc-msm8917.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/clk/qcom/gcc-msm8917.c 
> b/drivers/clk/qcom/gcc-msm8917.c
> index 0a1aa623cd49..248fa0fd86f5 100644
> --- a/drivers/clk/qcom/gcc-msm8917.c
> +++ b/drivers/clk/qcom/gcc-msm8917.c
> @@ -3770,6 +3770,7 @@ static const struct qcom_reset_map 
> gcc_msm8917_resets[] = {
>  	[GCC_QUSB2_PHY_BCR]		= { 0x4103c },
>  	[GCC_USB_HS_BCR]		= { 0x41000 },
>  	[GCC_USB2_HS_PHY_ONLY_BCR]	= { 0x41034 },
> +	[GCC_MDSS_BCR]			= { 0x4d074 },
>  };
> 
>  static const struct regmap_config gcc_msm8917_regmap_config = {

Re: [PATCH 2/4] clk: qcom: gcc-msm8917: Add missing MDSS reset

[Taniya Das]

On 11/17/2025 3:40 AM, Barnabás Czémán wrote:
> From: Barnabás Czémán <trabarni@gmail.com>
> 
> Add missing MDSS reset can be found on MSM8917 and MSM8937.
> 
> Signed-off-by: Barnabás Czémán <barnabas.czeman@mainlining.org>
> ---
>  drivers/clk/qcom/gcc-msm8917.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/clk/qcom/gcc-msm8917.c b/drivers/clk/qcom/gcc-msm8917.c
> index 0a1aa623cd49..248fa0fd86f5 100644
> --- a/drivers/clk/qcom/gcc-msm8917.c
> +++ b/drivers/clk/qcom/gcc-msm8917.c
> @@ -3770,6 +3770,7 @@ static const struct qcom_reset_map gcc_msm8917_resets[] = {
>  	[GCC_QUSB2_PHY_BCR]		= { 0x4103c },
>  	[GCC_USB_HS_BCR]		= { 0x41000 },
>  	[GCC_USB2_HS_PHY_ONLY_BCR]	= { 0x41034 },
> +	[GCC_MDSS_BCR]			= { 0x4d074 },

Reviewed-by: Taniya Das <taniya.das@oss.qualcomm.com>

>  };
>  
>  static const struct regmap_config gcc_msm8917_regmap_config = {
> 

-- 
Thanks,
Taniya Das

Re: [PATCH 4/4] arm64: dts: qcom: msm8937: add reset for display subsystem

[Konrad Dybcio]

On 11/16/25 11:10 PM, Barnabás Czémán wrote:
> Add reset for display subsystem, make sure it gets
> properly reset.

This sounds a little funny (and wrapped very early - usually 72c is
the norm)

> Signed-off-by: Barnabás Czémán <barnabas.czeman@mainlining.org>
> ---

in any case

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Konrad

Re: [PATCH 3/4] arm64: dts: qcom: msm8917: add reset for display subsystem

[Konrad Dybcio]

On 11/16/25 11:10 PM, Barnabás Czémán wrote:
> Add reset for display subsystem, make sure it gets
> properly reset.
> 
> Signed-off-by: Barnabás Czémán <barnabas.czeman@mainlining.org>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Konrad