We already have the product number and revision stored in the info
structure and the switch private state.

It is not necessary to clutter the header file with shifted product
number for devices that we don't even support yet. Remove them.

Signed-off-by: Vivien Didelot <vivien.dide...@savoirfairelinux.com>
---
 drivers/net/dsa/mv88e6xxx.c |  1 -
 drivers/net/dsa/mv88e6xxx.h | 34 ----------------------------------
 2 files changed, 35 deletions(-)

diff --git a/drivers/net/dsa/mv88e6xxx.c b/drivers/net/dsa/mv88e6xxx.c
index d40ac4d..7ec87df 100644
--- a/drivers/net/dsa/mv88e6xxx.c
+++ b/drivers/net/dsa/mv88e6xxx.c
@@ -3027,7 +3027,6 @@ found:
        ps->bus = bus;
        ps->sw_addr = sw_addr;
        ps->info = info;
-       ps->id = id & 0xfff0;
        ps->rev = id & 0xf;
 
        dev_info(&ps->bus->dev, "found switch %s, revision %u\n",
diff --git a/drivers/net/dsa/mv88e6xxx.h b/drivers/net/dsa/mv88e6xxx.h
index 7e86cc6..fb81945 100644
--- a/drivers/net/dsa/mv88e6xxx.h
+++ b/drivers/net/dsa/mv88e6xxx.h
@@ -68,39 +68,6 @@
 #define PORT_PCS_CTRL_UNFORCED         0x03
 #define PORT_PAUSE_CTRL                0x02
 #define PORT_SWITCH_ID         0x03
-#define PORT_SWITCH_ID_PROD_NUM_MASK   0xfff0
-#define PORT_SWITCH_ID_REV_MASK                0x000f
-#define PORT_SWITCH_ID_6031    0x0310
-#define PORT_SWITCH_ID_6035    0x0350
-#define PORT_SWITCH_ID_6046    0x0480
-#define PORT_SWITCH_ID_6061    0x0610
-#define PORT_SWITCH_ID_6065    0x0650
-#define PORT_SWITCH_ID_6085    0x04a0
-#define PORT_SWITCH_ID_6092    0x0970
-#define PORT_SWITCH_ID_6095    0x0950
-#define PORT_SWITCH_ID_6096    0x0980
-#define PORT_SWITCH_ID_6097    0x0990
-#define PORT_SWITCH_ID_6108    0x1070
-#define PORT_SWITCH_ID_6121    0x1040
-#define PORT_SWITCH_ID_6122    0x1050
-#define PORT_SWITCH_ID_6123    0x1210
-#define PORT_SWITCH_ID_6131    0x1060
-#define PORT_SWITCH_ID_6152    0x1a40
-#define PORT_SWITCH_ID_6155    0x1a50
-#define PORT_SWITCH_ID_6161    0x1610
-#define PORT_SWITCH_ID_6165    0x1650
-#define PORT_SWITCH_ID_6171    0x1710
-#define PORT_SWITCH_ID_6172    0x1720
-#define PORT_SWITCH_ID_6175    0x1750
-#define PORT_SWITCH_ID_6176    0x1760
-#define PORT_SWITCH_ID_6182    0x1a60
-#define PORT_SWITCH_ID_6185    0x1a70
-#define PORT_SWITCH_ID_6240    0x2400
-#define PORT_SWITCH_ID_6320    0x1150
-#define PORT_SWITCH_ID_6321    0x3100
-#define PORT_SWITCH_ID_6350    0x3710
-#define PORT_SWITCH_ID_6351    0x3750
-#define PORT_SWITCH_ID_6352    0x3520
 #define PORT_CONTROL           0x04
 #define PORT_CONTROL_USE_CORE_TAG      BIT(15)
 #define PORT_CONTROL_DROP_ON_LOCK      BIT(14)
@@ -450,7 +417,6 @@ struct mv88e6xxx_priv_state {
         */
        struct mutex eeprom_mutex;
 
-       int             id; /* switch product id */
        struct mv88e6xxx_priv_port      ports[DSA_MAX_PORTS];
 
        DECLARE_BITMAP(port_state_update_mask, DSA_MAX_PORTS);
-- 
2.8.0

Reply via email to