Re: ath10k_htt_rx_amsdu_allowed() noise
On 18.07.2017 09:14, Ryan Hsu wrote: On 07/16/2017 03:56 PM, Gabriel C wrote: Can ath10k_warn() be ath10k_dbg() there ? Maybe this ? Looks good to me. From d4138d936635ca7b69ed7f7b0cda4914f0f07917 Mon Sep 17 00:00:00 2001 From: Gabriel Craciunescu Date: Mon, 17 Jul 2017 00:45:29 +0200 Subject: [PATCH] ath10k_htt_rx_amsdu_allowed(): change ath10k_warn() to th10k_dbg() Signed-off-by: Gabriel Craciunescu --- drivers/net/wireless/ath/ath10k/htt_rx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath10k/htt_rx.c b/drivers/net/wireless/ath/ath10k/htt_rx.c index 398dda978d6e..ad0306cd6ee1 100644 --- a/drivers/net/wireless/ath/ath10k/htt_rx.c +++ b/drivers/net/wireless/ath/ath10k/htt_rx.c @@ -1514,7 +1514,7 @@ static bool ath10k_htt_rx_amsdu_allowed(struct ath10k *ar, */ if (!rx_status->freq) { - ath10k_warn(ar, "no channel configured; ignoriframe(s)!\n"); + ath10k_dbg(ar, ATH10K_DBG_HTT, "no channel configured, ignoring frame(s)!\n"); return false; } Can you send this as patch for reviewing, please? Sure .. I'll try to send the patch to the list later on today.. Regards, Gabriel C
Re: ath10k_htt_rx_amsdu_allowed() noise
On 07/16/2017 03:56 PM, Gabriel C wrote: > > Can ath10k_warn() be ath10k_dbg() there ? Maybe this ? > Looks good to me. > > From d4138d936635ca7b69ed7f7b0cda4914f0f07917 Mon Sep 17 00:00:00 2001 > From: Gabriel Craciunescu > Date: Mon, 17 Jul 2017 00:45:29 +0200 > Subject: [PATCH] ath10k_htt_rx_amsdu_allowed(): change ath10k_warn() to > th10k_dbg() > > Signed-off-by: Gabriel Craciunescu > --- > drivers/net/wireless/ath/ath10k/htt_rx.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/ath/ath10k/htt_rx.c > b/drivers/net/wireless/ath/ath10k/htt_rx.c > index 398dda978d6e..ad0306cd6ee1 100644 > --- a/drivers/net/wireless/ath/ath10k/htt_rx.c > +++ b/drivers/net/wireless/ath/ath10k/htt_rx.c > @@ -1514,7 +1514,7 @@ static bool ath10k_htt_rx_amsdu_allowed(struct ath10k > *ar, > */ > > if (!rx_status->freq) { > - ath10k_warn(ar, "no channel configured; ignoriframe(s)!\n"); > + ath10k_dbg(ar, ATH10K_DBG_HTT, "no channel configured, > ignoring frame(s)!\n"); > return false; > } > Can you send this as patch for reviewing, please? -- Ryan Hsu