[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20160927154725.609A161405@smtp.codeaurora.org>
Date: Tue, 27 Sep 2016 15:47:25 +0000 (UTC)
From: Kalle Valo <kvalo@...eaurora.org>
To: Rafał Miłecki <zajec5@...il.com>
Cc: Arend van Spriel <arend.vanspriel@...adcom.com>,
Franky Lin <franky.lin@...adcom.com>,
Hante Meuleman <hante.meuleman@...adcom.com>,
Pieter-Paul Giesberts <pieter-paul.giesberts@...adcom.com>,
Franky Lin <frankyl@...adcom.com>,
linux-wireless@...r.kernel.org,
brcm80211-dev-list.pdl@...adcom.com, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org,
Rafał Miłecki <rafal@...ecki.pl>
Subject: Re: brcmfmac: replace WARNING on timeout with a simple error message
Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@...ecki.pl>
>
> Even with timeout increased to 950 ms we get WARNINGs from time to time.
> It mostly happens on A-MPDU stalls (e.g. when station goes out of
> range). It may take up to 5-10 secods for the firmware to recover and
> for that time it doesn't process packets.
>
> It's still useful to have a message on time out as it may indicate some
> firmware problem and incorrect key update. Raising a WARNING however
> wasn't really that necessary, it doesn't point to any driver bug anymore
> and backtrace wasn't much useful.
>
> Signed-off-by: Rafał Miłecki <rafal@...ecki.pl>
> Acked-by: Arend van Spriel <arend.vanspriel@...adcom.com>
Patch applied to wireless-drivers-next.git, thanks.
2f0e56fa37cc brcmfmac: replace WARNING on timeout with a simple error message
--
https://patchwork.kernel.org/patch/9351657/
Documentation about submitting wireless patches and checking status
from patchwork:
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
Powered by blists - more mailing lists