Re: [PATCH] drm/amd/display: [FIX] update clock under two conditions
Hi
[This is an automated email]
This commit has been processed because it contains a "Fixes:" tag
fixing commit: .
The bot has tested the following trees: v5.8.10, v5.4.66, v4.19.146, v4.14.198,
v4.9.236, v4.4.236.
v5.8.10: Failed to apply! Possible dependencies:
598c13b21e25 ("drm/amd/display: update clock when non-seamless boot stream
exist")
b7efa4f5cdb4 ("drm/amd/display: Move call to disable DPG")
v5.4.66: Failed to apply! Possible dependencies:
3a4d180d4a9d ("drm/amd/display: Optimize clocks on clock change")
598c13b21e25 ("drm/amd/display: update clock when non-seamless boot stream
exist")
6b5d7730d226 ("drm/amd/display: Add wait for flip not pending on pipe
unlock")
7f7652ee8c8c ("drm/amd/display: enable single dp seamless boot")
9ae1b27f31d0 ("drm/amd/display: fix hotplug during display off")
b6e881c94741 ("drm/amd/display: update navi to use new surface programming
behaviour")
ccce745c28d6 ("drm/amd/display: Enable Seamless Boot Transition for
Multiple Streams")
ce10a0f39b19 ("drm/amd/display: use vbios message to call smu for dpm
level")
f2988e67144a ("drm/amd/display: optimize bandwidth after commit streams.")
v4.19.146: Failed to apply! Possible dependencies:
04a789bef315 ("drm/amd/display: add stream ID and otg instance in
dc_stream_state")
077d0b6ba211 ("drm/amd/display: Remove i2caux folder")
097578091327 ("drm/amd/display: Set gamma not working on MPO planes")
0cf5eb76e2b4 ("drm/amd/display: Add tracing to dc")
1e7e86c43f38 ("drm/amd/display: decouple front and backend pgm using
dpms_off as backend enable flag")
37cd85ce3322 ("drm/amd/display: Remove dc_stream_state->status")
56780940389a ("drm/amd/display: Remove redundant non-zero and overflow
check")
8c3db1284a01 ("drm/amdgpu: fill in
amdgpu_dm_remove_sink_from_freesync_module")
98e6436d3af5 ("drm/amd/display: Refactor FreeSync module")
9b93eb475aa9 ("drm/amd/display: move clk_mgr files to right place")
ad6756b4d773 ("drm/amd/display: Shift dc link aux to aux_payload")
ccce745c28d6 ("drm/amd/display: Enable Seamless Boot Transition for
Multiple Streams")
ceb3dbb4690d ("drm/amd/display: remove sink reference in dc_stream_state")
d82f99422b21 ("drm/amd/display: move edp fast boot optimization flag to
stream")
dc6c981d2027 ("drm/amd/display: Use DGAM ROM or RAM")
eae5ffa9bd7b ("drm/amd/display: Switch ddc to new aux interface")
fcee01b9f82d ("drm/amd/display: Add DCN2 clk mgr")
v4.14.198: Failed to apply! Possible dependencies:
1b0c0f9dc5ca ("drm/amdgpu: move userptr BOs to CPU domain during CS v2")
3fe89771cb0a ("drm/amdgpu: stop reserving the BO in the MMU callback v3")
4562236b3bc0 ("drm/amd/dc: Add dc display driver (v2)")
60de1c1740f3 ("drm/amdgpu: use a rw_semaphore for MMU notifiers")
9a18999640fa ("drm/amdgpu: move MMU notifier related defines to
amdgpu_mn.h")
9b93eb475aa9 ("drm/amd/display: move clk_mgr files to right place")
9cca0b8e5df0 ("drm/amdgpu: move amdgpu_cs_sysvm_access_required into
find_mapping")
a216ab09955d ("drm/amdgpu: fix userptr put_page handling")
b72cf4fca2bb ("drm/amdgpu: move taking mmap_sem into get_user_pages v2")
ca666a3c298f ("drm/amdgpu: stop using BO status for user pages")
ccce745c28d6 ("drm/amd/display: Enable Seamless Boot Transition for
Multiple Streams")
fcee01b9f82d ("drm/amd/display: Add DCN2 clk mgr")
v4.9.236: Failed to apply! Possible dependencies:
1cec20f0ea0e ("dma-buf: Restart reservation_object_wait_timeout_rcu() after
writes")
248a1d6f1ac4 ("drm/amd: fix include notation and remove -Iinclude/drm flag")
4562236b3bc0 ("drm/amd/dc: Add dc display driver (v2)")
78010cd9736e ("dma-buf/fence: add an lockdep_assert_held()")
9b93eb475aa9 ("drm/amd/display: move clk_mgr files to right place")
ccce745c28d6 ("drm/amd/display: Enable Seamless Boot Transition for
Multiple Streams")
f54d1867005c ("dma-buf: Rename struct fence to dma_fence")
fcee01b9f82d ("drm/amd/display: Add DCN2 clk mgr")
fedf54132d24 ("dma-buf: Restart reservation_object_get_fences_rcu() after
writes")
v4.4.236: Failed to apply! Possible dependencies:
0f477c6dea70 ("staging/android/sync: add sync_fence_create_dma")
1f7371b2a5fa ("drm/amd/powerplay: add basic powerplay framework")
248a1d6f1ac4 ("drm/amd: fix include notation and remove -Iinclude/drm flag")
288912cb95d1 ("drm/amdgpu: use $(src) in Makefile (v2)")
375fb53ec1be ("staging: android: replace explicit NULL comparison")
395dec6f6bc5 ("Documentation: add doc for sync_file_get_fence()")
4325198180e5 ("drm/amdgpu: remove GART page addr array")
4562236b3bc0 ("drm/amd/dc: Add dc display driver (v2)")
62304fb1fc08 ("dma-buf/sync_file: de-stage sync_file")
9b93eb475aa9 ("drm/amd/display: move clk_mgr files to right place")
a1d29476d666 ("drm/amdgpu: optionally enable GART debugfs file")
a8fe58cec351 ("drm/amd: add
[PATCH] drm/amd/display: [FIX] update clock under two conditions
[Why]
Update clock only when non-seamless boot stream exists
creates regression on multiple scenerios.
[How]
Update clock in two conditions
1. Non-seamless boot stream exist.
2. Stream_count = 0
Fixes:06f9b1475d98("drm/amd/display: update clock
when non-seamless boot stream exist")
Signed-off-by: Lewis Huang
Acked-by: Qingqing Zhuo
Cc:
---
drivers/gpu/drm/amd/display/dc/core/dc.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c
b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 1efc823c2a14..7e74ddc1c708 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -1286,7 +1286,8 @@ static enum dc_status dc_commit_state_no_check(struct dc
*dc, struct dc_state *c
dc->optimize_seamless_boot_streams++;
}
- if (context->stream_count > dc->optimize_seamless_boot_streams)
+ if (context->stream_count > dc->optimize_seamless_boot_streams ||
+ context->stream_count == 0)
dc->hwss.prepare_bandwidth(dc, context);
disable_dangling_plane(dc, context);
@@ -1368,7 +1369,8 @@ static enum dc_status dc_commit_state_no_check(struct dc
*dc, struct dc_state *c
dc_enable_stereo(dc, context, dc_streams, context->stream_count);
- if (context->stream_count > dc->optimize_seamless_boot_streams) {
+ if (context->stream_count > dc->optimize_seamless_boot_streams ||
+ context->stream_count == 0) {
/* Must wait for no flips to be pending before doing optimize
bw */
wait_for_no_pipes_pending(dc, context);
/* pplib is notified if disp_num changed */
--
2.17.1
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
