i2c: pci1xxxx: prevent signed integer overflow
authorWolfram Sang <wsa@kernel.org>
Wed, 28 Sep 2022 20:12:14 +0000 (22:12 +0200)
committerWolfram Sang <wsa@kernel.org>
Sat, 1 Oct 2022 22:59:24 +0000 (00:59 +0200)
Some constants need 'UL' markings, otherwise they are shifted into the
sign bit.

Fixes: 361693697249 ("i2c: microchip: pci1xxxx: Add driver for I2C host controller in multifunction endpoint of pci1xxxx switch")
Signed-off-by: Wolfram Sang <wsa@kernel.org>
drivers/i2c/busses/i2c-mchp-pci1xxxx.c

index b2b7dbf..f534220 100644 (file)
  * Bus Idle Minimum time = BUS_IDLE_MIN[7:0] x Baud_Clock_Period x
  * (BUS_IDLE_MIN_XK_TICKS[7] ? 4,1)
  */
-#define BUS_IDLE_MIN_100K_TICKS                167
-#define BUS_IDLE_MIN_400K_TICKS                139
-#define BUS_IDLE_MIN_1000K_TICKS               133
+#define BUS_IDLE_MIN_100K_TICKS                167UL
+#define BUS_IDLE_MIN_400K_TICKS                139UL
+#define BUS_IDLE_MIN_1000K_TICKS               133UL
 
 /*
  * CTRL_CUM_TIME_OUT_XK_TICKS defines SMBus Controller Cumulative Time-Out.