[<prev] [next>] [day] [month] [year] [list]
Message-ID: <20250407085247.1992-1-vulab@iscas.ac.cn>
Date: Mon, 7 Apr 2025 16:52:47 +0800
From: Wentao Liang <vulab@...as.ac.cn>
To: miriam.rachel.korenblit@...el.com,
kvalo@...nel.org
Cc: johannes.berg@...el.com,
emmanuel.grumbach@...el.com,
golan.ben.ami@...el.com,
linux-wireless@...r.kernel.org,
linux-kernel@...r.kernel.org,
Wentao Liang <vulab@...as.ac.cn>
Subject: [PATCH v2] wifi: iwlwifi: mvm: Add error logging for iwl_finish_nic_init()
The function iwl_pci_resume() calls the function iwl_finish_nic_init(),
but does not check their return values.
Log a detailed error message with the error code to aid in diagnosing
root causes if encountering irreparable errors. While this does not fix
the underlying problem, it assist debugging by making the failure
visible in logs.
Signed-off-by: Wentao Liang <vulab@...as.ac.cn>
---
v2: Fix improper code.
drivers/net/wireless/intel/iwlwifi/pcie/drv.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/net/wireless/intel/iwlwifi/pcie/drv.c b/drivers/net/wireless/intel/iwlwifi/pcie/drv.c
index 8fb2aa282242..34a7a3e4f7c0 100644
--- a/drivers/net/wireless/intel/iwlwifi/pcie/drv.c
+++ b/drivers/net/wireless/intel/iwlwifi/pcie/drv.c
@@ -1616,6 +1616,7 @@ static int _iwl_pci_resume(struct device *device, bool restore)
struct iwl_trans *trans = pci_get_drvdata(pdev);
struct iwl_trans_pcie *trans_pcie = IWL_TRANS_GET_PCIE_TRANS(trans);
bool device_was_powered_off = false;
+ int err;
/* Before you put code here, think about WoWLAN. You cannot check here
* whether WoWLAN is enabled or not, and your code will run even if
@@ -1647,7 +1648,11 @@ static int _iwl_pci_resume(struct device *device, bool restore)
* won't really know how to recover.
*/
iwl_pcie_prepare_card_hw(trans);
- iwl_finish_nic_init(trans);
+ err = iwl_finish_nic_init(trans);
+ if (err)
+ IWL_ERR(trans,
+ "NIC initialization failed after power-off (error %d).",
+ err);
iwl_op_mode_device_powered_off(trans->op_mode);
}
--
2.42.0.windows.2
Powered by blists - more mailing lists