From: Ben Greear <gree...@candelatech.com> This gives user-space a normal-ish way to detect that firmware has failed to start and that a reboot is probably required.
Signed-off-by: Ben Greear <gree...@candelatech.com> --- v2: New patch for this series, goes well with the other ethtool patch. drivers/net/wireless/ath/ath10k/core.h | 2 +- drivers/net/wireless/ath/ath10k/debug.c | 2 ++ drivers/net/wireless/ath/ath10k/pci.c | 12 +++++++++--- 3 files changed, 12 insertions(+), 4 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/core.h b/drivers/net/wireless/ath/ath10k/core.h index 7b220b1..601d573 100644 --- a/drivers/net/wireless/ath/ath10k/core.h +++ b/drivers/net/wireless/ath/ath10k/core.h @@ -399,7 +399,7 @@ struct ath10k { struct ieee80211_hw *hw; struct device *dev; u8 mac_addr[ETH_ALEN]; - + bool fw_powerup_failed; /* If true, might take reboot to recover. */ u32 chip_id; u32 target_version; u8 fw_version_major; diff --git a/drivers/net/wireless/ath/ath10k/debug.c b/drivers/net/wireless/ath/ath10k/debug.c index af1ca3e..77b60f4 100644 --- a/drivers/net/wireless/ath/ath10k/debug.c +++ b/drivers/net/wireless/ath/ath10k/debug.c @@ -1091,6 +1091,7 @@ static const char ath10k_gstrings_stats[][ETH_GSTRING_LEN] = { "d_fw_crash_count", "d_fw_warm_reset_count", "d_fw_cold_reset_count", + "d_fw_powerup_failed", /* boolean */ }; #define ATH10K_SSTATS_LEN ARRAY_SIZE(ath10k_gstrings_stats) @@ -1175,6 +1176,7 @@ void ath10k_get_et_stats(struct ieee80211_hw *hw, data[i++] = ar->fw_crash_counter; data[i++] = ar->fw_warm_reset_counter; data[i++] = ar->fw_cold_reset_counter; + data[i++] = ar->fw_powerup_failed; spin_unlock_bh(&ar->data_lock); diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c index 38f7386..ebe2ee1 100644 --- a/drivers/net/wireless/ath/ath10k/pci.c +++ b/drivers/net/wireless/ath/ath10k/pci.c @@ -1851,7 +1851,7 @@ static int ath10k_pci_hif_power_up(struct ath10k *ar) ret); if (ath10k_pci_reset_mode == ATH10K_PCI_RESET_WARM_ONLY) - return ret; + goto out; ath10k_warn(ar, "trying cold reset\n"); @@ -1859,11 +1859,17 @@ static int ath10k_pci_hif_power_up(struct ath10k *ar) if (ret) { ath10k_err(ar, "failed to power up target using cold reset too (%d)\n", ret); - return ret; + goto out; } } - return 0; +out: + /* If we have failed to power-up, it may take a reboot to + * get the NIC back online. + * Set flag accordinly so that user-space can know. + */ + ar->fw_powerup_failed = !!ret; + return ret; } static void ath10k_pci_hif_power_down(struct ath10k *ar) -- 1.7.11.7 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html