iwlwifi: fix and describe iwl_adjust_beacon_interval

The iwl_adjust_beacon_interval function is a bit
of black magic, so add comments to it describing
what it does. Also, in the case when there's no
beacon interval set, program the default into
the device (instead of adjusting, which results
in the max) since using the max in that case
interacts badly with dual-mode/PAN parameters.

Also update the PAN parameters accordingly and
use the same constant as here.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/drivers/net/wireless/iwlwifi/iwl-agn-hcmd.c b/drivers/net/wireless/iwlwifi/iwl-agn-hcmd.c
index a63582f060f1..d86902b83630 100644
--- a/drivers/net/wireless/iwlwifi/iwl-agn-hcmd.c
+++ b/drivers/net/wireless/iwlwifi/iwl-agn-hcmd.c
@@ -321,7 +321,7 @@ static int iwlagn_set_pan_params(struct iwl_priv *priv)
                         bcnint = max_t(int, bcnint,
                                        ctx_bss->vif->bss_conf.beacon_int);
                 if (!bcnint)
-                        bcnint = 100;
+                        bcnint = DEFAULT_BEACON_INTERVAL;
                 slot0 = bcnint / 2;
                 slot1 = bcnint - slot0;
 
@@ -339,7 +339,7 @@ static int iwlagn_set_pan_params(struct iwl_priv *priv)
                 slot0 = 0;
                 slot1 = max_t(int, 1, ctx_pan->vif->bss_conf.dtim_period) *
                                         ctx_pan->vif->bss_conf.beacon_int;
-                slot1 = max_t(int, 100, slot1);
+                slot1 = max_t(int, DEFAULT_BEACON_INTERVAL, slot1);
 
                 if (test_bit(STATUS_SCAN_HW, &priv->status)) {
                         slot0 = slot1 * 3 - 20;