On Tue, Jul 21, 2020 at 04:40:23PM +0800, Zheng Yongjun wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/base/module.c: In function 'module_add_driver': > drivers/base/module.c:33: warning: > variable 'no_warn' set but not used [-Wunused-but-set-variable] > > Signed-off-by: Zheng Yongjun <[email protected]> > --- > drivers/base/module.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/base/module.c b/drivers/base/module.c > index 46ad4d636731..d1060c504c1b 100644 > --- a/drivers/base/module.c > +++ b/drivers/base/module.c > @@ -33,7 +33,6 @@ static void module_create_drivers_dir(struct module_kobject > *mk) > void module_add_driver(struct module *mod, struct device_driver *drv) > { > char *driver_name; > - int no_warn; > struct module_kobject *mk = NULL; > > if (!drv) > @@ -59,11 +58,11 @@ void module_add_driver(struct module *mod, struct > device_driver *drv) > return; > > /* Don't check return codes; these calls are idempotent */ > - no_warn = sysfs_create_link(&drv->p->kobj, &mk->kobj, "module"); > + sysfs_create_link(&drv->p->kobj, &mk->kobj, "module");
Always test-build your patches before sending them out :(
