Re: Revert "arm64: dts: qcom: sm8250: remove bus clock from the mdss node for sm8250 target"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 10/14/21 10:42 AM, Steev Klimaszewski wrote:

On 10/14/21 8:54 AM, Dmitry Baryshkov wrote:
From: Amit Pundir <amit.pundir@xxxxxxxxxx>

This reverts commit 001ce9785c0674d913531345e86222c965fc8bf4.

This upstream commit broke AOSP (post Android 12 merge) build
on RB5. The device either silently crashes into USB crash mode
after android boot animation or we see a blank blue screen
with following dpu errors in dmesg:

[  T444] hw recovery is not complete for ctl:3
[  T444] [drm:dpu_encoder_phys_vid_prepare_for_kickoff:539] [dpu error]enc31 intf1 ctl 3 reset failure: -22 [  T444] [drm:dpu_encoder_phys_vid_wait_for_commit_done:513] [dpu error]vblank timeout [  T444] [drm:dpu_kms_wait_for_commit_done:454] [dpu error]wait for commit done returned -110 [    C7] [drm:dpu_encoder_frame_done_timeout:2127] [dpu error]enc31 frame done timeout [  T444] [drm:dpu_encoder_phys_vid_wait_for_commit_done:513] [dpu error]vblank timeout [  T444] [drm:dpu_kms_wait_for_commit_done:454] [dpu error]wait for commit done returned -110

Signed-off-by: Amit Pundir <amit.pundir@xxxxxxxxxx>
---
  arch/arm64/boot/dts/qcom/sm8250.dtsi | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi
index 8c15d9fed08f..d12e4cbfc852 100644
--- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
@@ -2590,9 +2590,10 @@
              power-domains = <&dispcc MDSS_GDSC>;
                clocks = <&dispcc DISP_CC_MDSS_AHB_CLK>,
+                 <&gcc GCC_DISP_HF_AXI_CLK>,
                   <&gcc GCC_DISP_SF_AXI_CLK>,
                   <&dispcc DISP_CC_MDSS_MDP_CLK>;
-            clock-names = "iface", "nrt_bus", "core";
+            clock-names = "iface", "bus", "nrt_bus", "core";
                assigned-clocks = <&dispcc DISP_CC_MDSS_MDP_CLK>;
              assigned-clock-rates = <460000000>;


Is this the correct fix for the issue?  I see those same messages on the Lenovo Yoga C630 every so often as well as the blue screen, however I see them on both 5.14 (which does not have the related commit to this), and in 5.15-rcX, which does.  Or does this just revert it enough that it doesn't occur as often?

Apologies, I hit send too soon.  To test here, I reverted 111c52854102983f which is the related sdm845 commit, and the first reboot to test reverting it, I ended up with the bluescreen and the messages mentioned in the commit message above.



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux