summaryrefslogtreecommitdiff
path: root/drivers/gpu/drm/amd/display/dc/irq/irq_service.c
diff options
context:
space:
mode:
authorAlex Hung <alex.hung@amd.com>2024-05-29 17:50:44 -0600
committerAlex Deucher <alexander.deucher@amd.com>2024-06-14 16:17:17 -0400
commit82c94233e5c8f7609e244267797e1e2183c2c4c0 (patch)
tree15c64cb5c26e7142102093c3f3d041b4de1c2c8e /drivers/gpu/drm/amd/display/dc/irq/irq_service.c
parent97dc6a480c8189d1f130fcd026391cef5f7d8b34 (diff)
downloadlinux-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/irq/irq_service.c')
-rw-r--r--drivers/gpu/drm/amd/display/dc/irq/irq_service.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/display/dc/irq/irq_service.c b/drivers/gpu/drm/amd/display/dc/irq/irq_service.c
index d100edaedbbb..eca3d7ee7e4e 100644
--- a/drivers/gpu/drm/amd/display/dc/irq/irq_service.c
+++ b/drivers/gpu/drm/amd/display/dc/irq/irq_service.c
@@ -76,7 +76,7 @@ static const struct irq_source_info *find_irq_source_info(
struct irq_service *irq_service,
enum dc_irq_source source)
{
- if (source >= DAL_IRQ_SOURCES_NUMBER || source < DC_IRQ_SOURCE_INVALID)
+ if (source >= DAL_IRQ_SOURCES_NUMBER)
return NULL;
return &irq_service->info[source];