From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161927AbbBDTA0 (ORCPT ); Wed, 4 Feb 2015 14:00:26 -0500 Received: from mail-ig0-f182.google.com ([209.85.213.182]:61364 "EHLO mail-ig0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1161462AbbBDTAY (ORCPT ); Wed, 4 Feb 2015 14:00:24 -0500 Date: Wed, 4 Feb 2015 11:00:18 -0800 From: Dmitry Torokhov To: Geert Uytterhoeven Cc: Andrew Morton , Andrey Ryabinin , "James E.J. Bottomley" , Helge Deller , linux-parisc@vger.kernel.org, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] MODULE_DEVICE_TABLE: fix gscps2 Message-ID: <20150204190018.GA7818@dtor-ws> References: <1423062935-2025-1-git-send-email-geert@linux-m68k.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1423062935-2025-1-git-send-email-geert@linux-m68k.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 04, 2015 at 04:15:35PM +0100, Geert Uytterhoeven wrote: > The patch "module: fix types of device tables aliases" newly requires > that invocations of > > MODULE_DEVICE_TABLE(type, name); > > come *after* the definition of `name'. That is reasonable, but gscps2 > wasn't doing this. Fix it. > > Signed-off-by: Geert Uytterhoeven Applied, thank you. > --- > One more fix for your collection... > > parisc/allmodconfig > http://kisskb.ellerman.id.au/kisskb/buildresult/12358386/ > --- > drivers/input/serio/gscps2.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/input/serio/gscps2.c b/drivers/input/serio/gscps2.c > index 8d9ba0c3827c5b2a..94ab494a6ade8d49 100644 > --- a/drivers/input/serio/gscps2.c > +++ b/drivers/input/serio/gscps2.c > @@ -40,7 +40,6 @@ > MODULE_AUTHOR("Laurent Canet , Thibaut Varene , Helge Deller "); > MODULE_DESCRIPTION("HP GSC PS2 port driver"); > MODULE_LICENSE("GPL"); > -MODULE_DEVICE_TABLE(parisc, gscps2_device_tbl); > > #define PFX "gscps2.c: " > > @@ -439,6 +438,7 @@ static struct parisc_device_id gscps2_device_tbl[] = { > #endif > { 0, } /* 0 terminated list */ > }; > +MODULE_DEVICE_TABLE(parisc, gscps2_device_tbl); > > static struct parisc_driver parisc_ps2_driver = { > .name = "gsc_ps2", > -- > 1.9.1 > -- Dmitry