Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755324Ab0AETCJ (ORCPT ); Tue, 5 Jan 2010 14:02:09 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754736Ab0AETCE (ORCPT ); Tue, 5 Jan 2010 14:02:04 -0500 Received: from mail-ew0-f219.google.com ([209.85.219.219]:38936 "EHLO mail-ew0-f219.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754732Ab0AETAv (ORCPT ); Tue, 5 Jan 2010 14:00:51 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:subject:date:user-agent:cc:references:in-reply-to :mime-version:message-id:content-type:content-transfer-encoding; b=X/+zrYdoFodQvOW2T/YxDGzVigEUOyJkXPQvfxLtEfUInS0FTl8jNjVWq2S1X9IEyF WemzHHss+2yqqHEE0zQDBjon3oR4obeLOsnMGTBC4T5kTkiRoAMijuVvwwvz1qA3fAXk tv+ZRJwklsH9AsLztojSIVXV8K/n9A0AMqnvA= From: Bartlomiej Zolnierkiewicz To: Jiri Kosina Subject: Re: [PATCH 2/2] ide: add __init/__exit macros to drivers/ide/scc_pata.c Date: Tue, 5 Jan 2010 19:39:07 +0100 User-Agent: KMail/1.12.2 (Linux/2.6.31.8-0.1-desktop; KDE/4.3.1; x86_64; ; ) Cc: Peter Huewe , kernel-janitors@vger.kernel.org, "David S. Miller" , Sergei Shtylyov , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org References: <1261470564-23754-1-git-send-email-peterhuewe@gmx.de> <1261470564-23754-2-git-send-email-peterhuewe@gmx.de> In-Reply-To: MIME-Version: 1.0 Message-Id: <201001051939.07756.bzolnier@gmail.com> Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3333 Lines: 104 On Tuesday 05 January 2010 11:25:13 am Jiri Kosina wrote: > On Tue, 22 Dec 2009, peterhuewe@gmx.de wrote: > > > From: Peter Huewe > > > > Trivial patch which adds the __init/__exit macros to the module_init/ > > module_exit functions of > > > > drivers/ide/scc_pata.c > > > > Please have a look at the small patch and either pull it through > > your tree, or please ack' it so Jiri can pull it through the trivial > > tree. > > > > Patch against linux-next-tree, 22. Dez 08:38:18 CET 2009 > > but also present in linus tree. > > > > Signed-off-by: Peter Huewe > > --- > > drivers/ide/scc_pata.c | 4 ++-- > > 1 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/ide/scc_pata.c b/drivers/ide/scc_pata.c > > index 1104bb3..58c9c91 100644 > > --- a/drivers/ide/scc_pata.c > > +++ b/drivers/ide/scc_pata.c > > @@ -872,14 +872,14 @@ static struct pci_driver scc_pci_driver = { > > .remove = __devexit_p(scc_remove), > > }; > > > > -static int scc_ide_init(void) > > +static int __init scc_ide_init(void) > > { > > return ide_pci_register_driver(&scc_pci_driver); > > } > > > > module_init(scc_ide_init); > > /* -- No exit code? > > -static void scc_ide_exit(void) > > +static void __exit scc_ide_exit(void) > > { > > ide_pci_unregister_driver(&scc_pci_driver); > > } > > This code is commented out anyway. So we should either nuke it out > completely if it is not needed, or uncomment and fix. > > Bartlomiej, David? We can fix it nowadays, though I worry that by fixing it we may be 'creating competition' for 'officially blessed yet unmaintained' PATA stack... 8-) From: Bartlomiej Zolnierkiewicz Subject: [PATCH] scc_pata: fix module unloading scc_pata host driver predated module unloading support for IDE host drivers so even though it supports PCI hot-unplug and implements PCI device ->remove method it doesn't allow module removal. Fix it. Add missing __init/__exit tags to module_init/module_exit functions while at it (from Peter Huewe). Noticed-by: Jiri Kosina Cc: Peter Huewe Signed-off-by: Bartlomiej Zolnierkiewicz --- drivers/ide/scc_pata.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) Index: b/drivers/ide/scc_pata.c =================================================================== --- a/drivers/ide/scc_pata.c +++ b/drivers/ide/scc_pata.c @@ -872,20 +872,18 @@ static struct pci_driver scc_pci_driver .remove = __devexit_p(scc_remove), }; -static int scc_ide_init(void) +static int __init scc_ide_init(void) { return ide_pci_register_driver(&scc_pci_driver); } -module_init(scc_ide_init); -/* -- No exit code? -static void scc_ide_exit(void) +static void __exit scc_ide_exit(void) { - ide_pci_unregister_driver(&scc_pci_driver); + pci_unregister_driver(&scc_pci_driver); } -module_exit(scc_ide_exit); - */ +module_init(scc_ide_init); +module_exit(scc_ide_exit); MODULE_DESCRIPTION("PCI driver module for Toshiba SCC IDE"); MODULE_LICENSE("GPL"); -- 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/