bna: get rid of mac_t
authorIvan Vecera <ivecera@redhat.com>
Thu, 11 Jun 2015 13:52:14 +0000 (15:52 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 11 Jun 2015 22:57:14 +0000 (15:57 -0700)
The patch converts mac_t type to widely used 'u8 [ETH_ALEN]'.

Signed-off-by: Ivan Vecera <ivecera@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
12 files changed:
drivers/net/ethernet/brocade/bna/bfa_defs.h
drivers/net/ethernet/brocade/bna/bfa_defs_cna.h
drivers/net/ethernet/brocade/bna/bfa_ioc.c
drivers/net/ethernet/brocade/bna/bfa_ioc.h
drivers/net/ethernet/brocade/bna/bfi.h
drivers/net/ethernet/brocade/bna/bfi_enet.h
drivers/net/ethernet/brocade/bna/bna.h
drivers/net/ethernet/brocade/bna/bna_enet.c
drivers/net/ethernet/brocade/bna/bna_tx_rx.c
drivers/net/ethernet/brocade/bna/bnad.c
drivers/net/ethernet/brocade/bna/bnad.h
drivers/net/ethernet/brocade/bna/cna.h

index 3bfd9da92630edb450ad586798f36a76bb48335b..f55887b5121b572d6610fe5dffc687314a90298f 100644 (file)
@@ -55,7 +55,7 @@ struct bfa_adapter_attr {
        char            optrom_ver[BFA_VERSION_LEN];
        char            os_type[BFA_ADAPTER_OS_TYPE_LEN];
        struct bfa_mfg_vpd vpd;
-       struct mac mac;
+       u8              mac[ETH_ALEN];
 
        u8              nports;
        u8              max_speed;
@@ -211,7 +211,7 @@ struct bfa_mfg_block {
        char    supplier_partnum[STRSZ(BFA_MFG_SUPPLIER_PARTNUM_SIZE)];
        char    supplier_serialnum[STRSZ(BFA_MFG_SUPPLIER_SERIALNUM_SIZE)];
        char    supplier_revision[STRSZ(BFA_MFG_SUPPLIER_REVISION_SIZE)];
-       mac_t   mfg_mac;        /* base mac address */
+       u8      mfg_mac[ETH_ALEN]; /* base mac address */
        u8      num_mac;        /* number of mac addresses */
        u8      rsv2;
        u32     card_type;      /* card type          */
index a37326d44fbb3c9fc41852e6d177fad2ec85f755..ce0b2283061a54f4baf0a3640d9654e37601ce56 100644 (file)
@@ -188,7 +188,7 @@ struct bfa_cee_attr {
        u8 error_reason;
        struct bfa_cee_lldp_cfg lldp_remote;
        struct bfa_cee_dcbx_cfg dcbx_remote;
-       mac_t src_mac;
+       u8 src_mac[ETH_ALEN];
        u8 link_speed;
        u8 nw_priority;
        u8 filler[2];
index 68f3c13c9ef6d992ac7eadde882c16b51375d6e8..82c95f810d283c976a170c2d5a2a4153f7e982fd 100644 (file)
@@ -2796,7 +2796,7 @@ bfa_ioc_get_adapter_attr(struct bfa_ioc *ioc,
                ad_attr->prototype = 0;
 
        ad_attr->pwwn = bfa_ioc_get_pwwn(ioc);
-       ad_attr->mac  = bfa_nw_ioc_get_mac(ioc);
+       bfa_nw_ioc_get_mac(ioc, ad_attr->mac);
 
        ad_attr->pcie_gen = ioc_attr->pcie_gen;
        ad_attr->pcie_lanes = ioc_attr->pcie_lanes;
@@ -2942,10 +2942,10 @@ bfa_ioc_get_pwwn(struct bfa_ioc *ioc)
        return ioc->attr->pwwn;
 }
 
-mac_t
-bfa_nw_ioc_get_mac(struct bfa_ioc *ioc)
+void
+bfa_nw_ioc_get_mac(struct bfa_ioc *ioc, u8 *mac)
 {
-       return ioc->attr->mac;
+       ether_addr_copy(mac, ioc->attr->mac);
 }
 
 /* Firmware failure detected. Start recovery actions. */
index effb7156e7a4799d606627340312e00b5bef9617..b37bc163b5544aa439f6353f7a4b526624e481ff 100644 (file)
@@ -309,7 +309,7 @@ void bfa_nw_ioc_fwver_get(struct bfa_ioc *ioc,
                        struct bfi_ioc_image_hdr *fwhdr);
 bool bfa_nw_ioc_fwver_cmp(struct bfa_ioc *ioc,
                        struct bfi_ioc_image_hdr *fwhdr);
-mac_t bfa_nw_ioc_get_mac(struct bfa_ioc *ioc);
+void bfa_nw_ioc_get_mac(struct bfa_ioc *ioc, u8 *mac);
 void bfa_nw_ioc_debug_memclaim(struct bfa_ioc *ioc, void *dbg_fwsave);
 int bfa_nw_ioc_debug_fwtrc(struct bfa_ioc *ioc, void *trcdata, int *trclen);
 int bfa_nw_ioc_debug_fwsave(struct bfa_ioc *ioc, void *trcdata, int *trclen);
index 2bcde4042268add6440deaca93a3255740e54a16..63645accb5bc3ed57d25f60e0687b7241f167c12 100644 (file)
@@ -189,14 +189,14 @@ struct bfi_ioc_getattr_req {
 struct bfi_ioc_attr {
        u64             mfg_pwwn;       /*!< Mfg port wwn          */
        u64             mfg_nwwn;       /*!< Mfg node wwn          */
-       mac_t           mfg_mac;        /*!< Mfg mac               */
+       u8              mfg_mac[ETH_ALEN]; /*!< Mfg mac            */
        u8              port_mode;      /* enum bfi_port_mode      */
        u8              rsvd_a;
        u64             pwwn;
        u64             nwwn;
-       mac_t           mac;            /*!< PBC or Mfg mac        */
+       u8              mac[ETH_ALEN];  /*!< PBC or Mfg mac        */
        u16     rsvd_b;
-       mac_t           fcoe_mac;
+       u8              fcoe_mac[ETH_ALEN];
        u16     rsvd_c;
        char            brcd_serialnum[STRSZ(BFA_MFG_SERIALNUM_SIZE)];
        u8              pcie_gen;
index bccca3bbadb85c4edacb2f6e439a35e4541d64d7..5b0c6c8db3d885e055f375e74cdd26a208a0cbb4 100644 (file)
@@ -573,7 +573,7 @@ struct bfi_enet_rss_cfg_req {
  */
 struct bfi_enet_ucast_req {
        struct bfi_msgq_mhdr    mh;
-       mac_t                   mac_addr;
+       u8                      mac_addr[ETH_ALEN];
        u8                      rsvd[2];
 };
 
@@ -581,7 +581,7 @@ struct bfi_enet_ucast_req {
 struct bfi_enet_mac_n_vlan_req {
        struct bfi_msgq_mhdr    mh;
        u16                     vlan_id;
-       mac_t                   mac_addr;
+       u8                      mac_addr[ETH_ALEN];
 };
 
 /* MAC Multicast
@@ -591,7 +591,7 @@ struct bfi_enet_mac_n_vlan_req {
  */
 struct bfi_enet_mcast_add_req {
        struct bfi_msgq_mhdr    mh;
-       mac_t                   mac_addr;
+       u8                      mac_addr[ETH_ALEN];
        u8                      rsvd[2];
 };
 
index 8ba72b1f36d9f249883b48f07727d4779f83399c..0cad6bdac46d723db2025260c0757606a19cedd3 100644 (file)
@@ -36,7 +36,7 @@ extern const u32 bna_napi_dim_vector[][BNA_BIAS_T_MAX];
 #define bna_is_small_rxq(_id) ((_id) & 0x1)
 
 #define BNA_MAC_IS_EQUAL(_mac1, _mac2)                                 \
-       (!memcmp((_mac1), (_mac2), sizeof(mac_t)))
+       (!memcmp((_mac1), (_mac2), ETH_ALEN))
 
 #define BNA_POWER_OF_2(x) (((x) & ((x) - 1)) == 0)
 
@@ -536,7 +536,7 @@ void bna_enet_pause_config(struct bna_enet *enet,
                           void (*cbfn)(struct bnad *));
 void bna_enet_mtu_set(struct bna_enet *enet, int mtu,
                      void (*cbfn)(struct bnad *));
-void bna_enet_perm_mac_get(struct bna_enet *enet, mac_t *mac);
+void bna_enet_perm_mac_get(struct bna_enet *enet, u8 *mac);
 
 /* IOCETH */
 
index deb8da6ab9cc4a4fcb434a8d8d819203b397454f..5a1e6b5f208e75953dff16d9b8271d2b41fc92ba 100644 (file)
@@ -1330,9 +1330,9 @@ bna_enet_mtu_set(struct bna_enet *enet, int mtu,
 }
 
 void
-bna_enet_perm_mac_get(struct bna_enet *enet, mac_t *mac)
+bna_enet_perm_mac_get(struct bna_enet *enet, u8 *mac)
 {
-       *mac = bfa_nw_ioc_get_mac(&enet->bna->ioceth.ioc);
+       bfa_nw_ioc_get_mac(&enet->bna->ioceth.ioc, mac);
 }
 
 /* IOCETH */
index 30d5e7ff699b2fb856008620b74fa1d54881e6f8..8943188f0b54dc3afbd21b89da5956b36625e514 100644 (file)
@@ -283,7 +283,7 @@ bna_bfi_ucast_req(struct bna_rxf *rxf, struct bna_mac *mac,
        bfi_msgq_mhdr_set(req->mh, BFI_MC_ENET, req_type, 0, rxf->rx->rid);
        req->mh.num_entries = htons(
        bfi_msgq_num_cmd_entries(sizeof(struct bfi_enet_ucast_req)));
-       memcpy(&req->mac_addr, &mac->addr, sizeof(mac_t));
+       ether_addr_copy(req->mac_addr, mac->addr);
        bfa_msgq_cmd_set(&rxf->msgq_cmd, NULL, NULL,
                sizeof(struct bfi_enet_ucast_req), &req->mh);
        bfa_msgq_cmd_post(&rxf->rx->bna->msgq, &rxf->msgq_cmd);
@@ -299,7 +299,7 @@ bna_bfi_mcast_add_req(struct bna_rxf *rxf, struct bna_mac *mac)
                0, rxf->rx->rid);
        req->mh.num_entries = htons(
        bfi_msgq_num_cmd_entries(sizeof(struct bfi_enet_mcast_add_req)));
-       memcpy(&req->mac_addr, &mac->addr, sizeof(mac_t));
+       ether_addr_copy(req->mac_addr, mac->addr);
        bfa_msgq_cmd_set(&rxf->msgq_cmd, NULL, NULL,
                sizeof(struct bfi_enet_mcast_add_req), &req->mh);
        bfa_msgq_cmd_post(&rxf->rx->bna->msgq, &rxf->msgq_cmd);
index 3e7c0921c7fd7729423c7dd866474aba80c23d3f..6e871a7b4569d00251e8e011258817b8dfce36d2 100644 (file)
@@ -876,9 +876,9 @@ bnad_set_netdev_perm_addr(struct bnad *bnad)
 {
        struct net_device *netdev = bnad->netdev;
 
-       ether_addr_copy(netdev->perm_addr, bnad->perm_addr.mac);
+       ether_addr_copy(netdev->perm_addr, bnad->perm_addr);
        if (is_zero_ether_addr(netdev->dev_addr))
-               ether_addr_copy(netdev->dev_addr, bnad->perm_addr.mac);
+               ether_addr_copy(netdev->dev_addr, bnad->perm_addr);
 }
 
 /* Control Path Handlers */
@@ -1095,8 +1095,8 @@ bnad_cb_tx_resume(struct bnad *bnad, struct bna_tx *tx)
         * get a 0 MAC address. We try to get the MAC address
         * again here.
         */
-       if (is_zero_ether_addr(&bnad->perm_addr.mac[0])) {
-               bna_enet_perm_mac_get(&bnad->bna.enet, &bnad->perm_addr);
+       if (is_zero_ether_addr(bnad->perm_addr)) {
+               bna_enet_perm_mac_get(&bnad->bna.enet, bnad->perm_addr);
                bnad_set_netdev_perm_addr(bnad);
        }
 }
@@ -3740,7 +3740,7 @@ bnad_pci_probe(struct pci_dev *pdev,
 
        /* Get the burnt-in mac */
        spin_lock_irqsave(&bnad->bna_lock, flags);
-       bna_enet_perm_mac_get(&bna->enet, &bnad->perm_addr);
+       bna_enet_perm_mac_get(&bna->enet, bnad->perm_addr);
        bnad_set_netdev_perm_addr(bnad);
        spin_unlock_irqrestore(&bnad->bna_lock, flags);
 
index 7ead6c23edb65ce2d9fa9351b02e3ced609bbe30..91fd82d9e9bc94968204c75f079c2bc4fa27cf56 100644 (file)
@@ -344,7 +344,7 @@ struct bnad {
        struct bnad_completion bnad_completions;
 
        /* Burnt in MAC address */
-       mac_t                   perm_addr;
+       u8                      perm_addr[ETH_ALEN];
 
        struct workqueue_struct *work_q;
 
index 28e7d0ffeab18e427e5bae07d41527fc7ce1648d..b42aafddae999f07d3af23969e2d177309241372 100644 (file)
@@ -42,12 +42,6 @@ extern char bfa_version[];
 #define CNA_FW_FILE_CT2        "ct2fw-3.2.5.1.bin"
 #define FC_SYMNAME_MAX 256     /*!< max name server symbolic name size */
 
-#pragma pack(1)
-
-typedef struct mac { u8 mac[ETH_ALEN]; } mac_t;
-
-#pragma pack()
-
 #define bfa_q_first(_q) ((void *)(((struct list_head *) (_q))->next))
 #define bfa_q_next(_qe)        (((struct list_head *) (_qe))->next)
 #define bfa_q_prev(_qe) (((struct list_head *) (_qe))->prev)