[PATCH] video: rockchip: Set return code

Andrew Goodbody andrew.goodbody at linaro.org
Wed Oct 1 12:37:01 CEST 2025


In rk_edp_start_aux_transaction the check for "AUX CH command reply
received" returns ret as the error code but it has not been assigned a
value so this could lead to the calling code silently ignoring the
error. Instead return -EIO.

This issue was found by Smatch.

Signed-off-by: Andrew Goodbody <andrew.goodbody at linaro.org>
---
 drivers/video/rockchip/rk_edp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/rockchip/rk_edp.c b/drivers/video/rockchip/rk_edp.c
index 0ba68dbce3d126f8a3230435ffa0f780c1e08d61..7186abc9c6c93d05e774e61efa136bf98ff727df 100644
--- a/drivers/video/rockchip/rk_edp.c
+++ b/drivers/video/rockchip/rk_edp.c
@@ -216,7 +216,7 @@ static int rk_edp_start_aux_transaction(struct rk3288_edp *regs)
 	/* Is AUX CH command reply received? */
 	if (rk_edp_is_aux_reply(regs)) {
 		debug("AUX CH command reply failed!\n");
-		return ret;
+		return -EIO;
 	}
 
 	/* Clear interrupt source for AUX CH access error */

---
base-commit: b2d5e46c67002e46ac339ef549ea18cf5adfcd87
change-id: 20251001-video_rockchip-a480d9178ae6

Best regards,
-- 
Andrew Goodbody <andrew.goodbody at linaro.org>



More information about the U-Boot mailing list