From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wm0-x22e.google.com ([2a00:1450:400c:c09::22e]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1dXqTD-00059T-Qe for ath10k@lists.infradead.org; Wed, 19 Jul 2017 15:01:01 +0000 Received: by mail-wm0-x22e.google.com with SMTP id w191so1669445wmw.1 for ; Wed, 19 Jul 2017 08:00:39 -0700 (PDT) Subject: Re: ath10k_htt_rx_amsdu_allowed() noise References: <79f47b0b-eb71-db71-2700-e30b01b11c76@qca.qualcomm.com> From: Gabriel C Message-ID: Date: Wed, 19 Jul 2017 17:00:36 +0200 MIME-Version: 1.0 In-Reply-To: <79f47b0b-eb71-db71-2700-e30b01b11c76@qca.qualcomm.com> Content-Language: en-US List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Ryan Hsu , "ath10k@lists.infradead.org" Cc: "netdev@vger.kernel.org" , "linux-wireless@vger.kernel.org" , "linux-kernel@vger.kernel.org" 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 _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k