From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Tobias Schandinat Date: Mon, 19 Dec 2011 23:15:27 +0000 Subject: Re: [PATCH 3/3] video: pnx4008: convert drivers/video/pnx4008/* to Message-Id: <4EEFC58F.5020106@gmx.de> List-Id: References: <1323395873.3740.4.camel@phoenix> <1323396075.3740.8.camel@phoenix> In-Reply-To: <1323396075.3740.8.camel@phoenix> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Axel Lin Cc: linux-kernel@vger.kernel.org, Grigory Tolstolytkin , linux-fbdev@vger.kernel.org On 12/09/2011 02:01 AM, Axel Lin wrote: > This patch converts the drivers in drivers/video/pnx4008/* to use the > module_platform_driver() macro which makes the code smaller and a bit > simpler. > > Cc: Grigory Tolstolytkin > Signed-off-by: Axel Lin Applied. Thanks, Florian Tobias Schandinat > --- > drivers/video/pnx4008/pnxrgbfb.c | 13 +------------ > drivers/video/pnx4008/sdum.c | 13 +------------ > 2 files changed, 2 insertions(+), 24 deletions(-) > > diff --git a/drivers/video/pnx4008/pnxrgbfb.c b/drivers/video/pnx4008/pnxrgbfb.c > index b2252fe..6d30428 100644 > --- a/drivers/video/pnx4008/pnxrgbfb.c > +++ b/drivers/video/pnx4008/pnxrgbfb.c > @@ -193,17 +193,6 @@ static struct platform_driver rgbfb_driver = { > .remove = rgbfb_remove, > }; > > -static int __init rgbfb_init(void) > -{ > - return platform_driver_register(&rgbfb_driver); > -} > - > -static void __exit rgbfb_exit(void) > -{ > - platform_driver_unregister(&rgbfb_driver); > -} > - > -module_init(rgbfb_init); > -module_exit(rgbfb_exit); > +module_platform_driver(rgbfb_driver); > > MODULE_LICENSE("GPL"); > diff --git a/drivers/video/pnx4008/sdum.c b/drivers/video/pnx4008/sdum.c > index 50e0039..c5c7414 100644 > --- a/drivers/video/pnx4008/sdum.c > +++ b/drivers/video/pnx4008/sdum.c > @@ -856,17 +856,6 @@ static struct platform_driver sdum_driver = { > .resume = sdum_resume, > }; > > -int __init sdum_init(void) > -{ > - return platform_driver_register(&sdum_driver); > -} > - > -static void __exit sdum_exit(void) > -{ > - platform_driver_unregister(&sdum_driver); > -}; > - > -module_init(sdum_init); > -module_exit(sdum_exit); > +module_platform_driver(sdum_driver); > > MODULE_LICENSE("GPL"); From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753411Ab1LSXPf (ORCPT ); Mon, 19 Dec 2011 18:15:35 -0500 Received: from mailout-de.gmx.net ([213.165.64.23]:44773 "HELO mailout-de.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1752453Ab1LSXPc (ORCPT ); Mon, 19 Dec 2011 18:15:32 -0500 X-Authenticated: #10250065 X-Provags-ID: V01U2FsdGVkX1/sx10zGRDsAwWkQiwm9P/uHnIE4qHePL7hCHmOa7 vXDtK8plhRgvZo Message-ID: <4EEFC58F.5020106@gmx.de> Date: Mon, 19 Dec 2011 23:15:27 +0000 From: Florian Tobias Schandinat User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.16) Gecko/20111004 Icedove/3.0.11 MIME-Version: 1.0 To: Axel Lin CC: linux-kernel@vger.kernel.org, Grigory Tolstolytkin , linux-fbdev@vger.kernel.org Subject: Re: [PATCH 3/3] video: pnx4008: convert drivers/video/pnx4008/* to use module_platform_driver() References: <1323395873.3740.4.camel@phoenix> <1323396075.3740.8.camel@phoenix> In-Reply-To: <1323396075.3740.8.camel@phoenix> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Y-GMX-Trusted: 0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/09/2011 02:01 AM, Axel Lin wrote: > This patch converts the drivers in drivers/video/pnx4008/* to use the > module_platform_driver() macro which makes the code smaller and a bit > simpler. > > Cc: Grigory Tolstolytkin > Signed-off-by: Axel Lin Applied. Thanks, Florian Tobias Schandinat > --- > drivers/video/pnx4008/pnxrgbfb.c | 13 +------------ > drivers/video/pnx4008/sdum.c | 13 +------------ > 2 files changed, 2 insertions(+), 24 deletions(-) > > diff --git a/drivers/video/pnx4008/pnxrgbfb.c b/drivers/video/pnx4008/pnxrgbfb.c > index b2252fe..6d30428 100644 > --- a/drivers/video/pnx4008/pnxrgbfb.c > +++ b/drivers/video/pnx4008/pnxrgbfb.c > @@ -193,17 +193,6 @@ static struct platform_driver rgbfb_driver = { > .remove = rgbfb_remove, > }; > > -static int __init rgbfb_init(void) > -{ > - return platform_driver_register(&rgbfb_driver); > -} > - > -static void __exit rgbfb_exit(void) > -{ > - platform_driver_unregister(&rgbfb_driver); > -} > - > -module_init(rgbfb_init); > -module_exit(rgbfb_exit); > +module_platform_driver(rgbfb_driver); > > MODULE_LICENSE("GPL"); > diff --git a/drivers/video/pnx4008/sdum.c b/drivers/video/pnx4008/sdum.c > index 50e0039..c5c7414 100644 > --- a/drivers/video/pnx4008/sdum.c > +++ b/drivers/video/pnx4008/sdum.c > @@ -856,17 +856,6 @@ static struct platform_driver sdum_driver = { > .resume = sdum_resume, > }; > > -int __init sdum_init(void) > -{ > - return platform_driver_register(&sdum_driver); > -} > - > -static void __exit sdum_exit(void) > -{ > - platform_driver_unregister(&sdum_driver); > -}; > - > -module_init(sdum_init); > -module_exit(sdum_exit); > +module_platform_driver(sdum_driver); > > MODULE_LICENSE("GPL");