Messages in this thread | | | Date | Tue, 3 Oct 2023 09:16:50 +0200 | From | neil.armstrong@linaro ... | Subject | Re: [PATCH v3 3/3] clk: qcom: add SM8550 DISPCC driver |
| |
Hi,
On 03/10/2023 04:17, Sophon Wu wrote: > On 09/01/2023 16:47, Neil Armstrong wrote: > >> Add support for the display clock controller found in SM8550 >> based devices. > >> This clock controller feeds the Multimedia Display SubSystem (MDSS). >> This driver is based on the SM8450 support. > >> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> >> Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org> >> --- > > Hi Neil, > > I'm trying to enable display on SM8550 but having trouble with clocks. Do you > have any idea on this maybe? Full dmesg here: https://bpa.st/7E6Q
You may need to remove the cont-splash memory zone and the simple-framebuffer, if you leave the cont-splash the bootloader will leave the MDSS on and Linux will fail to take over and initialize the clocks.
This is known issue, and for now we re-initialize everything from Linux after ABL disables the display subsystem entirely.
Neil
> > Regards, > Xilin > > ------------[ cut here ]------------ > disp_cc_mdss_mdp_clk_src: rcg didn't update its configuration. > WARNING: CPU: 4 PID: 73 at drivers/clk/qcom/clk-rcg2.c:133 > update_config+0xd4/0xf0 > Modules linked in: > CPU: 4 PID: 73 Comm: kworker/u16:2 Tainted: G S > 6.6.0-rc3-next-20230929-00006-g019d41d2e78e-dirty #252 > d692e32c53d7001f1537e28f80b45291db95a48f > Hardware name: AYN Odin 2 (DT) > Workqueue: events_unbound deferred_probe_work_func > pstate: 614000c5 (nZCv daIF +PAN -UAO -TCO +DIT -SSBS BTYPE=--) > pc : update_config+0xd4/0xf0 > lr : update_config+0xd4/0xf0 > sp : ffff8000861435d0 > x29: ffff8000861435d0 x28: 0000000000000000 x27: 0000000000000000 > x26: ffff0008011da0f4 x25: 0000000000000000 x24: 0000000000000000 > x23: 0000000000000004 x22: ffff000804f0eac0 x21: ffff800082395f68 > x20: ffff800085109c20 x19: 0000000000000000 x18: 0000000000000014 > x17: 0000000043567dcd x16: 00000000c6ba16a0 x15: 0000000040b16970 > x14: 0000000000000001 x13: 2e6e6f6974617275 x12: 6769666e6f632073 > x11: 7469206574616470 x10: 752074276e646964 x9 : ffff8000801bfc5c > x8 : 2073746920657461 x7 : 6470752074276e64 x6 : 0000000000000010 > x5 : 40000000ffff0254 x4 : ffff8000848907d0 x3 : 00000000ffffffff > x2 : 0000000000000000 x1 : 0000000000000000 x0 : ffff000801320000 > Call trace: > update_config+0xd4/0xf0 > clk_rcg2_shared_enable+0x58/0x98 > clk_core_enable+0x78/0x1f8 > clk_core_enable+0x58/0x1f8 > clk_enable+0x34/0x60 > clk_bulk_enable+0x54/0xe8 > msm_mdss_enable+0xac/0x250 > mdss_runtime_resume+0x3c/0x58 > pm_generic_runtime_resume+0x34/0x58 > __genpd_runtime_resume+0x38/0x90 > genpd_runtime_resume+0x11c/0x2b0 > __rpm_callback+0x50/0x1f0 > rpm_callback+0x74/0x88 > rpm_resume+0x534/0x760 > __pm_runtime_resume+0x64/0xc0 > __device_attach+0x8c/0x1c0 > device_initial_probe+0x1c/0x30 > bus_probe_device+0xb4/0xc0 > device_add+0x64c/0x848 > of_device_add+0x4c/0x70 > of_platform_device_create_pdata+0x94/0x130 > of_platform_bus_create+0x188/0x4c8 > of_platform_populate+0x60/0x160 > mdss_probe+0x278/0x398 > platform_probe+0x70/0xd8 > really_probe+0x190/0x3d8 > __driver_probe_device+0x84/0x180 > driver_probe_device+0x44/0x120 > __device_attach_driver+0xc4/0x168 > bus_for_each_drv+0x8c/0xf0 > __device_attach+0xa4/0x1c0 > device_initial_probe+0x1c/0x30 > bus_probe_device+0xb4/0xc0 > deferred_probe_work_func+0xbc/0x118 > process_one_work+0x154/0x3c8 > worker_thread+0x2bc/0x3e0 > kthread+0x118/0x128 > ret_from_fork+0x10/0x20 > ---[ end trace 0000000000000000 ]--- > Failed to enable clk '(null)': -16 > msm-mdss ae00000.display-subsystem: clock enable failed, ret:-16 > ------------[ cut here ]------------ > disp_cc_mdss_ahb_clk status stuck at 'off' > WARNING: CPU: 4 PID: 73 at drivers/clk/qcom/clk-branch.c:86 > clk_branch_wait+0x148/0x168 > Modules linked in: > CPU: 4 PID: 73 Comm: kworker/u16:2 Tainted: G S W > 6.6.0-rc3-next-20230929-00006-g019d41d2e78e-dirty #252 > d692e32c53d7001f1537e28f80b45291db95a48f > Hardware name: AYN Odin 2 (DT) > Workqueue: events_unbound deferred_probe_work_func > pstate: 614000c5 (nZCv daIF +PAN -UAO -TCO +DIT -SSBS BTYPE=--) > pc : clk_branch_wait+0x148/0x168 > lr : clk_branch_wait+0x148/0x168 > sp : ffff800086143650 > x29: ffff800086143650 x28: 0000000000000000 x27: 0000000000000000 > x26: 0000000000000000 x25: ffff000a7e7b50d8 x24: ffff80008517ac90 > x23: ffff800082395ed0 x22: 0000000000000001 x21: ffff800080bb09f0 > x20: 0000000000000000 x19: ffff8000851094d0 x18: 0000000000000006 > x17: 0000000000000000 x16: 0000000000000020 x15: 0000000000000002 > x14: 0000000000000001 x13: 0000000000000004 x12: 0000000000000000 > x11: 0000000000000000 x10: 0000000000000020 x9 : ffff8000801bfc5c > x8 : 0000000000000020 x7 : 00000000ffffffff x6 : 0000000000000000 > x5 : 0000000000000050 x4 : ffff800083df7e80 x3 : 00000000ffffffff > x2 : 0000000000000000 x1 : 0000000000000000 x0 : ffff000801320000 > Call trace: > clk_branch_wait+0x148/0x168 > clk_branch2_enable+0x38/0x50 > clk_core_enable+0x78/0x1f8 > clk_enable+0x34/0x60 > dsi_phy_enable_resource+0x98/0xb8 > dsi_phy_driver_probe+0x26c/0x408 > platform_probe+0x70/0xd8 > really_probe+0x190/0x3d8 > __driver_probe_device+0x84/0x180 > driver_probe_device+0x44/0x120 > __device_attach_driver+0xc4/0x168 > bus_for_each_drv+0x8c/0xf0 > __device_attach+0xa4/0x1c0 > device_initial_probe+0x1c/0x30 > bus_probe_device+0xb4/0xc0 > device_add+0x64c/0x848 > of_device_add+0x4c/0x70 > of_platform_device_create_pdata+0x94/0x130 > of_platform_bus_create+0x188/0x4c8 > of_platform_populate+0x60/0x160 > mdss_probe+0x278/0x398 > platform_probe+0x70/0xd8 > really_probe+0x190/0x3d8 > __driver_probe_device+0x84/0x180 > driver_probe_device+0x44/0x120 > __device_attach_driver+0xc4/0x168 > bus_for_each_drv+0x8c/0xf0 > __device_attach+0xa4/0x1c0 > device_initial_probe+0x1c/0x30 > bus_probe_device+0xb4/0xc0 > deferred_probe_work_func+0xbc/0x118 > process_one_work+0x154/0x3c8 > worker_thread+0x2bc/0x3e0 > kthread+0x118/0x128 > ret_from_fork+0x10/0x20 > ---[ end trace 0000000000000000 ]--- > msm_dsi_phy ae97000.phy: [drm:dsi_phy_enable_resource] *ERROR* > dsi_phy_enable_resource: can't enable ahb clk, -16 > msm_dsi_phy: probe of ae97000.phy failed with error -16
| |