OSDN Git Service

net: aquantia: Add const qualifiers for hardware ops tables
authorIgor Russkikh <igor.russkikh@aquantia.com>
Mon, 15 Jan 2018 13:41:17 +0000 (16:41 +0300)
committerDavid S. Miller <davem@davemloft.net>
Tue, 16 Jan 2018 19:40:00 +0000 (14:40 -0500)
Hardware operations and capabilities tables are constants and
never changed. Declare these as constants.

Signed-off-by: Igor Russkikh <igor.russkikh@aquantia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
14 files changed:
drivers/net/ethernet/aquantia/atlantic/aq_hw.h
drivers/net/ethernet/aquantia/atlantic/aq_main.c
drivers/net/ethernet/aquantia/atlantic/aq_nic.c
drivers/net/ethernet/aquantia/atlantic/aq_nic.h
drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c
drivers/net/ethernet/aquantia/atlantic/aq_pci_func.h
drivers/net/ethernet/aquantia/atlantic/aq_vec.c
drivers/net/ethernet/aquantia/atlantic/aq_vec.h
drivers/net/ethernet/aquantia/atlantic/hw_atl/hw_atl_a0.c
drivers/net/ethernet/aquantia/atlantic/hw_atl/hw_atl_a0.h
drivers/net/ethernet/aquantia/atlantic/hw_atl/hw_atl_b0.c
drivers/net/ethernet/aquantia/atlantic/hw_atl/hw_atl_b0.h
drivers/net/ethernet/aquantia/atlantic/hw_atl/hw_atl_utils.c
drivers/net/ethernet/aquantia/atlantic/hw_atl/hw_atl_utils.h

index 2f7ecd6..cdd1acd 100644 (file)
@@ -196,7 +196,8 @@ struct aq_hw_ops {
                               struct aq_rss_parameters *rss_params);
 
        int (*hw_get_regs)(struct aq_hw_s *self,
-                          struct aq_hw_caps_s *aq_hw_caps, u32 *regs_buff);
+                          const struct aq_hw_caps_s *aq_hw_caps,
+                          u32 *regs_buff);
 
        int (*hw_update_stats)(struct aq_hw_s *self);
 
index 5d6c40d..1b0399c 100644 (file)
@@ -35,9 +35,9 @@ MODULE_VERSION(AQ_CFG_DRV_VERSION);
 MODULE_AUTHOR(AQ_CFG_DRV_AUTHOR);
 MODULE_DESCRIPTION(AQ_CFG_DRV_DESC);
 
