diff options
| author | Andrew Lunn <andrew@lunn.ch> | 2016-04-14 23:47:12 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-04-16 19:23:45 -0400 |
| commit | 48ace4ef4c3f99ebf6f801c9a8326a4a39f31dbf (patch) | |
| tree | a743037cc79c8ae42da8dcc63a5d19c2c7065e4c /drivers/net/dsa/mv88e6xxx.h | |
| parent | 756ca874417695f77941948a77e9b8562635cc0a (diff) | |
| download | linux-48ace4ef4c3f99ebf6f801c9a8326a4a39f31dbf.tar.bz2 | |
dsa: mv88e6xxx: Kill the REG_READ and REG_WRITE macros
These macros hide a ds variable and a return statement on error, which
can lead to locking issues. Kill them off.
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Tested-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/dsa/mv88e6xxx.h')
| -rw-r--r-- | drivers/net/dsa/mv88e6xxx.h | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/drivers/net/dsa/mv88e6xxx.h b/drivers/net/dsa/mv88e6xxx.h index 5d27decc85cb..0debb9f3cf0a 100644 --- a/drivers/net/dsa/mv88e6xxx.h +++ b/drivers/net/dsa/mv88e6xxx.h @@ -542,25 +542,4 @@ extern struct dsa_switch_driver mv88e6123_switch_driver; extern struct dsa_switch_driver mv88e6352_switch_driver; extern struct dsa_switch_driver mv88e6171_switch_driver; -#define REG_READ(addr, reg) \ - ({ \ - int __ret; \ - \ - __ret = mv88e6xxx_reg_read(ds, addr, reg); \ - if (__ret < 0) \ - return __ret; \ - __ret; \ - }) - -#define REG_WRITE(addr, reg, val) \ - ({ \ - int __ret; \ - \ - __ret = mv88e6xxx_reg_write(ds, addr, reg, val); \ - if (__ret < 0) \ - return __ret; \ - }) - - - #endif |