[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200210150826.35200-1-yuehaibing@huawei.com>
Date: Mon, 10 Feb 2020 23:08:26 +0800
From: YueHaibing <yuehaibing@...wei.com>
To: <harry.wentland@....com>, <sunpeng.li@....com>,
<alexander.deucher@....com>, <christian.koenig@....com>,
<David1.Zhou@....com>, <airlied@...ux.ie>, <daniel@...ll.ch>,
<Bhawanpreet.Lakha@....com>, <Anthony.Koo@....com>,
<aric.cyr@....com>, <Dmytro.Laktyushkin@....com>,
<Eric.Yang2@....com>, <yongqiang.sun@....com>,
<martin.leung@....com>, <charlene.liu@....com>
CC: <amd-gfx@...ts.freedesktop.org>, <dri-devel@...ts.freedesktop.org>,
<linux-kernel@...r.kernel.org>, YueHaibing <yuehaibing@...wei.com>
Subject: [RFC PATCH -next] drm/amd/display: Remove set but not unused variable 'stream_status'
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@...wei.com>
---
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
Powered by blists - more mailing lists