Re: [PATCH V3 2/2] brcmfmac: don't warn user about NVRAM if fallback to platform one succeeds

From: Kalle Valo
Date: Tue Feb 21 2017 - 06:47:02 EST

Arend Van Spriel <arend.vanspriel@xxxxxxxxxxxx> writes:

> On 21-2-2017 10:47, RafaÅ MiÅecki wrote:
>> From: RafaÅ MiÅecki <rafal@xxxxxxxxxx>
>> Failing to load NVRAM file isn't critical if we manage to get platform
>> one in the fallback path. It means warnings like:
>> [ 10.801506] brcmfmac 0000:01:00.0: Direct firmware load for
>> brcm/brcmfmac43602-pcie.txt failed with error -2
>> are unnecessary & disturbing for people with platform NVRAM. This is
>> very common case for Broadcom home routers.
>> So instead of printing warning immediately with the firmware subsystem
>> let's first try our fallback code. If that fails as well, then it's a
>> right moment to print an error.
>> This should reduce amount of false reports from users seeing this
>> warning while having wireless working perfectly fine.
> I think FW_OPT_NO_WARN does not cover all warnings in firmware_class
> although I did not check. Anyway...
> Acked-by: Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx>

Acked-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx>

Feel free to this via some another tree.

Kalle Valo