diff mbox

ath6kl: Don't print error message when recv is canceled

Message ID 1447887088-19018-1-git-send-email-steve.derosier@lairdtech.com (mailing list archive)
State Accepted
Delegated to: Kalle Valo
Headers show

Commit Message

Steve deRosier Nov. 18, 2015, 10:51 p.m. UTC
An error message ath6kl_htc_rxmsg_pending_handler isn't appropate for when
the error is ECANCELED. This could be the result of a perfectly appropriate
RX cancel due to shutdown or suspend. This allows the right cleanup to
continue, but without an alarming error message in this particular case.

Signed-off-by: Steve deRosier <steve.derosier@lairdtech.com>
---
 drivers/net/wireless/ath/ath6kl/htc_mbox.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Kalle Valo Dec. 8, 2015, 2:56 p.m. UTC | #1
Steve deRosier <derosier@gmail.com> writes:

> An error message ath6kl_htc_rxmsg_pending_handler isn't appropate for when
> the error is ECANCELED. This could be the result of a perfectly appropriate
> RX cancel due to shutdown or suspend. This allows the right cleanup to
> continue, but without an alarming error message in this particular case.
>
> Signed-off-by: Steve deRosier <steve.derosier@lairdtech.com>

Applied, thanks.
diff mbox

Patch

diff --git a/drivers/net/wireless/ath/ath6kl/htc_mbox.c b/drivers/net/wireless/ath/ath6kl/htc_mbox.c
index fffb65b..65c31da 100644
--- a/drivers/net/wireless/ath/ath6kl/htc_mbox.c
+++ b/drivers/net/wireless/ath/ath6kl/htc_mbox.c
@@ -2222,8 +2222,9 @@  int ath6kl_htc_rxmsg_pending_handler(struct htc_target *target,
 	}
 
 	if (status) {
-		ath6kl_err("failed to get pending recv messages: %d\n",
-			   status);
+		if (status != -ECANCELED)
+			ath6kl_err("failed to get pending recv messages: %d\n",
+				   status);
 
 		/* cleanup any packets in sync completion queue */
 		list_for_each_entry_safe(packets, tmp_pkt, &comp_pktq, list) {