[AMD Official Use Only - General] > -----Original Message----- > From: Mahfooz, Hamza <hamza.mahf...@amd.com> > Sent: Wednesday, August 31, 2022 1:58 PM > To: linux-ker...@vger.kernel.org > Cc: Mahfooz, Hamza <hamza.mahf...@amd.com>; Wentland, Harry > <harry.wentl...@amd.com>; Li, Sun peng (Leo) <sunpeng...@amd.com>; > Siqueira, Rodrigo <rodrigo.sique...@amd.com>; Deucher, Alexander > <alexander.deuc...@amd.com>; Koenig, Christian > <christian.koe...@amd.com>; Pan, Xinhui <xinhui....@amd.com>; David > Airlie <airl...@linux.ie>; Daniel Vetter <dan...@ffwll.ch>; Kazlauskas, > Nicholas <nicholas.kazlaus...@amd.com>; Lei, Jun <jun....@amd.com>; > Somasundaram, Meenakshikumar > <meenakshikumar.somasunda...@amd.com>; Lee, Alvin > <alvin.l...@amd.com>; Leung, Martin <martin.le...@amd.com>; Tam, > Samson <samson....@amd.com>; Hung, Alex <alex.h...@amd.com>; > Aberback, Joshua <joshua.aberb...@amd.com>; Liu, Wenjing > <wenjing....@amd.com>; amd-...@lists.freedesktop.org; dri- > de...@lists.freedesktop.org > Subject: [PATCH] drm/amd/display: fix indentation in > commit_planes_for_stream() > > Address the following warning: > drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc.c:3508:9: warning: this > ‘if’ clause does not guard... [-Wmisleading-indentation] > 3508 | if (update_type != UPDATE_TYPE_FAST) > | ^~ > drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc.c:3510:17: note: ...this > statement, but the latter is misleadingly indented as if it were guarded by > the > ‘if’ > 3510 | if (update_type != UPDATE_TYPE_FAST) > | ^~ > > Signed-off-by: Hamza Mahfooz <hamza.mahf...@amd.com>
Acked-by: Alex Deucher <alexander.deuc...@amd.com> > --- > drivers/gpu/drm/amd/display/dc/core/dc.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c > b/drivers/gpu/drm/amd/display/dc/core/dc.c > index b49237390cce..6c4948916e7e 100644 > --- a/drivers/gpu/drm/amd/display/dc/core/dc.c > +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c > @@ -3505,11 +3505,11 @@ static void commit_planes_for_stream(struct dc > *dc, > top_pipe_to_program- > >stream_res.tg); > } > > - if (update_type != UPDATE_TYPE_FAST) > + if (update_type != UPDATE_TYPE_FAST) { > dc->hwss.post_unlock_program_front_end(dc, context); > - if (update_type != UPDATE_TYPE_FAST) > - if (dc->hwss.commit_subvp_config) > - dc->hwss.commit_subvp_config(dc, > context); > + if (dc->hwss.commit_subvp_config) > + dc->hwss.commit_subvp_config(dc, context); > + } > > /* Since phantom pipe programming is moved to > post_unlock_program_front_end, > * move the SubVP lock to after the phantom pipes have been setup > -- > 2.37.2