Messages in this thread | | | Date | Wed, 5 Dec 2018 11:39:19 +0000 | From | Charles Keepax <> | Subject | Re: [PATCH 20/22] mfd: wm8350-i2c: Make it explicitly non-modular |
| |
On Sun, Dec 02, 2018 at 11:23:27PM -0500, Paul Gortmaker wrote: > The Kconfig currently controlling compilation of this code is: > > drivers/mfd/Kconfig:config MFD_WM8350_I2C > drivers/mfd/Kconfig: bool "Wolfson Microelectronics WM8350 with I2C" > > ...meaning that it currently is not being built as a module by anyone. > > Lets remove the modular code that is essentially orphaned, so that > when reading the driver there is no doubt it is builtin-only. > > We 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 don't replace module.h with init.h since the file already has that. > But we do delete an unused moduleparam.h > > 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: Lee Jones <lee.jones@linaro.org> > Cc: patches@opensource.cirrus.com > Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> > --- > -static int wm8350_i2c_remove(struct i2c_client *i2c) > -{ > - struct wm8350 *wm8350 = i2c_get_clientdata(i2c); > - > - wm8350_device_exit(wm8350); > - > - return 0; > -}
Likewise here it removes the only caller of wm8350_device_exit.
Thanks, Charles
| |