net: thunderx: move filter register related macro into proper place
authorVadim Lomovtsev <Vadim.Lomovtsev@cavium.com>
Fri, 30 Mar 2018 11:59:47 +0000 (04:59 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sun, 1 Apr 2018 02:18:16 +0000 (22:18 -0400)
The ThunderX NIC has set of registers which allows to configure
filter policy for ingress packets. There are three possible regimes
of filtering multicasts, broadcasts and unicasts: accept all, reject all
and accept filter allowed only.

Current implementation has enum with all of them and two generic macro
for enabling filtering et all (CAM_ACCEPT) and enabling/disabling
broadcast packets, which also should be corrected in order to represent
register bits properly. All these values are private for driver and
there is no need to â€˜publish’ them via header file.

This commit is to move filtering register manipulation values from
header file into source with explicit assignment of exact register
values to them to be used while register configuring.

Signed-off-by: Vadim Lomovtsev <Vadim.Lomovtsev@cavium.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/cavium/thunder/thunder_bgx.c
drivers/net/ethernet/cavium/thunder/thunder_bgx.h

index 91d34ea40e2c9e40507aee5a47748b61e94785f6..0dd211605eb1a935ccf2337e9092adac0d863fc4 100644 (file)
 #define DRV_NAME       "thunder_bgx"
 #define DRV_VERSION    "1.0"
 
+/* RX_DMAC_CTL configuration */
+enum MCAST_MODE {
+               MCAST_MODE_REJECT = 0x0,
+               MCAST_MODE_ACCEPT = 0x1,
+               MCAST_MODE_CAM_FILTER = 0x2,
+               RSVD = 0x3
+};
+
+#define BCAST_ACCEPT      BIT(0)
+#define CAM_ACCEPT        BIT(3)
+#define MCAST_MODE_MASK   0x3
+#define BGX_MCAST_MODE(x) (x << 1)
+
 struct lmac {
        struct bgx              *bgx;
        int                     dmac;
index 5a7567d3113815abc99392e0e9faa6cee9699940..52439da62c972c02444ea213aac3455e396989b4 100644 (file)
 #define LMAC_INTR_LINK_UP      BIT(0)
 #define LMAC_INTR_LINK_DOWN    BIT(1)
 
-/*  RX_DMAC_CTL configuration*/
-enum MCAST_MODE {
-               MCAST_MODE_REJECT,
-               MCAST_MODE_ACCEPT,
-               MCAST_MODE_CAM_FILTER,
-               RSVD
-};
-
-#define BCAST_ACCEPT   1
-#define CAM_ACCEPT     1
-
 void octeon_mdiobus_force_mod_depencency(void);
 void bgx_lmac_rx_tx_enable(int node, int bgx_idx, int lmacid, bool enable);
 void bgx_add_dmac_addr(u64 dmac, int node, int bgx_idx, int lmac);