Hi Sean,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on v5.8 next-20200807]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url:
https://github.com/0day-ci/linux/commits/Sean-Young/media-rc-uevent-sysfs...
base:
git://linuxtv.org/media_tree.git master
config: x86_64-randconfig-m001-20200808 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
smatch warnings:
drivers/media/rc/rc-main.c:1630 rc_dev_uevent() warn: inconsistent returns
'dev->lock'.
vim +1630 drivers/media/rc/rc-main.c
d8b4b5822f51e2 David Härdeman 2010-10-29 1603
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1604 #define ADD_HOTPLUG_VAR(fmt,
val...) \
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1605 do { \
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1606 int err = add_uevent_var(env,
fmt, val); \
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1607 if (err) \
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1608 return err; \
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1609 } while (0)
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1610
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1611 static int rc_dev_uevent(struct
device *device, struct kobj_uevent_env *env)
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1612 {
d8b4b5822f51e2 David Härdeman 2010-10-29 1613 struct rc_dev *dev =
to_rc_dev(device);
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1614
e4b7677d34e789 Sean Young 2020-08-08 1615 mutex_lock(&dev->lock);
e4b7677d34e789 Sean Young 2020-08-08 1616 if (!dev->registered) {
e4b7677d34e789 Sean Young 2020-08-08 1617 mutex_unlock(&dev->lock);
e4b7677d34e789 Sean Young 2020-08-08 1618 return -ENODEV;
e4b7677d34e789 Sean Young 2020-08-08 1619 }
e4b7677d34e789 Sean Young 2020-08-08 1620
b088ba658b3438 Mauro Carvalho Chehab 2010-11-17 1621 if (dev->rc_map.name)
b088ba658b3438 Mauro Carvalho Chehab 2010-11-17 1622
ADD_HOTPLUG_VAR("NAME=%s", dev->rc_map.name);
d8b4b5822f51e2 David Härdeman 2010-10-29 1623 if (dev->driver_name)
d8b4b5822f51e2 David Härdeman 2010-10-29 1624
ADD_HOTPLUG_VAR("DRV_NAME=%s", dev->driver_name);
b9f407e31c5073 Sean Young 2017-09-01 1625 if (dev->device_name)
b9f407e31c5073 Sean Young 2017-09-01 1626
ADD_HOTPLUG_VAR("DEV_NAME=%s", dev->device_name);
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1627
e4b7677d34e789 Sean Young 2020-08-08 1628 mutex_unlock(&dev->lock);
e4b7677d34e789 Sean Young 2020-08-08 1629
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 @1630 return 0;
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1631 }
bc2a6c5719efd7 Mauro Carvalho Chehab 2010-11-09 1632
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org