aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorTrent Piepho <tpiepho@impinj.com>2019-05-22 18:43:24 +0000
committerDavid S. Miller <davem@davemloft.net>2019-05-22 17:40:18 -0700
commit1b9b295435c38bfb3936ad6033595ead4aa51c0e (patch)
treed96f2cfca3ebeba96ef012bb591906da78c5d25c /drivers
parentnet: phy: dp83867: Rework delay rgmii delay handling (diff)
downloadlinux-dev-1b9b295435c38bfb3936ad6033595ead4aa51c0e.tar.xz
linux-dev-1b9b295435c38bfb3936ad6033595ead4aa51c0e.zip
net: phy: dp83867: Use unsigned variables to store unsigned properties
The variables used to store u32 DT properties were signed ints. This doesn't work properly if the value of the property were to overflow. Use unsigned variables so this doesn't happen. Cc: Andrew Lunn <andrew@lunn.ch> Cc: Florian Fainelli <f.fainelli@gmail.com> Cc: Heiner Kallweit <hkallweit1@gmail.com> Signed-off-by: Trent Piepho <tpiepho@impinj.com> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/net/phy/dp83867.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/net/phy/dp83867.c b/drivers/net/phy/dp83867.c
index fc5baa5d14d0..59051b0f5be9 100644
--- a/drivers/net/phy/dp83867.c
+++ b/drivers/net/phy/dp83867.c
@@ -93,9 +93,9 @@ enum {
};
struct dp83867_private {
- int rx_id_delay;
- int tx_id_delay;
- int fifo_depth;
+ u32 rx_id_delay;
+ u32 tx_id_delay;
+ u32 fifo_depth;
int io_impedance;
int port_mirroring;
bool rxctrl_strap_quirk;