Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754559Ab2FZWdh (ORCPT ); Tue, 26 Jun 2012 18:33:37 -0400 Received: from mail-lb0-f174.google.com ([209.85.217.174]:36078 "EHLO mail-lb0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753789Ab2FZWde convert rfc822-to-8bit (ORCPT ); Tue, 26 Jun 2012 18:33:34 -0400 MIME-Version: 1.0 In-Reply-To: <20120621202435.16865.1340.stgit@amt.stowe> References: <20120621202415.16865.6226.stgit@amt.stowe> <20120621202435.16865.1340.stgit@amt.stowe> From: Bjorn Helgaas Date: Tue, 26 Jun 2012 16:33:12 -0600 Message-ID: Subject: Re: [PATCH 3/9] arm/PCI: move final fixup quirks from __init to __devinit To: Myron Stowe Cc: linux-pci@vger.kernel.org, linux@arm.linux.org.uk, ralf@linux-mips.org, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT X-System-Of-Record: true Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2098 Lines: 48 On Thu, Jun 21, 2012 at 2:24 PM, Myron Stowe wrote: > The PCI subsystem's final fixups are executed once during boot, after the > pci-device is found. ?As long as the system does not support hot-plug, > specifying __init is fine. > > With hot-plug, either physically based hot-plug events or pseudo hot-plug > events such as "echo 1 > /sys/bus/pci/rescan", it is possible to remove a > PCI bus during run time and have it rediscovered which will require the > call of the fixups again in order for the device to function properly. > > This patch prepares specific quirk(s) for use with hot-plug events. > > Signed-off-by: Myron Stowe > --- > > ?arch/arm/mach-iop32x/n2100.c | ? ?2 +- > ?1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-iop32x/n2100.c b/arch/arm/mach-iop32x/n2100.c > index 5a7ae91..04c4110 100644 > --- a/arch/arm/mach-iop32x/n2100.c > +++ b/arch/arm/mach-iop32x/n2100.c > @@ -126,7 +126,7 @@ static struct hw_pci n2100_pci __initdata = { > ?* the ->broken_parity_status flag for both ports so that the r8169 > ?* driver knows it should ignore error interrupts. > ?*/ > -static void n2100_fixup_r8169(struct pci_dev *dev) > +static void __devinit n2100_fixup_r8169(struct pci_dev *dev) These actually move functions from normal text to __devinit, not from __init to __devinit. That should be safe for most quirks (although enable, suspend, resume, and resume_early quirks can be called after we free initmem, even if we have CONFIG_HOTPLUG=n), but given the discussion about whether we should even bother with __devinit any more, I think I'll hold off on these for now. If we were to get rid of __devinit, these would all have to revert to being normal text. > ?{ > ? ? ? ?if (dev->bus->number == 0 && > ? ? ? ? ? ?(dev->devfn == PCI_DEVFN(1, 0) || > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/