]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
iwlwifi: remove the code under IWLWIFI_PCIE_RTPM
authorEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Thu, 6 Jun 2019 08:22:16 +0000 (11:22 +0300)
committerLuca Coelho <luciano.coelho@intel.com>
Fri, 6 Sep 2019 12:31:15 +0000 (15:31 +0300)
This flag should never be set unless integration work with the
platform is done.  We don't support any platforms officially and don't
plan to do so in the near future, so we can remove this option
entirely in order to avoid having it enabled by mistake.

This has been marked with "depends on EXPERT", so there shouldn't be
many systems running with it set.  And, if there are systems, they
should not be using this flag.

Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
drivers/net/wireless/intel/iwlwifi/Kconfig
drivers/net/wireless/intel/iwlwifi/pcie/drv.c
drivers/net/wireless/intel/iwlwifi/pcie/trans.c

index 235349a33a3c530eeebada1ebc4d8fe19251ebab..7dbc0d38bb3bb5db765fc7019c27cb7ea5e84e76 100644 (file)
@@ -92,20 +92,6 @@ config IWLWIFI_BCAST_FILTERING
          If unsure, don't enable this option, as some programs might
          expect incoming broadcasts for their normal operations.
 
-config IWLWIFI_PCIE_RTPM
-       bool "Enable runtime power management mode for PCIe devices"
-       depends on IWLMVM && PM && EXPERT
-       help
-         Say Y here to enable runtime power management for PCIe
-         devices.  If enabled, the device will go into low power mode
-         when idle for a short period of time, allowing for improved
-         power saving during runtime. Note that this feature requires
-         a tight integration with the platform. It is not recommended
-         to enable this feature without proper validation with the
-         specific target platform.
-
-        If unsure, say N.
-
 menu "Debugging Options"
 
 config IWLWIFI_DEBUG
index 63a04cec32afc21d07d797aaa5d7f8b75a9170d2..a037dfedb56ee495d221a118d4b88730ff278c85 100644 (file)
@@ -1269,80 +1269,9 @@ int iwl_pci_fw_exit_d0i3(struct iwl_trans *trans)
        return ret;
 }
 
-#ifdef CONFIG_IWLWIFI_PCIE_RTPM
-static int iwl_pci_runtime_suspend(struct device *device)
-{
-       struct pci_dev *pdev = to_pci_dev(device);
-       struct iwl_trans *trans = pci_get_drvdata(pdev);
-       int ret;
-
-       IWL_DEBUG_RPM(trans, "entering runtime suspend\n");
-
-       if (test_bit(STATUS_DEVICE_ENABLED, &trans->status)) {
-               ret = iwl_pci_fw_enter_d0i3(trans);
-               if (ret < 0)
-                       return ret;
-       }
-
-       trans->system_pm_mode = IWL_PLAT_PM_MODE_D0I3;
-
-       iwl_trans_d3_suspend(trans, false, false);
-
-       return 0;
-}
-
-static int iwl_pci_runtime_resume(struct device *device)
-{
-       struct pci_dev *pdev = to_pci_dev(device);
-       struct iwl_trans *trans = pci_get_drvdata(pdev);
-       enum iwl_d3_status d3_status;
-
-       IWL_DEBUG_RPM(trans, "exiting runtime suspend (resume)\n");
-
-       iwl_trans_d3_resume(trans, &d3_status, false, false);
-
-       if (test_bit(STATUS_DEVICE_ENABLED, &trans->status))
-               return iwl_pci_fw_exit_d0i3(trans);
-
-       return 0;
-}
-
-static int iwl_pci_system_prepare(struct device *device)
-{
-       struct pci_dev *pdev = to_pci_dev(device);
-       struct iwl_trans *trans = pci_get_drvdata(pdev);
-
-       IWL_DEBUG_RPM(trans, "preparing for system suspend\n");
-
-       /* Wake the device up from runtime suspend before going to
-        * platform suspend.  This is needed because we don't know
-        * whether wowlan any is set and, if it's not, mac80211 will
-        * disconnect (in which case, we can't be in D0i3).
-        */
-       pm_runtime_resume(device);
-
-       return 0;
-}
-
-static void iwl_pci_system_complete(struct device *device)
-{
-       struct pci_dev *pdev = to_pci_dev(device);
-       struct iwl_trans *trans = pci_get_drvdata(pdev);
-
-       IWL_DEBUG_RPM(trans, "completing system suspend\n");
-}
-#endif /* CONFIG_IWLWIFI_PCIE_RTPM */
-
 static const struct dev_pm_ops iwl_dev_pm_ops = {
        SET_SYSTEM_SLEEP_PM_OPS(iwl_pci_suspend,
                                iwl_pci_resume)
-#ifdef CONFIG_IWLWIFI_PCIE_RTPM
-       SET_RUNTIME_PM_OPS(iwl_pci_runtime_suspend,
-                          iwl_pci_runtime_resume,
-                          NULL)
-       .prepare = iwl_pci_system_prepare,
-       .complete = iwl_pci_system_complete,
-#endif /* CONFIG_IWLWIFI_PCIE_RTPM */
 };
 
 #define IWL_PM_OPS     (&iwl_dev_pm_ops)
index d8b0f3b01acb787127f8b6355fb8db034ff7d2a1..1bbbf978245db2b5555ff38956a288d4c9a5bfa6 100644 (file)
@@ -3654,11 +3654,7 @@ struct iwl_trans *iwl_trans_pcie_alloc(struct pci_dev *pdev,
                                                   WQ_HIGHPRI | WQ_UNBOUND, 1);
        INIT_WORK(&trans_pcie->rba.rx_alloc, iwl_pcie_rx_allocator_work);
 
-#ifdef CONFIG_IWLWIFI_PCIE_RTPM
-       trans->runtime_pm_mode = IWL_PLAT_PM_MODE_D0I3;
-#else
        trans->runtime_pm_mode = IWL_PLAT_PM_MODE_DISABLED;
-#endif /* CONFIG_IWLWIFI_PCIE_RTPM */
 
 #ifdef CONFIG_IWLWIFI_DEBUGFS
        trans_pcie->fw_mon_data.state = IWL_FW_MON_DBGFS_STATE_CLOSED;