Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755011AbbGJQAT (ORCPT ); Fri, 10 Jul 2015 12:00:19 -0400 Received: from mail-oi0-f49.google.com ([209.85.218.49]:35399 "EHLO mail-oi0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754963AbbGJQAL (ORCPT ); Fri, 10 Jul 2015 12:00:11 -0400 MIME-Version: 1.0 In-Reply-To: <20150710123831.GA7877@vaishali-Ideapad-Z570> References: <20150710123831.GA7877@vaishali-Ideapad-Z570> Date: Fri, 10 Jul 2015 10:00:11 -0600 Message-ID: Subject: Re: [PATCH v3] coresight: replicator: Use builtin_platform_driver() From: Mathieu Poirier To: Vaishali Thakkar Cc: Paul Bolle , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2397 Lines: 67 On 10 July 2015 at 06:38, Vaishali Thakkar wrote: > Macro builtin_platform_driver can be used for builtin drivers > that don't do anything in driver init. This file depends on > Kconfig CONFIG_CORESIGHT_LINKS_AND_SINKS which eventually > depends on CORESIGHT. Both CONFIG_CORESIGHT_LINKS_AND_SINKS and > CORESIGHT are bool. So, use builtin_platform_driver and remove > some boilerplate code. > > Also, remove header file init.h as functionality like module_init > and module_exit is now relocated to module.h. > > Signed-off-by: Vaishali Thakkar > Suggested-by: Paul Bolle > --- > Changes since v1: > - As per Mathieu's suggestion, remove the comment and > code of Coccinelle > - Also, add blank line before module_platform_driver > Changes since v2: > - Use builtin_platform_driver instead of > module_platform_driver and change commit log and subject > accordingly. > --- > drivers/hwtracing/coresight/coresight-replicator.c | 13 +------------ > 1 file changed, 1 insertion(+), 12 deletions(-) > > diff --git a/drivers/hwtracing/coresight/coresight-replicator.c b/drivers/hwtracing/coresight/coresight-replicator.c > index 7974b7c..963ac19 100644 > --- a/drivers/hwtracing/coresight/coresight-replicator.c > +++ b/drivers/hwtracing/coresight/coresight-replicator.c > @@ -12,7 +12,6 @@ > > #include > #include > -#include > #include > #include > #include > @@ -184,17 +183,7 @@ static struct platform_driver replicator_driver = { > }, > }; > > -static int __init replicator_init(void) > -{ > - return platform_driver_register(&replicator_driver); > -} > -module_init(replicator_init); > - > -static void __exit replicator_exit(void) > -{ > - platform_driver_unregister(&replicator_driver); > -} > -module_exit(replicator_exit); > +builtin_platform_driver(replicator_driver); > > MODULE_LICENSE("GPL v2"); > MODULE_DESCRIPTION("CoreSight Replicator driver"); > -- > 1.9.1 > Applied, thanks. -- 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/