[openwrt/openwrt] mediatek: cast literal value to char

LEDE Commits lede-commits at lists.infradead.org
Sun Sep 18 19:13:29 PDT 2022


dangole pushed a commit to openwrt/openwrt.git, branch openwrt-22.03:
https://git.openwrt.org/cd0283788fdb73a832f8476f94a3f357bcb89e65

commit cd0283788fdb73a832f8476f94a3f357bcb89e65
Author: Sven Wegener <sven.wegener at stealer.net>
AuthorDate: Sat Jun 12 21:56:21 2021 +0200

    mediatek: cast literal value to char
    
    Or the comparison against a signed char is always true, because the
    literal 0xaa is treated as an unsigned int, to which the signed char is
    casted during comparison. 0xaa is above the positive values of a signed
    char and negative signed char values result in values larger than 0xaa
    when casted to unsigned int.
    
    Signed-off-by: Sven Wegener <sven.wegener at stealer.net>
    (cherry picked from commit e9a22ce2535c4a0faf83d5595304fe0e2c7bd1b2)
---
 target/linux/mediatek/files-5.10/drivers/leds/leds-ubnt-ledbar.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target/linux/mediatek/files-5.10/drivers/leds/leds-ubnt-ledbar.c b/target/linux/mediatek/files-5.10/drivers/leds/leds-ubnt-ledbar.c
index 9bfe0e62f4..8168391d7e 100644
--- a/target/linux/mediatek/files-5.10/drivers/leds/leds-ubnt-ledbar.c
+++ b/target/linux/mediatek/files-5.10/drivers/leds/leds-ubnt-ledbar.c
@@ -26,7 +26,7 @@
 #define UBNT_LEDBAR_MAX_BRIGHTNESS	0xff
 
 #define UBNT_LEDBAR_TRANSACTION_LENGTH	8
-#define UBNT_LEDBAR_TRANSACTION_SUCCESS	0xaa
+#define UBNT_LEDBAR_TRANSACTION_SUCCESS	(char) 0xaa
 
 #define UBNT_LEDBAR_TRANSACTION_BLUE_IDX	2
 #define UBNT_LEDBAR_TRANSACTION_GREEN_IDX	3




More information about the lede-commits mailing list