[PATCH] arm64: dts: qcom: msm8996: add reset for display subsystem

Barnabás Czémán posted 1 patch 1 year, 8 months ago
arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 ++
1 file changed, 2 insertions(+)
[PATCH] arm64: dts: qcom: msm8996: add reset for display subsystem
Posted by Barnabás Czémán 1 year, 8 months ago
Add reset for display subsystem, make sure it gets
properly reset.

Signed-off-by: Barnabás Czémán <trabarni@gmail.com>
---
 arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
index 8d2cb6f41095..5348feac026e 100644
--- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
@@ -982,6 +982,8 @@ mdss: display-subsystem@900000 {
 				 <&mmcc MDSS_MDP_CLK>;
 			clock-names = "iface", "core";
 
+			resets = <&mmcc MDSS_BCR>;
+
 			#address-cells = <1>;
 			#size-cells = <1>;
 			ranges;

---
base-commit: 3689b0ef08b70e4e03b82ebd37730a03a672853a
change-id: 20240525-mdss-reset-8b83eaa24b8c

Best regards,
-- 
Barnabás Czémán <trabarni@gmail.com>

Re: [PATCH] arm64: dts: qcom: msm8996: add reset for display subsystem
Posted by Bjorn Andersson 1 year, 8 months ago
On Sat, 25 May 2024 16:08:57 +0200, Barnabás Czémán wrote:
> Add reset for display subsystem, make sure it gets
> properly reset.
> 
> 

Applied, thanks!

[1/1] arm64: dts: qcom: msm8996: add reset for display subsystem
      commit: b3f8cdef8a5c0bc89ceb1095856f64ba7445c6b9

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>
Re: [PATCH] arm64: dts: qcom: msm8996: add reset for display subsystem
Posted by Dmitry Baryshkov 1 year, 8 months ago
On Sat, 25 May 2024 at 17:09, Barnabás Czémán <trabarni@gmail.com> wrote:
>
> Add reset for display subsystem, make sure it gets
> properly reset.
>
> Signed-off-by: Barnabás Czémán <trabarni@gmail.com>
> ---
>  arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 ++
>  1 file changed, 2 insertions(+)

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

[No need to apply any fixes tags, it is not a bug that was present
before and is being fixed, it is an additional functionality]


-- 
With best wishes
Dmitry
Re: [PATCH] arm64: dts: qcom: msm8996: add reset for display subsystem
Posted by Markus Elfring 1 year, 8 months ago
> Add reset for display subsystem, make sure it gets
> properly reset.

* I suggest to use occasionally more than 50 characters in lines of
  such change descriptions.

* Please reconsider word wrapping accordingly.

* Can such information be relevant for the tag “Fixes”?

Regards,
Markus
Re: [PATCH] arm64: dts: qcom: msm8996: add reset for display subsystem
Posted by Krzysztof Kozlowski 1 year, 8 months ago
On 25/05/2024 20:20, Markus Elfring wrote:
>> Add reset for display subsystem, make sure it gets
>> properly reset.
> 
> * I suggest to use occasionally more than 50 characters in lines of
>   such change descriptions.
> 
> * Please reconsider word wrapping accordingly.
> 
> * Can such information be relevant for the tag “Fixes”?

<form letter>
Feel free to ignore all comments from Markus, regardless whether the
suggestion is reasonable or not. This person is banned from LKML and
most maintainers ignore Markus' feedback, because it is just a waste of
time.
</form letter>

Best regards,
Krzysztof