The 'ret' variable usually is used to store returns from some functions, which return either zero on success or negative error codes on failure. 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. Change "ret" from u32/unsigned int to int type. No effect on runtime. Qianfeng Rong (3): pinctrl: armada-37xx: Use int type to store negative error codes pinctrl: ma35: Use int type to store negative error codes pinctrl: renesas: Use int type to store negative error codes drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 6 ++++-- drivers/pinctrl/nuvoton/pinctrl-ma35.c | 3 ++- drivers/pinctrl/renesas/pinctrl.c | 3 ++- 3 files changed, 8 insertions(+), 4 deletions(-) -- 2.34.1