-static struct aq_hw_ops *aq_pci_probe_get_hw_ops_by_id(struct pci_dev *pdev)
+static const struct aq_hw_ops *aq_pci_probe_get_hw_ops_by_id(struct pci_dev *pdev)
 {
-       struct aq_hw_ops *ops = NULL;
+       const struct aq_hw_ops *ops = NULL;
 
        ops = hw_atl_a0_get_ops_by_id(pdev);
        if (!ops)
@@ -174,7 +174,7 @@ static const struct net_device_ops aq_ndev_ops = {
 static int aq_pci_probe(struct pci_dev *pdev,
                        const struct pci_device_id *pci_id)
 {
-       struct aq_hw_ops *aq_hw_ops = NULL;
+       const struct aq_hw_ops *aq_hw_ops = NULL;
        struct aq_pci_func_s *aq_pci_func = NULL;
        int err = 0;
 
index f934b09..f210e62 100644 (file)
@@ -295,7 +295,7 @@ err_exit:
 
 int aq_nic_ndev_init(struct aq_nic_s *self)
 {
-       struct aq_hw_caps_s *aq_hw_caps = self->aq_nic_cfg.aq_hw_caps;
+       const struct aq_hw_caps_s *aq_hw_caps = self->aq_nic_cfg.aq_hw_caps;
        struct aq_nic_cfg_s *aq_nic_cfg = &self->aq_nic_cfg;
 
        self->ndev->hw_features |= aq_hw_caps->hw_features;
index be3b4ce..715b53c 100644 (file)
@@ -34,7 +34,7 @@ struct aq_hw_ops;
 #define AQ_NIC_RATE_100M       BIT(5)
 
 struct aq_nic_cfg_s {
-       struct aq_hw_caps_s *aq_hw_caps;
+       const struct aq_hw_caps_s *aq_hw_caps;
        u64 hw_features;
        u32 rxds;               /* rx ring size, descriptors # */
        u32 txds;               /* tx ring size, descriptors # */
index 58c29d0..bc85809 100644 (file)
@@ -29,7 +29,7 @@ struct aq_pci_func_s {
        struct aq_hw_caps_s aq_hw_caps;
 };
 
-struct aq_pci_func_s *aq_pci_func_alloc(struct aq_hw_ops *aq_hw_ops,
+struct aq_pci_func_s *aq_pci_func_alloc(const struct aq_hw_ops *aq_hw_ops,
                                        struct pci_dev *pdev,
                                        const struct net_device_ops *ndev_ops,
                                        const struct ethtool_ops *eth_ops)
index a174d90..8446522 100644 (file)
@@ -15,7 +15,7 @@
 #include "aq_common.h"
 #include "aq_nic.h"
 
-struct aq_pci_func_s *aq_pci_func_alloc(struct aq_hw_ops *hw_ops,
+struct aq_pci_func_s *aq_pci_func_alloc(const struct aq_hw_ops *hw_ops,
                                        struct pci_dev *pdev,
                                        const struct net_device_ops *ndev_ops,
                                        const struct ethtool_ops *eth_ops);
index 5477524..f890b8a 100644 (file)
@@ -19,7 +19,7 @@
 #include <linux/netdevice.h>
 
 struct aq_vec_s {
-       struct aq_hw_ops *aq_hw_ops;
+       const struct aq_hw_ops *aq_hw_ops;
        struct aq_hw_s *aq_hw;
        struct aq_nic_s *aq_nic;
        unsigned int tx_rings;
@@ -165,7 +165,7 @@ err_exit:
        return self;
 }
 
-int aq_vec_init(struct aq_vec_s *self, struct aq_hw_ops *aq_hw_ops,
+int aq_vec_init(struct aq_vec_s *self, const struct aq_hw_ops *aq_hw_ops,
                struct aq_hw_s *aq_hw)
 {
        struct aq_ring_s *ring = NULL;
index 6c68b18..78cc6c3 100644 (file)
@@ -26,7 +26,7 @@ irqreturn_t aq_vec_isr(int irq, void *private);
 irqreturn_t aq_vec_isr_legacy(int irq, void *private);
 struct aq_vec_s *aq_vec_alloc(struct aq_nic_s *aq_nic, unsigned int idx,
                              struct aq_nic_cfg_s *aq_nic_cfg);
-int aq_vec_init(struct aq_vec_s *self, struct aq_hw_ops *aq_hw_ops,
+int aq_vec_init(struct aq_vec_s *self, const struct aq_hw_ops *aq_hw_ops,
                struct aq_hw_s *aq_hw);
 void aq_vec_deinit(struct aq_vec_s *self);
 void aq_vec_free(struct aq_vec_s *self);
index 5c616e4..5b1f3c1 100644 (file)
@@ -851,7 +851,7 @@ err_exit:
        return err;
 }
 
-static struct aq_hw_ops hw_atl_ops_ = {
+static const struct aq_hw_ops hw_atl_ops_ = {
        .create               = hw_atl_a0_create,
        .destroy              = hw_atl_a0_destroy,
        .get_hw_caps          = hw_atl_a0_get_hw_caps,
@@ -894,7 +894,7 @@ static struct aq_hw_ops hw_atl_ops_ = {
        .hw_get_fw_version           = hw_atl_utils_get_fw_version,
 };
 
-struct aq_hw_ops *hw_atl_a0_get_ops_by_id(struct pci_dev *pdev)
+const struct aq_hw_ops *hw_atl_a0_get_ops_by_id(struct pci_dev *pdev)
 {
        bool is_vid_ok = (pdev->vendor == PCI_VENDOR_ID_AQUANTIA);
        bool is_did_ok = ((pdev->device == HW_ATL_DEVICE_ID_0001) ||
index 6e1d527..4fdd51b 100644 (file)
@@ -29,6 +29,6 @@
 
 #endif
 
-struct aq_hw_ops *hw_atl_a0_get_ops_by_id(struct pci_dev *pdev);
+const struct aq_hw_ops *hw_atl_a0_get_ops_by_id(struct pci_dev *pdev);
 
 #endif /* HW_ATL_A0_H */
index e2240ab..1c47811 100644 (file)
@@ -928,7 +928,7 @@ err_exit:
        return err;
 }
 
-static struct aq_hw_ops hw_atl_ops_ = {
+static const struct aq_hw_ops hw_atl_ops_ = {
        .create               = hw_atl_b0_create,
        .destroy              = hw_atl_b0_destroy,
        .get_hw_caps          = hw_atl_b0_get_hw_caps,
@@ -971,7 +971,7 @@ static struct aq_hw_ops hw_atl_ops_ = {
        .hw_get_fw_version           = hw_atl_utils_get_fw_version,
 };
 
-struct aq_hw_ops *hw_atl_b0_get_ops_by_id(struct pci_dev *pdev)
+const struct aq_hw_ops *hw_atl_b0_get_ops_by_id(struct pci_dev *pdev)
 {
        bool is_vid_ok = (pdev->vendor == PCI_VENDOR_ID_AQUANTIA);
        bool is_did_ok = ((pdev->device == HW_ATL_DEVICE_ID_0001) ||
index a1e1bce..3e10969 100644 (file)
@@ -29,6 +29,6 @@
 
 #endif
 
-struct aq_hw_ops *hw_atl_b0_get_ops_by_id(struct pci_dev *pdev);
+const struct aq_hw_ops *hw_atl_b0_get_ops_by_id(struct pci_dev *pdev);
 
 #endif /* HW_ATL_B0_H */
index 407dc06..0abb7d7 100644 (file)
@@ -117,7 +117,7 @@ err_exit:
 }
 
 static int hw_atl_utils_init_ucp(struct aq_hw_s *self,
-                                struct aq_hw_caps_s *aq_hw_caps)
+                                const struct aq_hw_caps_s *aq_hw_caps)
 {
        int err = 0;
 
@@ -564,7 +564,7 @@ static const u32 hw_atl_utils_hw_mac_regs[] = {
 };
 
 int hw_atl_utils_hw_get_regs(struct aq_hw_s *self,
-                            struct aq_hw_caps_s *aq_hw_caps,
+                            const struct aq_hw_caps_s *aq_hw_caps,
                             u32 *regs_buff)
 {
        unsigned int i = 0U;
index 1530abd..40e2319 100644 (file)
@@ -207,7 +207,7 @@ int hw_atl_utils_get_mac_permanent(struct aq_hw_s *self,
 unsigned int hw_atl_utils_mbps_2_speed_index(unsigned int mbps);
 
 int hw_atl_utils_hw_get_regs(struct aq_hw_s *self,
-                            struct aq_hw_caps_s *aq_hw_caps,
+                            const struct aq_hw_caps_s *aq_hw_caps,
                             u32 *regs_buff);
 
 int hw_atl_utils_hw_set_power(struct aq_hw_s *self,