net: aquantia: Remove create/destroy from hw ops
authorIgor Russkikh <igor.russkikh@aquantia.com>
Fri, 19 Jan 2018 14:03:22 +0000 (17:03 +0300)
committerDavid S. Miller <davem@davemloft.net>
Sun, 21 Jan 2018 23:19:03 +0000 (18:19 -0500)
These ops are not related to HW and are now implemented in pci module.
Thus, remove these ops pointers and implementation.

Signed-off-by: Igor Russkikh <igor.russkikh@aquantia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/aquantia/atlantic/aq_hw.h
drivers/net/ethernet/aquantia/atlantic/hw_atl/hw_atl_a0.c
drivers/net/ethernet/aquantia/atlantic/hw_atl/hw_atl_b0.c

index 8219157..5566486 100644 (file)
@@ -101,7 +101,6 @@ struct aq_stats_s {
 struct aq_hw_s {
        atomic_t flags;
        struct aq_nic_cfg_s *aq_nic_cfg;
-       struct aq_pci_func_s *aq_pci_func;
        void __iomem *mmio;
        unsigned int not_ff_addr;
        struct aq_hw_link_status_s aq_link_status;
@@ -126,10 +125,6 @@ struct aq_nic_cfg_s;
 struct sk_buff;
 
 struct aq_hw_ops {
-       struct aq_hw_s *(*create)(struct aq_pci_func_s *aq_pci_func,
-                                 unsigned int port);
-
-       void (*destroy)(struct aq_hw_s *self);
 
        int (*hw_ring_tx_xmit)(struct aq_hw_s *self, struct aq_ring_s *aq_ring,
                               unsigned int frags);
index 491a901..9c9fb1e 100644 (file)
@@ -79,29 +79,6 @@ const struct aq_hw_caps_s hw_atl_a0_caps_aqc109 = {
                          HW_ATL_A0_RATE_100M,
 };
 
-
-static struct aq_hw_s *hw_atl_a0_create(struct aq_pci_func_s *aq_pci_func,
-                                       unsigned int port)
-{
-       struct aq_hw_s *self = NULL;
-
-       self = kzalloc(sizeof(*self), GFP_KERNEL);
-       if (!self)
-               goto err_exit;
-
-       self->aq_pci_func = aq_pci_func;
-
-       self->not_ff_addr = 0x10U;
-
-err_exit:
-       return self;
-}
-
-static void hw_atl_a0_destroy(struct aq_hw_s *self)
-{
-       kfree(self);
-}
-
 static int hw_atl_a0_hw_reset(struct aq_hw_s *self)
 {
        int err = 0;
@@ -906,8 +883,6 @@ err_exit:
 }
 
 const struct aq_hw_ops hw_atl_ops_a0 = {
-       .create               = hw_atl_a0_create,
-       .destroy              = hw_atl_a0_destroy,
        .hw_get_mac_permanent = hw_atl_utils_get_mac_permanent,
        .hw_set_mac_address   = hw_atl_a0_hw_mac_addr_set,
        .hw_get_link_status   = hw_atl_utils_mpi_get_link_status,
index 87e7d0f..00edad1 100644 (file)
@@ -82,28 +82,6 @@ const struct aq_hw_caps_s hw_atl_b0_caps_aqc109 = {
                          HW_ATL_B0_RATE_100M,
 };
 
-static struct aq_hw_s *hw_atl_b0_create(struct aq_pci_func_s *aq_pci_func,
-                                       unsigned int port)
-{
-       struct aq_hw_s *self = NULL;
-
-       self = kzalloc(sizeof(*self), GFP_KERNEL);
-       if (!self)
-               goto err_exit;
-
-       self->aq_pci_func = aq_pci_func;
-
-       self->not_ff_addr = 0x10U;
-
-err_exit:
-       return self;
-}
-
-static void hw_atl_b0_destroy(struct aq_hw_s *self)
-{
-       kfree(self);
-}
-
 static int hw_atl_b0_hw_reset(struct aq_hw_s *self)
 {
        int err = 0;
@@ -981,8 +959,6 @@ err_exit:
 }
 
 const struct aq_hw_ops hw_atl_ops_b0 = {
-       .create               = hw_atl_b0_create,
-       .destroy              = hw_atl_b0_destroy,
        .hw_get_mac_permanent = hw_atl_utils_get_mac_permanent,
        .hw_set_mac_address   = hw_atl_b0_hw_mac_addr_set,
        .hw_get_link_status   = hw_atl_utils_mpi_get_link_status,