Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751488AbdGRHOx (ORCPT ); Tue, 18 Jul 2017 03:14:53 -0400 Received: from sabertooth02.qualcomm.com ([65.197.215.38]:3750 "EHLO sabertooth02.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751348AbdGRHOv (ORCPT ); Tue, 18 Jul 2017 03:14:51 -0400 X-IronPort-AV: E=Sophos;i="5.40,377,1496127600"; d="scan'208";a="111783914" X-IronPort-AV: E=McAfee;i="5800,7501,8594"; a="20520296" X-MGA-submission: =?us-ascii?q?MDHBB3/WR7TfSpjfHpq3/7/aUxZl7rv7qknvzI?= =?us-ascii?q?ba1GJg/4aO6hX7NBWlQ+nAt9W6+HwaLOGgPiEnbHvNe5m87xNRTVVUmc?= =?us-ascii?q?LqEpMuOrykEQOIXuHI1SLDu+qH3gaSFNmUBZUZjDaPLowfVJRzDGVyu1?= =?us-ascii?q?HL?= From: Ryan Hsu To: Gabriel C , "ath10k@lists.infradead.org" CC: "netdev@vger.kernel.org" , "linux-wireless@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: ath10k_htt_rx_amsdu_allowed() noise Thread-Topic: ath10k_htt_rx_amsdu_allowed() noise Thread-Index: AQHS/obikCFebdXDTUe7NutUPgo4/KJZo0iA Date: Tue, 18 Jul 2017 07:14:19 +0000 Message-ID: <79f47b0b-eb71-db71-2700-e30b01b11c76@qca.qualcomm.com> References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [199.106.107.6] Content-Type: text/plain; charset="Windows-1252" Content-ID: MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id v6I7ExpN011890 Content-Length: 1204 Lines: 37 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