Make i2c list terminators explicitely unsigned
PATCH 1/1 Make i2c list terminators explicitely unsigned
Hi Greg,
Shouldn't the i2c list terminators be explicitely declared as unsigned? I'd hope it to help code analysis tools and possibly avoid false positives. Coverity's SWAT pointed my attention to these constants.
Thanks.
Signed-off-by Jean Delvare <[email protected]>
include/linux/i2c.h renamed from include/linux/i2c.h.orig +2 -2
@@ -302,8 +302,8 @@ };/* Internal numbers to terminate lists */ -#define I2C_CLIENT_END 0xfffe -#define I2C_CLIENT_ISA_END 0xfffefffe +#define I2C_CLIENT_END 0xfffeU +#define I2C_CLIENT_ISA_END 0xfffefffeU/* The numbers to use to set I2C bus address */ #define ANY_I2C_BUS 0xffff
Signature · 1 line
Jean Delvare
Greg KH
AppliedPATCH 1/1 Make i2c list terminators explicitely unsigned
Quoted 9 lines · Jean Delvare
Hi Greg,
Shouldn't the i2c list terminators be explicitely declared as unsigned? I'd hope it to help code analysis tools and possibly avoid false positives. Coverity's SWAT pointed my attention to these constants.
Thanks.
Signed-off-by Jean Delvare <[email protected]>
Applied, thanks.
greg k-h