summaryrefslogtreecommitdiff
path: root/drivers/net/sungem_phy.c
diff options
context:
space:
mode:
authorMarek Vasut <marex@denx.de>2019-10-16 15:33:24 +0200
committerDavid S. Miller <davem@davemloft.net>2019-10-17 15:38:46 -0400
commit013572a236ef53cbd1e315e0acf2ee632cc816d4 (patch)
tree6ac54794f75a795368c788cd325776e27b98a100 /drivers/net/sungem_phy.c
parent7f238ca984b9c6a89e2339c661440c775d617386 (diff)
net: dsa: microchip: Add shared regmap mutex
The KSZ driver uses one regmap per register width (8/16/32), each with it's own lock, but accessing the same set of registers. In theory, it is possible to create a race condition between these regmaps, although the underlying bus (SPI or I2C) locking should assure nothing bad will really happen and the accesses would be correct. To make the driver do the right thing, add one single shared mutex for all the regmaps used by the driver instead. This assures that even if some future hardware is on a bus which does not serialize the accesses the same way SPI or I2C does, nothing bad will happen. Note that the status_mutex was unused and only initied, hence it was renamed and repurposed as the regmap mutex. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Andrew Lunn <andrew@lunn.ch> Cc: David S. Miller <davem@davemloft.net> Cc: Florian Fainelli <f.fainelli@gmail.com> Cc: George McCollister <george.mccollister@gmail.com> Cc: Tristram Ha <Tristram.Ha@microchip.com> Cc: Woojung Huh <woojung.huh@microchip.com> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/sungem_phy.c')
0 files changed, 0 insertions, 0 deletions