diff options
author | Michael McConville <mmcco@mykolab.com> | 2016-02-05 20:46:25 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-02-13 06:10:21 -0500 |
commit | db92ea5d4df00271b57d79c2d03dae5a5d60fcc1 (patch) | |
tree | 2406f11c0b4156ef084d40709f339f633a1d6d07 /drivers/net/wan | |
parent | 66d9cd0f5422bf35e3897b2022858dafd8c6abd2 (diff) |
dscc4: Undefined signed int shift
My analysis in the below mail applies, although the second part is
unnecessary because i isn't used in arithmetic operations here:
https://marc.info/?l=openbsd-tech&m=145377854103866&w=2
Thanks for your time.
Signed-off-by: Michael McConville <mmcco@mykolab.com>
Acked-by: Francois Romieu <romieu@fr.zoreil.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wan')
-rw-r--r-- | drivers/net/wan/dscc4.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wan/dscc4.c b/drivers/net/wan/dscc4.c index 7a72407208b1..629225980463 100644 --- a/drivers/net/wan/dscc4.c +++ b/drivers/net/wan/dscc4.c | |||
@@ -1626,7 +1626,7 @@ try: | |||
1626 | if (state & Xpr) { | 1626 | if (state & Xpr) { |
1627 | void __iomem *scc_addr; | 1627 | void __iomem *scc_addr; |
1628 | unsigned long ring; | 1628 | unsigned long ring; |
1629 | int i; | 1629 | unsigned int i; |
1630 | 1630 | ||
1631 | /* | 1631 | /* |
1632 | * - the busy condition happens (sometimes); | 1632 | * - the busy condition happens (sometimes); |