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 <[email protected]> > Cc: [email protected] > Signed-off-by: Paul Gortmaker <[email protected]> > --- > -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
