On Mon, 2019-09-30 at 14:06 +0200, Matteo Croce wrote:
> On Fri, Jun 28, 2019 at 11:20 AM Luca Coelho <luca@xxxxxxxxx> wrote:
> > From: Haim Dreyfuss <haim.dreyfuss@xxxxxxxxx>
> >
> > Inform users when SAR status is changing.
> >
> > Signed-off-by: Haim Dreyfuss <haim.dreyfuss@xxxxxxxxx>
> > Signed-off-by: Luca Coelho <luciano.coelho@xxxxxxxxx>
> > ---
> > drivers/net/wireless/intel/iwlwifi/mvm/fw.c | 3 +++
> > drivers/net/wireless/intel/iwlwifi/mvm/nvm.c | 9 +++++++++
> > 2 files changed, 12 insertions(+)
> >
> > diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/fw.c b/drivers/net/wireless/intel/iwlwifi/mvm/fw.c
> > index b27be2e3eca2..41a98cf01d0e 100644
> > --- a/drivers/net/wireless/intel/iwlwifi/mvm/fw.c
> > +++ b/drivers/net/wireless/intel/iwlwifi/mvm/fw.c
> > @@ -850,6 +850,9 @@ int iwl_mvm_sar_select_profile(struct iwl_mvm *mvm, int prof_a, int prof_b)
> > return -ENOENT;
> > }
> >
> > + IWL_DEBUG_INFO(mvm,
> > + "SAR EWRD: chain %d profile index %d\n",
> > + i, profs[i]);
> > IWL_DEBUG_RADIO(mvm, " Chain[%d]:\n", i);
> > for (j = 0; j < ACPI_SAR_NUM_SUB_BANDS; j++) {
> > idx = (i * ACPI_SAR_NUM_SUB_BANDS) + j;
> > diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/nvm.c b/drivers/net/wireless/intel/iwlwifi/mvm/nvm.c
> > index 7bdbd010ae6b..719f793b3487 100644
> > --- a/drivers/net/wireless/intel/iwlwifi/mvm/nvm.c
> > +++ b/drivers/net/wireless/intel/iwlwifi/mvm/nvm.c
> > @@ -620,6 +620,7 @@ void iwl_mvm_rx_chub_update_mcc(struct iwl_mvm *mvm,
> > enum iwl_mcc_source src;
> > char mcc[3];
> > struct ieee80211_regdomain *regd;
> > + u32 wgds_tbl_idx;
> >
> > lockdep_assert_held(&mvm->mutex);
> >
> > @@ -643,6 +644,14 @@ void iwl_mvm_rx_chub_update_mcc(struct iwl_mvm *mvm,
> > if (IS_ERR_OR_NULL(regd))
> > return;
> >
> > + wgds_tbl_idx = iwl_mvm_get_sar_geo_profile(mvm);
> > + if (wgds_tbl_idx < 0)
> > + IWL_DEBUG_INFO(mvm, "SAR WGDS is disabled (%d)\n",
> > + wgds_tbl_idx);
> > + else
> > + IWL_DEBUG_INFO(mvm, "SAR WGDS: geo profile %d is configured\n",
> > + wgds_tbl_idx);
> > +
> > regulatory_set_wiphy_regd(mvm->hw->wiphy, regd);
> > kfree(regd);
> > }
> > --
> > 2.20.1
> >
>
> Hi all,
>
> this patch seems to cause a long microcode dump:
>
> [ 16.792357] iwlwifi 0000:04:00.0: Applying debug destination EXTERNAL_DRAM
> [ 16.939172] iwlwifi 0000:04:00.0: Applying debug destination EXTERNAL_DRAM
> [ 17.019127] iwlwifi 0000:04:00.0: FW already configured (0) - re-configuring
> [ 17.053395] iwlwifi 0000:04:00.0: Applying debug destination EXTERNAL_DRAM
> [ 17.200436] iwlwifi 0000:04:00.0: Applying debug destination EXTERNAL_DRAM
> [ 17.280572] iwlwifi 0000:04:00.0: FW already configured (0) - re-configuring
> [ 20.822983] iwlwifi 0000:04:00.0: Microcode SW error detected.
> Restarting 0x82000000.
> [ 20.823153] iwlwifi 0000:04:00.0: Start IWL Error Log Dump:
> [ 20.823155] iwlwifi 0000:04:00.0: Status: 0x00000080, count: 6
> [ 20.823157] iwlwifi 0000:04:00.0: Loaded firmware version: 36.77d01142.0
> [ 20.823158] iwlwifi 0000:04:00.0: 0x00000038 | BAD_COMMAND
[...]
> Reverting the patch fixes it, this is my card:
> $ lspci -d ::280
> 04:00.0 Network controller: Intel Corporation Wireless 8260 (rev 3a)
Thanks for reporting!
But this bug was already reported and fixed. Please check this:
https://bugzilla.kernel.org/show_bug.cgi?id=204151#c15
(specifically comment 15)
HTH.
--
Cheers,
Luca.
