From 4ddda80541a292c9dde742b5d6ee7b7400cc3d84 Mon Sep 17 00:00:00 2001 From: Yu Qiaowei Date: Thu, 9 Nov 2023 10:15:05 +0800 Subject: [PATCH] video: rockchip: rga3: fix the error report of Coverity 1. Fix the issue of scheduler and hardware maximum count mismatch. 2. Fix the issue of potential GCD returning 0. Change-Id: I083553e69ab60549b26465b5d1437e817575856f Signed-off-by: Yu Qiaowei --- drivers/video/rockchip/rga3/include/rga_drv.h | 2 +- drivers/video/rockchip/rga3/rga_policy.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/video/rockchip/rga3/include/rga_drv.h b/drivers/video/rockchip/rga3/include/rga_drv.h index fff02e176ce7..0efa7ef9f210 100644 --- a/drivers/video/rockchip/rga3/include/rga_drv.h +++ b/drivers/video/rockchip/rga3/include/rga_drv.h @@ -97,7 +97,7 @@ #define RGA_JOB_TIMEOUT_DELAY HZ #define RGA_RESET_TIMEOUT 1000 -#define RGA_MAX_SCHEDULER 3 +#define RGA_MAX_SCHEDULER RGA_HW_SIZE #define RGA_MAX_BUS_CLK 10 #define RGA_BUFFER_POOL_MAX_SIZE 64 diff --git a/drivers/video/rockchip/rga3/rga_policy.c b/drivers/video/rockchip/rga3/rga_policy.c index 1afab8993fa5..c00a67066fb0 100644 --- a/drivers/video/rockchip/rga3/rga_policy.c +++ b/drivers/video/rockchip/rga3/rga_policy.c @@ -15,7 +15,7 @@ #define GET_GCD(n1, n2) \ ({ \ int i; \ - int gcd = 0; \ + int gcd = 1; \ for (i = 1; i <= (n1) && i <= (n2); i++) { \ if ((n1) % i == 0 && (n2) % i == 0) \ gcd = i; \