From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-px0-f174.google.com ([209.85.212.174]:46900 "EHLO mail-px0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753888Ab0EZSBc (ORCPT ); Wed, 26 May 2010 14:01:32 -0400 Received: by pxi18 with SMTP id 18so2642254pxi.19 for ; Wed, 26 May 2010 11:01:32 -0700 (PDT) Message-ID: <4BFD620B.8010301@gmail.com> Date: Wed, 26 May 2010 11:01:47 -0700 From: "Justin P. Mattock" MIME-Version: 1.0 To: "Luis R. Rodriguez" CC: linville@tuxdriver.com, arnd@arndnet.de, linux-wireless@vger.kernel.org Subject: Re: [PATCH] ath9k: Fix ath_print in xmit for harware reset. References: <1274896002-10866-1-git-send-email-justinmattock@gmail.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 05/26/2010 10:50 AM, Luis R. Rodriguez wrote: > On Wed, May 26, 2010 at 10:46 AM, Justin P. Mattock > wrote: > >> ath_print in xmit.c should say "Reseting hardware" >> instead of Resetting HAL!(since HAL is being fazed out). >> dmesg shows: >> [ 8660.899624] ath: Failed to stop TX DMA in 100 msec after killing last frame >> [ 8660.899676] ath: Unable to stop TxDMA. Reset HAL! >> >> Signed-off-by: Justin P. Mattock >> >> --- >> drivers/net/wireless/ath/ath9k/xmit.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/drivers/net/wireless/ath/ath9k/xmit.c b/drivers/net/wireless/ath/ath9k/xmit.c >> index 3db1917..617db8e 100644 >> --- a/drivers/net/wireless/ath/ath9k/xmit.c >> +++ b/drivers/net/wireless/ath/ath9k/xmit.c >> @@ -1198,7 +1198,7 @@ void ath_drain_all_txq(struct ath_softc *sc, bool retry_tx) >> int r; >> >> ath_print(common, ATH_DBG_FATAL, >> - "Unable to stop TxDMA. Reset HAL!\n"); >> + "ath9k: Failed to stop TX DMA. Resetting hardware!\n"); >> > Nack, No need to put "ath9k:" since ath_print already prepends "ath: " > > Luis > > ah.. so the ATH_DBG_FATAL takes care of that? resent.. Justin P. Mattock