Commit 008a9e3e authored by Ben Cahill's avatar Ben Cahill Committed by John W. Linville

iwl3945: move iwl_power_initialize()

iwl_power_initialize() initializes driver data (not device hardware), and does
not need to execute more than once (when the driver initializes).  Therefore, it
does not belong in iwl3945_apm_init(), which initializes hardware, and may run
more than once.

Move it to iwl3945_pci_probe(), where it will run only once.  This agrees
with similar placement in iwl-agn.c's iwl_pci_probe(), although placement
under "services" seemed more appropriate than under "mac80211".
Signed-off-by: default avatarBen Cahill <ben.m.cahill@intel.com>
Signed-off-by: default avatarReinette Chatre <reinette.chatre@intel.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 743cdf1b
...@@ -991,8 +991,6 @@ static int iwl3945_apm_init(struct iwl_priv *priv) ...@@ -991,8 +991,6 @@ static int iwl3945_apm_init(struct iwl_priv *priv)
{ {
int ret; int ret;
iwl_power_initialize(priv);
/* Configure chip clock phase-lock-loop */ /* Configure chip clock phase-lock-loop */
iwl_set_bit(priv, CSR_ANA_PLL_CFG, CSR39_ANA_PLL_CFG_VAL); iwl_set_bit(priv, CSR_ANA_PLL_CFG, CSR39_ANA_PLL_CFG_VAL);
......
...@@ -4064,6 +4064,7 @@ static int iwl3945_pci_probe(struct pci_dev *pdev, const struct pci_device_id *e ...@@ -4064,6 +4064,7 @@ static int iwl3945_pci_probe(struct pci_dev *pdev, const struct pci_device_id *e
&priv->bands[IEEE80211_BAND_2GHZ].channels[5]); &priv->bands[IEEE80211_BAND_2GHZ].channels[5]);
iwl3945_setup_deferred_work(priv); iwl3945_setup_deferred_work(priv);
iwl3945_setup_rx_handlers(priv); iwl3945_setup_rx_handlers(priv);
iwl_power_initialize(priv);
/********************************* /*********************************
* 8. Setup and Register mac80211 * 8. Setup and Register mac80211
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment