Commit 43266337 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

V4L/DVB (10181): v4l2-device: Fix some sparse warnings

/home/v4l/master/v4l/v4l2-device.c:32:2: warning: Using plain integer as NULL pointer
/home/v4l/master/v4l/v4l2-device.c:64:2: warning: Using plain integer as NULL pointer
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent ffbc5f88
...@@ -29,7 +29,7 @@ int v4l2_device_register(struct device *dev, struct v4l2_device *v4l2_dev) ...@@ -29,7 +29,7 @@ int v4l2_device_register(struct device *dev, struct v4l2_device *v4l2_dev)
if (dev == NULL || v4l2_dev == NULL) if (dev == NULL || v4l2_dev == NULL)
return -EINVAL; return -EINVAL;
/* Warn if we apparently re-register a device */ /* Warn if we apparently re-register a device */
WARN_ON(dev_get_drvdata(dev)); WARN_ON(dev_get_drvdata(dev) != NULL);
INIT_LIST_HEAD(&v4l2_dev->subdevs); INIT_LIST_HEAD(&v4l2_dev->subdevs);
spin_lock_init(&v4l2_dev->lock); spin_lock_init(&v4l2_dev->lock);
v4l2_dev->dev = dev; v4l2_dev->dev = dev;
...@@ -61,7 +61,7 @@ int v4l2_device_register_subdev(struct v4l2_device *dev, struct v4l2_subdev *sd) ...@@ -61,7 +61,7 @@ int v4l2_device_register_subdev(struct v4l2_device *dev, struct v4l2_subdev *sd)
if (dev == NULL || sd == NULL || !sd->name[0]) if (dev == NULL || sd == NULL || !sd->name[0])
return -EINVAL; return -EINVAL;
/* Warn if we apparently re-register a subdev */ /* Warn if we apparently re-register a subdev */
WARN_ON(sd->dev); WARN_ON(sd->dev != NULL);
if (!try_module_get(sd->owner)) if (!try_module_get(sd->owner))
return -ENODEV; return -ENODEV;
sd->dev = dev; sd->dev = dev;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment