| Message ID | 20180423201205.20533-10-andresx7@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Andres Rodriguez <andresx7@gmail.com> writes: > The ath10k testmode uses request_firmware_direct() in order to avoid > producing firmware load warnings. Disabling the fallback mechanism was a > side effect of disabling warnings. > > We now have a new API that allows us to avoid warnings while keeping the > fallback mechanism enabled. So use that instead. > > Signed-off-by: Andres Rodriguez <andresx7@gmail.com> Thanks! Acked-by: Kalle Valo <kvalo@codeaurora.org>
diff --git a/drivers/net/wireless/ath/ath10k/testmode.c b/drivers/net/wireless/ath/ath10k/testmode.c index 568810b41657..0beb91be1b1c 100644 --- a/drivers/net/wireless/ath/ath10k/testmode.c +++ b/drivers/net/wireless/ath/ath10k/testmode.c @@ -157,7 +157,7 @@ static int ath10k_tm_fetch_utf_firmware_api_1(struct ath10k *ar, ar->hw_params.fw.dir, ATH10K_FW_UTF_FILE); /* load utf firmware image */ - ret = request_firmware_direct(&fw_file->firmware, filename, ar->dev); + ret = request_firmware_nowarn(&fw_file->firmware, filename, ar->dev); ath10k_dbg(ar, ATH10K_DBG_TESTMODE, "testmode fw request '%s': %d\n", filename, ret);
The ath10k testmode uses request_firmware_direct() in order to avoid producing firmware load warnings. Disabling the fallback mechanism was a side effect of disabling warnings. We now have a new API that allows us to avoid warnings while keeping the fallback mechanism enabled. So use that instead. Signed-off-by: Andres Rodriguez <andresx7@gmail.com> --- drivers/net/wireless/ath/ath10k/testmode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)