Skip to content

Commit

Permalink
drm/amd/display:: Fix NULL pointer in Raven hotplug
Browse files Browse the repository at this point in the history
Programming sequence to frontend and backend has been switched.
In such case, program_scaler() is getting called when programming
frontend, and should be removed from backend programming routine.

Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Jerry (Fangzhi) Zuo <Jerry.Zuo@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
  • Loading branch information
Jerry (Fangzhi) Zuo authored and Alex Deucher committed Oct 21, 2017
1 parent af09e48 commit 9b38bd1
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
Original file line number Diff line number Diff line change
Expand Up @@ -1369,13 +1369,6 @@ static enum dc_status apply_single_controller_ctx_to_hw(
}

pipe_ctx->plane_res.scl_data.lb_params.alpha_en = pipe_ctx->bottom_pipe != 0;
/* program_scaler and allocate_mem_input are not new asic */
if ((!pipe_ctx_old ||
memcmp(&pipe_ctx_old->plane_res.scl_data, &pipe_ctx->plane_res.scl_data,
sizeof(struct scaler_data)) != 0) &&
pipe_ctx->plane_state) {
program_scaler(dc, pipe_ctx);
}

/* mst support - use total stream count */
if (pipe_ctx->plane_res.mi != NULL) {
Expand Down

0 comments on commit 9b38bd1

Please sign in to comment.