Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751340AbdFETai (ORCPT ); Mon, 5 Jun 2017 15:30:38 -0400 Received: from mail.windriver.com ([147.11.1.11]:47644 "EHLO mail.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751077AbdFETag (ORCPT ); Mon, 5 Jun 2017 15:30:36 -0400 Date: Mon, 5 Jun 2017 15:29:44 -0400 From: Paul Gortmaker To: Steve Twiss CC: Support Opensource , Lee Jones , Eric Miao , Mike Rapoport , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 1/4] mfd: da903x: Make it explicitly non-modular Message-ID: <20170605192944.GI29293@windriver.com> References: <20170603130351.13913-1-paul.gortmaker@windriver.com> <20170603130351.13913-2-paul.gortmaker@windriver.com> <6ED8E3B22081A4459DAC7699F3695FB7018CD91D68@SW-EX-MBX02.diasemi.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <6ED8E3B22081A4459DAC7699F3695FB7018CD91D68@SW-EX-MBX02.diasemi.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4584 Lines: 136 [RE: [PATCH 1/4] mfd: da903x: Make it explicitly non-modular] On 05/06/2017 (Mon 10:30) Steve Twiss wrote: > Hi Paul, > > On 03 June 2017 14:04 Paul Gortmaker wrote: > > > To: linux-kernel@vger.kernel.org > > Cc: Paul Gortmaker; Support Opensource; Lee Jones; Eric Miao; Mike Rapoport > > Subject: [PATCH 1/4] mfd: da903x: Make it explicitly non-modular > > > > The Kconfig currently controlling compilation of this code is: > > > > drivers/mfd/Kconfig:config PMIC_DA903X > > drivers/mfd/Kconfig: bool "Dialog Semiconductor DA9030/DA9034 PMIC > > Support" > > > > ...meaning that it currently is not being built as a module by anyone. > > With regards to your four patches, > > [4/4] mfd: da9055-i2c: Make it explicitly non-modular - - - 2017-06-03 > [3/4] mfd: da9055-core: make it explicitly non-modular - - - 2017-06-03 > [2/4] mfd: da9052-*: Make it explicitly non-modular - - - 2017-06-03 > [1/4] mfd: da903x: Make it explicitly non-modular - - - 2017-06-03 > > Is there an reason not to make the Kconfig tristate? > Would that not be simpler? I explicitly covered that in the 0/4 e-mail: As always, the option exists for someone with the hardware and the desire to extend the functionality to make any given driver tristate. But given the number of these tree wide and the fact that I can't test that new extended functionality in all cases, I just make the code consistent with existing Kconfig/Makefile settings that currently restrict them to "bool". Thanks, Paul. -- > > Previously, something similar was suggested by Geert Uytterhoeven for DA9063: > https://patchwork.kernel.org/patch/7788001/ > > Regards, > Steve > > > Remove the modular code that is essentially orphaned, so that > > when reading the driver there is no doubt it is builtin-only. > > > > Also explicitly disallow a driver unbind, since that doesn't have a > > sensible use case anyway, and it allows us to drop the ".remove" > > code for non-modular drivers. > > > > Since module_init was not in use by this code, the init ordering > > remains unchanged with this commit. > > > > We replace module.h with init.h and export.h ; the latter since this > > file does export some syms. > > > > Also note that MODULE_DEVICE_TABLE is a no-op for non-modular code. > > > > We also delete the MODULE_LICENSE tag etc. since all that information > > is already contained at the top of the file in the comments. > > > > Cc: Support Opensource > > Cc: Lee Jones > > Cc: Eric Miao > > Cc: Mike Rapoport > > Signed-off-by: Paul Gortmaker > > --- > > drivers/mfd/da903x.c | 26 +++----------------------- > > 1 file changed, 3 insertions(+), 23 deletions(-) > > > > diff --git a/drivers/mfd/da903x.c b/drivers/mfd/da903x.c > > index 09f367571c58..78edb0558a0f 100644 > > --- a/drivers/mfd/da903x.c > > +++ b/drivers/mfd/da903x.c > > @@ -13,7 +13,7 @@ > > */ > > > > #include > > -#include > > +#include > > #include > > #include > > #include > > @@ -446,7 +446,6 @@ static const struct i2c_device_id da903x_id_table[] = { > > { "da9034", 1 }, > > { }, > > }; > > -MODULE_DEVICE_TABLE(i2c, da903x_id_table); > > > > static int __remove_subdev(struct device *dev, void *unused) > > { > > @@ -535,20 +534,12 @@ static int da903x_probe(struct i2c_client *client, > > return da903x_add_subdevs(chip, pdata); > > } > > > > -static int da903x_remove(struct i2c_client *client) > > -{ > > - struct da903x_chip *chip = i2c_get_clientdata(client); > > - > > - da903x_remove_subdevs(chip); > > - return 0; > > -} > > - > > static struct i2c_driver da903x_driver = { > > .driver = { > > - .name = "da903x", > > + .name = "da903x", > > + .suppress_bind_attrs = true, > > }, > > .probe = da903x_probe, > > - .remove = da903x_remove, > > .id_table = da903x_id_table, > > }; > > > > @@ -557,14 +548,3 @@ static int __init da903x_init(void) > > return i2c_add_driver(&da903x_driver); > > } > > subsys_initcall(da903x_init); > > - > > -static void __exit da903x_exit(void) > > -{ > > - i2c_del_driver(&da903x_driver); > > -} > > -module_exit(da903x_exit); > > - > > -MODULE_DESCRIPTION("PMIC Driver for Dialog Semiconductor DA9034"); > > -MODULE_AUTHOR("Eric Miao "); > > -MODULE_AUTHOR("Mike Rapoport "); > > -MODULE_LICENSE("GPL v2"); > > -- > > 2.11.0 >