On 2019-12-19 6:55 a.m., Chen Zhou wrote: > Fix sparse warning: > > drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c:964:5: > warning: symbol 'shift_border_left_to_dst' was not declared. Should it be static? > drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c:975:6: > warning: symbol 'restore_border_left_from_dst' was not declared. Should it be static? > > Fixes: 89d07b662f5e (drm/amd/display: fix 270 degree rotation for mixed-SLS mode) > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Signed-off-by: Chen Zhou <chenzhou10@xxxxxxxxxx> Reviewed-by: Harry Wentland <harry.wentland@xxxxxxx> Harry > --- > drivers/gpu/drm/amd/display/dc/core/dc_resource.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c > index 64a0e08f..5843c16 100644 > --- a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c > +++ b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c > @@ -961,7 +961,7 @@ static void calculate_inits_and_adj_vp(struct pipe_ctx *pipe_ctx) > * We also need to make sure pipe_ctx->plane_res.scl_data.h_active uses the > * original h_border_left value in its calculation. > */ > -int shift_border_left_to_dst(struct pipe_ctx *pipe_ctx) > +static int shift_border_left_to_dst(struct pipe_ctx *pipe_ctx) > { > int store_h_border_left = pipe_ctx->stream->timing.h_border_left; > > @@ -972,7 +972,7 @@ int shift_border_left_to_dst(struct pipe_ctx *pipe_ctx) > return store_h_border_left; > } > > -void restore_border_left_from_dst(struct pipe_ctx *pipe_ctx, > +static void restore_border_left_from_dst(struct pipe_ctx *pipe_ctx, > int store_h_border_left) > { > pipe_ctx->stream->dst.x -= store_h_border_left; > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel