git: 28345b17efca - main - if_iwx: Make firmware load error messages clearer
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 28 May 2025 09:06:53 UTC
The branch main has been updated by thj:
URL: https://cgit.FreeBSD.org/src/commit/?id=28345b17efca045ba3a24cb05167bc03b6348faf
commit 28345b17efca045ba3a24cb05167bc03b6348faf
Author: Tom Jones <thj@FreeBSD.org>
AuthorDate: 2025-05-28 08:49:48 +0000
Commit: Tom Jones <thj@FreeBSD.org>
CommitDate: 2025-05-28 08:49:48 +0000
if_iwx: Make firmware load error messages clearer
Try to make it clearer where the errors are coming from in the iwx
firmware load path. In particular make it clearer that transferring
firmware to the card failed, not the loading of firmware.
Hide the init firmware failed message behind a debug level, there are
other errors which will show already in this path and for most people
this doesn't add much.
Reviewed by: adrian
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D49826
---
sys/dev/iwx/if_iwx.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/sys/dev/iwx/if_iwx.c b/sys/dev/iwx/if_iwx.c
index 03008e4fac93..029031802a26 100644
--- a/sys/dev/iwx/if_iwx.c
+++ b/sys/dev/iwx/if_iwx.c
@@ -2607,7 +2607,7 @@ iwx_apm_stop(struct iwx_softc *sc)
if (!iwx_poll_bit(sc, IWX_CSR_RESET,
IWX_CSR_RESET_REG_FLAG_MASTER_DISABLED,
IWX_CSR_RESET_REG_FLAG_MASTER_DISABLED, 100))
- printf("%s: timeout waiting for master\n", DEVNAME(sc));
+ printf("%s: timeout waiting for bus master\n", DEVNAME(sc));
/*
* Clear "initialization complete" bit to move adapter from
@@ -3697,7 +3697,7 @@ iwx_load_firmware(struct iwx_softc *sc)
/* wait for the firmware to load */
err = msleep(&sc->sc_uc, &sc->sc_mtx, 0, "iwxuc", hz);
if (err || !sc->sc_uc.uc_ok) {
- printf("%s: could not load firmware, %d\n", DEVNAME(sc), err);
+ printf("%s: firmware upload failed, %d\n", DEVNAME(sc), err);
iwx_ctxt_info_free_paging(sc);
}
@@ -4050,7 +4050,8 @@ iwx_run_init_mvm_ucode(struct iwx_softc *sc, int readnvm)
sc->sc_init_complete = 0;
err = iwx_load_ucode_wait_alive(sc);
if (err) {
- printf("%s: failed to load init firmware\n", DEVNAME(sc));
+ IWX_DPRINTF(sc, IWX_DEBUG_FIRMWARE_TLV,
+ "%s: failed to load init firmware\n", DEVNAME(sc));
return err;
} else {
IWX_DPRINTF(sc, IWX_DEBUG_FIRMWARE_TLV,