net: dsa: mv88e6xxx: get serdes lane after lock
authorVivien Didelot <vivien.didelot@gmail.com>
Wed, 28 Aug 2019 16:26:11 +0000 (12:26 -0400)
committerDavid S. Miller <davem@davemloft.net>
Fri, 30 Aug 2019 00:20:04 +0000 (17:20 -0700)
commitb98f0f530d7978157b553a97556c5cc6489b3108
tree8933dae4ba76e30e1090249e2e6f6e3404fd54c9
parent73a1dd8c908eee5bd34800d355759adb493f67ac
net: dsa: mv88e6xxx: get serdes lane after lock

This is a follow-up patch for commit 17deaf5cb37a ("net: dsa:
mv88e6xxx: create serdes_get_lane chip operation").

The .serdes_get_lane implementations access the CMODE of a port,
even though it is cached at the moment, it is safer to call them
after the mutex is locked, not before.

At the same time, check for an eventual error and return IRQ_DONE,
instead of blindly ignoring it.

Signed-off-by: Vivien Didelot <vivien.didelot@gmail.com>
Reviewed-by: Marek BehĂșn <marek.behun@nic.cz>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/mv88e6xxx/serdes.c