Reviewed-by: Bhawanpreet Lakha <[email protected]><mailto:[email protected]>
On 2019-06-24 2:00 p.m., Deucher, Alexander wrote: Acked-by: Alex Deucher <[email protected]><mailto:[email protected]> ________________________________ From: amd-gfx <[email protected]><mailto:[email protected]> on behalf of Nicholas Kazlauskas <[email protected]><mailto:[email protected]> Sent: Monday, June 24, 2019 1:58 PM To: [email protected]<mailto:[email protected]> Cc: Li, Sun peng (Leo); Wentland, Harry; Kazlauskas, Nicholas; Li, Roman Subject: [PATCH v2] drm/amd/display: update infoframe after dig fe is turned on (v2) [Why] The AVI infoframe is incorrectly programmed on DCN1/2 when enabling a stream - causing the wrong pixel encoding to be used for display. This is because the AVI infoframe is programmed before the DIG BE is connected to the FE and turned on, so enabling the AFMT block doesn't actually work and the registers subsequently can't be written to. [How] Program the infoframe *after* turning on the DIG FE. This was the behavior previously used but it was incorrectly reverted when adding the DCN2 HW sequencer code. v2: Don't call update_info_frame twice Fixes: 54ff35915948 ("drm/amd/display: Add DCN2 HW Sequencer and Resource") Cc: Harry Wentland <[email protected]><mailto:[email protected]> Cc: Roman Li <[email protected]><mailto:[email protected]> Cc: Leo Li <[email protected]><mailto:[email protected]> Signed-off-by: Nicholas Kazlauskas <[email protected]><mailto:[email protected]> --- drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c b/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c index 940e74b7d2c6..84d90b475e2a 100644 --- a/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c +++ b/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c @@ -686,6 +686,7 @@ void dce110_enable_stream(struct pipe_ctx *pipe_ctx) dmdata_dp : dmdata_hdmi); } #endif + dce110_update_info_frame(pipe_ctx); /* enable early control to avoid corruption on DP monitor*/ active_total_with_borders = -- 2.17.1 _______________________________________________ amd-gfx mailing list [email protected]<mailto:[email protected]> https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list [email protected]<mailto:[email protected]> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
