From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH v2] b44: fix resume, request_irq after hw reset Date: Thu, 21 Oct 2010 04:30:36 -0700 (PDT) Message-ID: <20101021.043036.57469867.davem@davemloft.net> References: <201010171249.00485.james@albanarts.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: zambrano@broadcom.com, jpirko@redhat.com, fujita.tomonori@lab.ntt.co.jp, hauke@hauke-m.de, Larry.Finger@lwfinger.net, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, akpm@linux-foundation.org To: james@albanarts.com Return-path: Received: from 74-93-104-97-Washington.hfc.comcastbusiness.net ([74.93.104.97]:43506 "EHLO sunset.davemloft.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755013Ab0JULaN (ORCPT ); Thu, 21 Oct 2010 07:30:13 -0400 In-Reply-To: <201010171249.00485.james@albanarts.com> Sender: netdev-owner@vger.kernel.org List-ID: From: James Hogan Date: Sun, 17 Oct 2010 12:48:59 +0100 > On resume, call request_irq() after resetting the hardware rather than > before. It's a shared interrupt so the handler could be called > immediately if another device on the same irq interrupts (and will be > called immediately if CONFIG_DEBUG_SHIRQ=y), but unless the hardware is > reinitialised with b44_init_hw() the read of the interrupt status > register will hang the system. > > Signed-off-by: James Hogan Applied.