Applied. Thanks! Alex
On Sun, Apr 24, 2022 at 4:15 PM Liu, Zhan <zhan....@amd.com> wrote: > > [AMD Official Use Only - General] > > > -----Original Message----- > > From: Guo Zhengkui <guozheng...@vivo.com> > > Sent: 2022/April/24, Sunday 5:06 AM > > To: 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>; Liu, Charlene > > <charlene....@amd.com>; Lei, Jun <jun....@amd.com>; Guo Zhengkui > > <guozheng...@vivo.com>; Liu, Zhan <zhan....@amd.com>; José Expósito > > <jose.exposit...@gmail.com>; open list:AMD DISPLAY CORE <amd- > > g...@lists.freedesktop.org>; open list:DRM DRIVERS <dri- > > de...@lists.freedesktop.org>; open list <linux-ker...@vger.kernel.org> > > Cc: zhengkui_...@outlook.com > > Subject: [PATCH] drm/amd/display: fix if == else warning > > > > Fix the following coccicheck warning: > > > > drivers/gpu/drm/amd/display/dc/dcn201/dcn201_hwseq.c:98:8-10: > > WARNING: possible condition with no effect (if == else) > > > > Signed-off-by: Guo Zhengkui <guozheng...@vivo.com> > > Thanks a lot for fixing this warning. > > Reviewed-by: Zhan Liu <zhan....@amd.com> > > > --- > > drivers/gpu/drm/amd/display/dc/dcn201/dcn201_hwseq.c | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_hwseq.c > > b/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_hwseq.c > > index fe22530242d2..05b3fba9ccce 100644 > > --- a/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_hwseq.c > > +++ b/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_hwseq.c > > @@ -95,8 +95,6 @@ static void gpu_addr_to_uma(struct dce_hwseq *hwseq, > > } else if (hwseq->fb_offset.quad_part <= addr->quad_part && > > addr->quad_part <= hwseq->uma_top.quad_part) { > > is_in_uma = true; > > - } else if (addr->quad_part == 0) { > > - is_in_uma = false; > > } else { > > is_in_uma = false; > > } > > -- > > 2.20.1 >