[media] radio-si4713: embed struct video_device instead of allocating it
authorHans Verkuil <hans.verkuil@cisco.com>
Mon, 8 Apr 2013 09:11:12 +0000 (06:11 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 14 Apr 2013 22:49:25 +0000 (19:49 -0300)
Also set the v4l2_dev pointer in struct video_device as this was missing.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Eduardo Valentin <edubezval@gmail.com>
Tested-by: Eduardo Valentin <edubezval@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/radio/radio-si4713.c

index 320f3012c850c79da06948a7dca099b5399970dd..70dc6527a12c9be467a78bec5517fa2eb4613e45 100644 (file)
@@ -48,7 +48,7 @@ MODULE_ALIAS("platform:radio-si4713");
 /* Driver state struct */
 struct radio_si4713_device {
        struct v4l2_device              v4l2_dev;
-       struct video_device             *radio_dev;
+       struct video_device             radio_dev;
 };
 
 /* radio_si4713_fops - file operations interface */
@@ -217,7 +217,7 @@ static struct v4l2_ioctl_ops radio_si4713_ioctl_ops = {
 static struct video_device radio_si4713_vdev_template = {
        .fops                   = &radio_si4713_fops,
        .name                   = "radio-si4713",
-       .release                = video_device_release,
+       .release                = video_device_release_empty,
        .ioctl_ops              = &radio_si4713_ioctl_ops,
        .vfl_dir                = VFL_DIR_TX,
 };
@@ -267,27 +267,18 @@ static int radio_si4713_pdriver_probe(struct platform_device *pdev)
                goto put_adapter;
        }
 
-       rsdev->radio_dev = video_device_alloc();
-       if (!rsdev->radio_dev) {
-               dev_err(&pdev->dev, "Failed to alloc video device.\n");
-               rval = -ENOMEM;
-               goto put_adapter;
-       }
-
-       memcpy(rsdev->radio_dev, &radio_si4713_vdev_template,
-              sizeof(radio_si4713_vdev_template));
-       video_set_drvdata(rsdev->radio_dev, rsdev);
-       if (video_register_device(rsdev->radio_dev, VFL_TYPE_RADIO, radio_nr)) {
+       rsdev->radio_dev = radio_si4713_vdev_template;
+       rsdev->radio_dev.v4l2_dev = &rsdev->v4l2_dev;
+       video_set_drvdata(&rsdev->radio_dev, rsdev);
+       if (video_register_device(&rsdev->radio_dev, VFL_TYPE_RADIO, radio_nr)) {
                dev_err(&pdev->dev, "Could not register video device.\n");
                rval = -EIO;
-               goto free_vdev;
+               goto put_adapter;
        }
        dev_info(&pdev->dev, "New device successfully probed\n");
 
        goto exit;
 
-free_vdev:
-       video_device_release(rsdev->radio_dev);
 put_adapter:
        i2c_put_adapter(adapter);
 unregister_v4l2_dev:
@@ -306,7 +297,7 @@ static int radio_si4713_pdriver_remove(struct platform_device *pdev)
        struct radio_si4713_device *rsdev;
 
        rsdev = container_of(v4l2_dev, struct radio_si4713_device, v4l2_dev);
-       video_unregister_device(rsdev->radio_dev);
+       video_unregister_device(&rsdev->radio_dev);
        i2c_put_adapter(client->adapter);
        v4l2_device_unregister(&rsdev->v4l2_dev);