diff options
| author |   Vivien Didelot <vivien.didelot@savoirfairelinux.com> | 2016-04-17 13:23:57 -0400 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2016-04-17 18:54:15 -0400 |
| commit | 54c6f4bda7dabab7a7a9eeb8f1ced4b0b5fc4fd0 (patch) | |
| tree | 9c85421f81756b9a289cd186bf107d6f51278b0c /drivers/net/dsa/mv88e6352.c | |
| parent | net: dsa: mv88e6xxx: drop double ds assignment (diff) | |
| download | linux-dev-54c6f4bda7dabab7a7a9eeb8f1ced4b0b5fc4fd0.tar.xz linux-dev-54c6f4bda7dabab7a7a9eeb8f1ced4b0b5fc4fd0.zip | |
net: dsa: mv88e6xxx: drop revision probing
There is no point in having a special case for the revision when probing
a switch model. The code gets cluttered with unnecessary defines, and
leads to errors when code such as mv88e6131_setup compares
PORT_SWITCH_ID_6131_B2 to ps->id which masks the revision.
Drop every revision definition, and lookup only the product number.
Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/dsa/mv88e6352.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/net/dsa/mv88e6352.c b/drivers/net/dsa/mv88e6352.c index 10c36abf4c64..34f92b17146b 100644 --- a/drivers/net/dsa/mv88e6352.c +++ b/drivers/net/dsa/mv88e6352.c @@ -27,14 +27,8 @@ static const struct mv88e6xxx_switch_id mv88e6352_table[] = { { PORT_SWITCH_ID_6176, "Marvell 88E6176" }, { PORT_SWITCH_ID_6240, "Marvell 88E6240" }, { PORT_SWITCH_ID_6320, "Marvell 88E6320" }, - { PORT_SWITCH_ID_6320_A1, "Marvell 88E6320 (A1)" }, - { PORT_SWITCH_ID_6320_A2, "Marvell 88e6320 (A2)" }, { PORT_SWITCH_ID_6321, "Marvell 88E6321" }, - { PORT_SWITCH_ID_6321_A1, "Marvell 88E6321 (A1)" }, - { PORT_SWITCH_ID_6321_A2, "Marvell 88e6321 (A2)" }, { PORT_SWITCH_ID_6352, "Marvell 88E6352" }, - { PORT_SWITCH_ID_6352_A0, "Marvell 88E6352 (A0)" }, - { PORT_SWITCH_ID_6352_A1, "Marvell 88E6352 (A1)" }, }; static const char *mv88e6352_drv_probe(struct device *dsa_dev, |