Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754598AbcJKXVq (ORCPT ); Tue, 11 Oct 2016 19:21:46 -0400 Received: from kvm5.telegraphics.com.au ([98.124.60.144]:58804 "EHLO kvm5.telegraphics.com.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753367AbcJKXVo (ORCPT ); Tue, 11 Oct 2016 19:21:44 -0400 Date: Wed, 12 Oct 2016 10:20:33 +1100 (AEDT) From: Finn Thain To: Ondrej Zary cc: Arnd Bergmann , "James E.J. Bottomley" , "Martin K. Petersen" , Michael Schmitz , Hannes Reinecke , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] scsi: NCR5380: no longer mark irq probing as __init In-Reply-To: <201610112258.05050.linux@rainbow-software.org> Message-ID: References: <20161011092356.2872181-1-arnd@arndb.de> <201610112258.05050.linux@rainbow-software.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1132 Lines: 33 On Tue, 11 Oct 2016, Ondrej Zary wrote: > > I've tested IRQ probing on my IRQ7-hardwired ISA card without any > success some time ago. No IRQ was detected because it wasn't even > generated (no increment in /proc/interruptts). The IRQ line does not appear in /proc/interrupts unless it has already been claimed. So if you saw that the interrupt count stayed the same, then the interrupt line wasn't available in the first place. > It also produced some warnings in dmesg about conflicting interrupt > flags. Presumably that's because there is no way to probe shared interrupt lines. > Can't the IRQ probing be simply deleted? > It isn't clear to me that there's a bug in the driver. I never saw a bug in the irq probe code when I went looking for a reason to delete that code a while ago. Anyway, assuming that the interrupt line on your card works fine, have you considered using probe_irq_on/probe_irq_off? See for example, sym53c416.c and NCR53c406a.c. Bug or no bug, I'd be very happy to delete the NCR5380 probe code if we can use the standard probe functions to retain the documented functionality. --