Messages in this thread | | | Date | Mon, 30 Oct 2023 13:39:15 -0400 | Subject | Re: [PATCH] drm/amd/display: remove redundant check | From | Aurabindo Pillai <> |
| |
On 2023-10-30 13:17, José Pekkarinen wrote: > This patch addresses the following warning spotted by > using coccinelle where the case checked does the same > than the else case. > > drivers/gpu/drm/amd/display/dc/dml/dcn32/display_mode_vba_util_32.c:4664:8-10: WARNING: possible condition with no effect (if == else) > > Fixes: 974ce181 ("drm/amd/display: Add check for PState change in DCN32") > > Cc: stable@vger.kernel.org > Signed-off-by: José Pekkarinen <jose.pekkarinen@foxhound.fi> > --- > .../drm/amd/display/dc/dml/dcn32/display_mode_vba_util_32.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/dml/dcn32/display_mode_vba_util_32.c b/drivers/gpu/drm/amd/display/dc/dml/dcn32/display_mode_vba_util_32.c > index ecea008f19d3..d940dfa5ae43 100644 > --- a/drivers/gpu/drm/amd/display/dc/dml/dcn32/display_mode_vba_util_32.c > +++ b/drivers/gpu/drm/amd/display/dc/dml/dcn32/display_mode_vba_util_32.c > @@ -4661,10 +4661,6 @@ void dml32_CalculateMinAndMaxPrefetchMode( > } else if (AllowForPStateChangeOrStutterInVBlankFinal == dm_prefetch_support_uclk_fclk_and_stutter) { > *MinPrefetchMode = 0; > *MaxPrefetchMode = 0; > - } else if (AllowForPStateChangeOrStutterInVBlankFinal == > - dm_prefetch_support_uclk_fclk_and_stutter_if_possible) { > - *MinPrefetchMode = 0; > - *MaxPrefetchMode = 3; > } else { > *MinPrefetchMode = 0; > *MaxPrefetchMode = 3;
Reviewed-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
| |