Commit 89e89630 authored by Amy Zhang's avatar Amy Zhang Committed by Alex Deucher

drm/amd/display: Fix Gamma Adjustment

- Gamma correction is not properly copied to the surface after refactor
- Make sure gamma correction is copied with correct retain
Signed-off-by: default avatarAmy Zhang <Amy.Zhang@amd.com>
Reviewed-by: default avatarAnthony Koo <Anthony.Koo@amd.com>
Acked-by: default avatarHarry Wentland <Harry.Wentland@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 9474980a
...@@ -1302,8 +1302,8 @@ bool dc_commit_surfaces_to_target( ...@@ -1302,8 +1302,8 @@ bool dc_commit_surfaces_to_target(
for (i = 0; i < new_surface_count; i++) { for (i = 0; i < new_surface_count; i++) {
updates[i].surface = new_surfaces[i]; updates[i].surface = new_surfaces[i];
updates[i].gamma = (struct dc_gamma *)new_surfaces[i]->gamma_correction; updates[i].gamma =
(struct dc_gamma *)new_surfaces[i]->gamma_correction;
flip_addr[i].address = new_surfaces[i]->address; flip_addr[i].address = new_surfaces[i]->address;
flip_addr[i].flip_immediate = new_surfaces[i]->flip_immediate; flip_addr[i].flip_immediate = new_surfaces[i]->flip_immediate;
plane_info[i].color_space = new_surfaces[i]->color_space; plane_info[i].color_space = new_surfaces[i]->color_space;
...@@ -1444,6 +1444,16 @@ void dc_update_surfaces_for_target(struct dc *dc, struct dc_surface_update *upda ...@@ -1444,6 +1444,16 @@ void dc_update_surfaces_for_target(struct dc *dc, struct dc_surface_update *upda
pipe_ctx->scl_data.recout.width -= 2; pipe_ctx->scl_data.recout.width -= 2;
} }
} }
if (updates[i].gamma) {
if (surface->public.gamma_correction != NULL)
dc_gamma_release(surface->public.
gamma_correction);
dc_gamma_retain(updates[i].gamma);
surface->public.gamma_correction =
updates[i].gamma;
}
} }
} }
......
...@@ -64,7 +64,8 @@ static bool construct(struct dc_context *ctx, struct surface *surface) ...@@ -64,7 +64,8 @@ static bool construct(struct dc_context *ctx, struct surface *surface)
static void destruct(struct surface *surface) static void destruct(struct surface *surface)
{ {
if (surface->protected.public.gamma_correction != NULL)
dc_gamma_release(surface->protected.public.gamma_correction);
} }
/******************************************************************************* /*******************************************************************************
......
...@@ -312,6 +312,7 @@ const struct dc_surface_status *dc_surface_get_status( ...@@ -312,6 +312,7 @@ const struct dc_surface_status *dc_surface_get_status(
void dc_surface_retain(const struct dc_surface *dc_surface); void dc_surface_retain(const struct dc_surface *dc_surface);
void dc_surface_release(const struct dc_surface *dc_surface); void dc_surface_release(const struct dc_surface *dc_surface);
void dc_gamma_retain(const struct dc_gamma *dc_gamma);
void dc_gamma_release(const struct dc_gamma *dc_gamma); void dc_gamma_release(const struct dc_gamma *dc_gamma);
struct dc_gamma *dc_create_gamma(void); struct dc_gamma *dc_create_gamma(void);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment