From 4d24120dcfa2969a51311319682df99d30c03ed0 Mon Sep 17 00:00:00 2001 From: Daniel DeGrasse Date: Tue, 3 Dec 2024 16:47:36 -0600 Subject: [PATCH] drivers: display: display_rm67162: fix unsigned compare against 0 "wlen" variable set to return value of mipi_dsi_transfer should be a ssize_t type, so that if a negative value is returned the error will be caught and returned. Fixes #81929 Signed-off-by: Daniel DeGrasse --- drivers/display/display_rm67162.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/display/display_rm67162.c b/drivers/display/display_rm67162.c index 0deebf03f3c..0170904b3a8 100644 --- a/drivers/display/display_rm67162.c +++ b/drivers/display/display_rm67162.c @@ -350,7 +350,7 @@ static int rm67162_write_fb(const struct device *dev, bool first_write, const uint8_t *src, uint32_t len) { const struct rm67162_config *config = dev->config; - uint32_t wlen = 0; + ssize_t wlen; struct mipi_dsi_msg msg = {0}; /* Note- we need to set custom flags on the DCS message, @@ -368,7 +368,7 @@ static int rm67162_write_fb(const struct device *dev, bool first_write, msg.tx_buf = src; wlen = mipi_dsi_transfer(config->mipi_dsi, config->channel, &msg); if (wlen < 0) { - return wlen; + return (int)wlen; } /* Advance source pointer and decrement remaining */ src += wlen;