Re: [PATCH] fbdev: mb862xxfb: Use int type to store negative error codes
From: Helge Deller
Date: Tue Sep 30 2025 - 14:43:42 EST
On 8/30/25 14:47, Qianfeng Rong wrote:
Change the 'ret' variable in of_platform_mb862xx_probe() from unsigned long
to int, as it needs to store either negative error codes or zero.
Storing the negative error codes in unsigned type, doesn't cause an issue
at runtime but can be confusing. Additionally, assigning negative error
codes to unsigned type may trigger a GCC warning when the -Wsign-conversion
flag is enabled.
No effect on runtime.
Signed-off-by: Qianfeng Rong <rongqianfeng@xxxxxxxx>
---
drivers/video/fbdev/mb862xx/mb862xxfbdrv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
applied.
Thanks!
Helge