diff options
| author | Alex Hung <alex.hung@amd.com> | 2024-05-29 17:50:44 -0600 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2024-06-14 16:17:17 -0400 |
| commit | 82c94233e5c8f7609e244267797e1e2183c2c4c0 (patch) | |
| tree | 15c64cb5c26e7142102093c3f3d041b4de1c2c8e /drivers/gpu/drm/amd/display/dc/gpio/hw_generic.c | |
| parent | 97dc6a480c8189d1f130fcd026391cef5f7d8b34 (diff) | |
| download | linux-82c94233e5c8f7609e244267797e1e2183c2c4c0.tar.gz linux-82c94233e5c8f7609e244267797e1e2183c2c4c0.tar.bz2 linux-82c94233e5c8f7609e244267797e1e2183c2c4c0.zip | |
drm/amd/display: Remove useless comparison of unsigned int vs. 0
[WHY & HOW]
The comparisons of unsigned int with 0 can have no meanings, i.e.
unsigned int >= 0 (always true) or unsigned int < 0 (always false), and
therefore they are removed.
This fixes 12 NO_EFFECT issues reported by Coverity.
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Acked-by: Zaeem Mohamed <zaeem.mohamed@amd.com>
Signed-off-by: Alex Hung <alex.hung@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/display/dc/gpio/hw_generic.c')
| -rw-r--r-- | drivers/gpu/drm/amd/display/dc/gpio/hw_generic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/display/dc/gpio/hw_generic.c b/drivers/gpu/drm/amd/display/dc/gpio/hw_generic.c index f9e847e6555d..6cd50232c432 100644 --- a/drivers/gpu/drm/amd/display/dc/gpio/hw_generic.c +++ b/drivers/gpu/drm/amd/display/dc/gpio/hw_generic.c @@ -106,7 +106,7 @@ void dal_hw_generic_init( enum gpio_id id, uint32_t en) { - if ((en < GPIO_DDC_LINE_MIN) || (en > GPIO_DDC_LINE_MAX)) { + if (en > GPIO_DDC_LINE_MAX) { ASSERT_CRITICAL(false); *hw_generic = NULL; } |