| Message ID | 20200210150826.35200-1-yuehaibing@huawei.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [RFC,-next] drm/amd/display: Remove set but not unused variable 'stream_status' | expand |
On Mon, Feb 10, 2020 at 10:38 AM YueHaibing <yuehaibing@huawei.com> wrote: > > drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c: > In function dcn10_post_unlock_program_front_end: > drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2623:29: > warning: variable stream_status set but not used [-Wunused-but-set-variable] > > commit bbf5f6c3f83b ("drm/amd/display: Split program front end part that occur outside lock") > involved this unused variable. > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> Applied. Thanks! Alex > --- > drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c | 8 +------- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c > index 42fcfee..b2ed0fa 100644 > --- a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c > +++ b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c > @@ -2610,7 +2610,7 @@ void dcn10_post_unlock_program_front_end( > struct dc *dc, > struct dc_state *context) > { > - int i, j; > + int i; > > DC_LOGGER_INIT(dc->ctx->logger); > > @@ -2620,14 +2620,8 @@ void dcn10_post_unlock_program_front_end( > if (!pipe_ctx->top_pipe && > !pipe_ctx->prev_odm_pipe && > pipe_ctx->stream) { > - struct dc_stream_status *stream_status = NULL; > struct timing_generator *tg = pipe_ctx->stream_res.tg; > > - for (j = 0; j < context->stream_count; j++) { > - if (pipe_ctx->stream == context->streams[j]) > - stream_status = &context->stream_status[j]; > - } > - > if (context->stream_status[i].plane_count == 0) > false_optc_underflow_wa(dc, pipe_ctx->stream, tg); > } > -- > 2.7.4 > > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c index 42fcfee..b2ed0fa 100644 --- a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c +++ b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c @@ -2610,7 +2610,7 @@ void dcn10_post_unlock_program_front_end( struct dc *dc, struct dc_state *context) { - int i, j; + int i; DC_LOGGER_INIT(dc->ctx->logger); @@ -2620,14 +2620,8 @@ void dcn10_post_unlock_program_front_end( if (!pipe_ctx->top_pipe && !pipe_ctx->prev_odm_pipe && pipe_ctx->stream) { - struct dc_stream_status *stream_status = NULL; struct timing_generator *tg = pipe_ctx->stream_res.tg; - for (j = 0; j < context->stream_count; j++) { - if (pipe_ctx->stream == context->streams[j]) - stream_status = &context->stream_status[j]; - } - if (context->stream_status[i].plane_count == 0) false_optc_underflow_wa(dc, pipe_ctx->stream, tg); }
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c: In function dcn10_post_unlock_program_front_end: drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2623:29: warning: variable stream_status set but not used [-Wunused-but-set-variable] commit bbf5f6c3f83b ("drm/amd/display: Split program front end part that occur outside lock") involved this unused variable. Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-)