iwlwifi: use direct calls for transport free
authorJohannes Berg <johannes.berg@intel.com>
Thu, 12 Apr 2012 13:24:30 +0000 (06:24 -0700)
committerWey-Yi Guy <wey-yi.w.guy@intel.com>
Mon, 23 Apr 2012 21:30:29 +0000 (14:30 -0700)
Since the transport allocates and frees itself in
the transport specific code, there's no need for
virtual functions for it. Remove the free method
and call the correct functions directly.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
drivers/net/wireless/iwlwifi/iwl-pci.c
drivers/net/wireless/iwlwifi/iwl-trans-pcie-int.h
drivers/net/wireless/iwlwifi/iwl-trans-pcie.c
drivers/net/wireless/iwlwifi/iwl-trans.h

index 49d2b1886f2fa3c1e3b8c5754f9dbbed523a88ca..e80e9aa80883921765ac6ef987a70c36d9861ddc 100644 (file)
@@ -285,7 +285,7 @@ static int iwl_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
        return 0;
 
 out_free_trans:
-       iwl_trans_free(iwl_trans);
+       iwl_trans_pcie_free(iwl_trans);
        pci_set_drvdata(pdev, NULL);
        return -EFAULT;
 }
@@ -296,7 +296,7 @@ static void __devexit iwl_pci_remove(struct pci_dev *pdev)
        struct iwl_trans_pcie *trans_pcie = IWL_TRANS_GET_PCIE_TRANS(trans);
 
        iwl_drv_stop(trans_pcie->drv);
-       iwl_trans_free(trans);
+       iwl_trans_pcie_free(trans);
 
        pci_set_drvdata(pdev, NULL);
 }
index 8a175e01f4832996d9823cdeeefb675a2ba500ea..6213c05a4b529c6ba0f4263b83d9050afcfdd614 100644 (file)
@@ -301,6 +301,11 @@ iwl_trans_pcie_get_trans(struct iwl_trans_pcie *trans_pcie)
                            trans_specific);
 }
 
+struct iwl_trans *iwl_trans_pcie_alloc(struct pci_dev *pdev,
+                                      const struct pci_device_id *ent,
+                                      const struct iwl_cfg *cfg);
+void iwl_trans_pcie_free(struct iwl_trans *trans);
+
 /*****************************************************
 * RX
 ******************************************************/
index b6522ca2d8798957b82892be29f79a5cb24052dc..3eab7d52f8ba8111a37dba167fd5836388e417cf 100644 (file)
@@ -1549,7 +1549,7 @@ static void iwl_trans_pcie_configure(struct iwl_trans *trans,
        trans_pcie->command_names = trans_cfg->command_names;
 }
 
-static void iwl_trans_pcie_free(struct iwl_trans *trans)
+void iwl_trans_pcie_free(struct iwl_trans *trans)
 {
        struct iwl_trans_pcie *trans_pcie =
                IWL_TRANS_GET_PCIE_TRANS(trans);
@@ -2044,7 +2044,7 @@ static int iwl_trans_pcie_dbgfs_register(struct iwl_trans *trans,
 
 #endif /*CONFIG_IWLWIFI_DEBUGFS */
 
-const struct iwl_trans_ops trans_ops_pcie = {
+static const struct iwl_trans_ops trans_ops_pcie = {
        .start_hw = iwl_trans_pcie_start_hw,
        .stop_hw = iwl_trans_pcie_stop_hw,
        .fw_alive = iwl_trans_pcie_fw_alive,
@@ -2061,8 +2061,6 @@ const struct iwl_trans_ops trans_ops_pcie = {
        .tx_agg_disable = iwl_trans_pcie_tx_agg_disable,
        .tx_agg_setup = iwl_trans_pcie_tx_agg_setup,
 
-       .free = iwl_trans_pcie_free,
-
        .dbgfs_register = iwl_trans_pcie_dbgfs_register,
 
        .wait_tx_queue_empty = iwl_trans_pcie_wait_tx_queue_empty,
index ba3c09d5472e983c2af8a79d38ef470bda95633b..7018d313a4e03e99a9fb59ba7b3eefb9573c3c1d 100644 (file)
@@ -354,10 +354,6 @@ struct iwl_trans;
  *     May sleep
  * @tx_agg_disable: de-configure a Tx queue to send AMPDUs
  *     Must be atomic
- * @free: release all the ressource for the transport layer itself such as
- *     irq, tasklet etc... From this point on, the device may not issue
- *     any interrupt (incl. RFKILL).
- *     May sleep
  * @wait_tx_queue_empty: wait until all tx queues are empty
  *     May sleep
  * @dbgfs_register: add the dbgfs files under this directory. Files will be
@@ -393,8 +389,6 @@ struct iwl_trans_ops {
                             int sta_id, int tid, int frame_limit, u16 ssn);
        void (*tx_agg_disable)(struct iwl_trans *trans, int queue);
 
-       void (*free)(struct iwl_trans *trans);
-
        int (*dbgfs_register)(struct iwl_trans *trans, struct dentry* dir);
        int (*wait_tx_queue_empty)(struct iwl_trans *trans);
 #ifdef CONFIG_PM_SLEEP
@@ -564,11 +558,6 @@ static inline void iwl_trans_tx_agg_setup(struct iwl_trans *trans, int queue,
                                 frame_limit, ssn);
 }
 
-static inline void iwl_trans_free(struct iwl_trans *trans)
-{
-       trans->ops->free(trans);
-}
-
 static inline int iwl_trans_wait_tx_queue_empty(struct iwl_trans *trans)
 {
        WARN_ONCE(trans->state != IWL_TRANS_FW_ALIVE,
@@ -616,19 +605,9 @@ static inline void iwl_trans_set_pmi(struct iwl_trans *trans, bool state)
 }
 
 /*****************************************************
-* Transport layers implementations + their allocation function
+* driver (transport) register/unregister functions
 ******************************************************/
-struct pci_dev;
-struct pci_device_id;
-extern const struct iwl_trans_ops trans_ops_pcie;
-struct iwl_trans *iwl_trans_pcie_alloc(struct pci_dev *pdev,
-                                      const struct pci_device_id *ent,
-                                      const struct iwl_cfg *cfg);
 int __must_check iwl_pci_register_driver(void);
 void iwl_pci_unregister_driver(void);
 
-extern const struct iwl_trans_ops trans_ops_idi;
-struct iwl_trans *iwl_trans_idi_alloc(void *pdev_void,
-                                     const void *ent_void,
-                                     const struct iwl_cfg *cfg);
 #endif /* __iwl_trans_h__ */