From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lf0-f54.google.com ([209.85.215.54]:33781 "EHLO mail-lf0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750885AbcEGSpo (ORCPT ); Sat, 7 May 2016 14:45:44 -0400 Received: by mail-lf0-f54.google.com with SMTP id y84so163629249lfc.0 for ; Sat, 07 May 2016 11:45:43 -0700 (PDT) Subject: Re: [PATCH v2 2/2 -net-next] ravb: Add missing free_irq() calls to ravb_close() To: Geert Uytterhoeven , "David S. Miller" References: <1462619832-5177-1-git-send-email-geert+renesas@glider.be> <1462619832-5177-3-git-send-email-geert+renesas@glider.be> Cc: Yoshihiro Kaneko , Simon Horman , netdev@vger.kernel.org, linux-renesas-soc@vger.kernel.org From: Sergei Shtylyov Message-ID: <572E37D4.7050202@cogentembedded.com> Date: Sat, 7 May 2016 21:45:40 +0300 MIME-Version: 1.0 In-Reply-To: <1462619832-5177-3-git-send-email-geert+renesas@glider.be> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: Hello. On 05/07/2016 02:17 PM, Geert Uytterhoeven wrote: > When reopening the network device on ra7795/salvator-x, e.g. after a > DHCP timeout: > > IP-Config: Reopening network devices... > genirq: Flags mismatch irq 139. 00000000 (eth0:ch24:emac) vs. 00000000 ( Unwrapped line this time? :-) > ravb e6800000.ethernet eth0: cannot request IRQ eth0:ch24:emac > IP-Config: Failed to open eth0 > IP-Config: No network devices available > > The "mismatch" is due to requesting an IRQ that is already in use, > while IRQF_PROBE_SHARED wasn't set. > > However, the real cause is that ravb_close() doesn't release any of the > R-Car Gen3-specific secondary IRQs. > > Add the missing free_irq() calls to fix this. > > Fixes: 22d4df8ff3a3cc72 ("ravb: Add support for r8a7795 SoC") > Fixes: f51bdc236b6c5835 ("ravb: Add dma queue interrupt support") > Signed-off-by: Geert Uytterhoeven Acked-by: Sergei Shtylyov MBR, Sergei