can: bittiming: can_fixup_bittiming(): change type of tseg1 and alltseg to unsigned int
authorMarc Kleine-Budde <mkl@pengutronix.de>
Wed, 13 Oct 2021 13:00:10 +0000 (15:00 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 18 Nov 2021 18:16:19 +0000 (19:16 +0100)
[ Upstream commit e346290439609a8ac67122418ca2efbad8d0a7e7 ]

All timing calculation is done with unsigned integers, so change type
of tseg1 and alltseg to unsigned int, too.

Link: https://lore.kernel.org/all/20211013130653.1513627-1-mkl@pengutronix.de
Link: https://github.com/linux-can/can-utils/pull/314
Reported-by: Gary Bisson <bisson.gary@gmail.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/can/dev/bittiming.c

index f49170eadd547c443a7e8b087990725806b9d3ed..b1b5a82f0829919eb1337346ed31e57221506ead 100644 (file)
@@ -209,7 +209,7 @@ static int can_fixup_bittiming(struct net_device *dev, struct can_bittiming *bt,
                               const struct can_bittiming_const *btc)
 {
        struct can_priv *priv = netdev_priv(dev);
-       int tseg1, alltseg;
+       unsigned int tseg1, alltseg;
        u64 brp64;
 
        tseg1 = bt->prop_seg + bt->phase_seg1;