Commit 9832e155 authored by Javier Martinez Canillas's avatar Javier Martinez Canillas Committed by Mauro Carvalho Chehab

[media] media-device: split media initialization and registration

The media device node is registered and so made visible to user-space
before entities are registered and links created which means that the
media graph obtained by user-space could be only partially enumerated
if that happens too early before all the graph has been created.

To avoid this race condition, split the media init and registration
in separate functions and only register the media device node when
all the pending subdevices have been registered, either explicitly
by the driver or asynchronously using v4l2_async_register_subdev().

The media_device_register() had a check for drivers not filling dev
and model fields but all drivers in mainline set them and not doing
it will be a driver bug so change the function return to void and
add a BUG_ON() for dev being NULL instead.

Also, add a media_device_cleanup() function that will destroy the
graph_mutex that is initialized in media_device_init().

[mchehab@osg.samsung.com: Fix compilation if !CONFIG_MEDIA_CONTROLLER
 and remove two warnings added by this changeset]
Suggested-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarJavier Martinez Canillas <javier@osg.samsung.com>
Acked-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent 821ed366
...@@ -617,6 +617,7 @@ static void smsdvb_media_device_unregister(struct smsdvb_client_t *client) ...@@ -617,6 +617,7 @@ static void smsdvb_media_device_unregister(struct smsdvb_client_t *client)
if (!coredev->media_dev) if (!coredev->media_dev)
return; return;
media_device_unregister(coredev->media_dev); media_device_unregister(coredev->media_dev);
media_device_cleanup(coredev->media_dev);
kfree(coredev->media_dev); kfree(coredev->media_dev);
coredev->media_dev = NULL; coredev->media_dev = NULL;
#endif #endif
......
...@@ -616,7 +616,7 @@ EXPORT_SYMBOL_GPL(media_device_unregister_entity); ...@@ -616,7 +616,7 @@ EXPORT_SYMBOL_GPL(media_device_unregister_entity);
/** /**
* media_device_register - register a media device * media_device_init() - initialize a media device
* @mdev: The media device * @mdev: The media device
* *
* The caller is responsible for initializing the media device before * The caller is responsible for initializing the media device before
...@@ -625,14 +625,8 @@ EXPORT_SYMBOL_GPL(media_device_unregister_entity); ...@@ -625,14 +625,8 @@ EXPORT_SYMBOL_GPL(media_device_unregister_entity);
* - dev must point to the parent device * - dev must point to the parent device
* - model must be filled with the device model name * - model must be filled with the device model name
*/ */
int __must_check __media_device_register(struct media_device *mdev, void media_device_init(struct media_device *mdev)
struct module *owner)
{ {
int ret;
if (WARN_ON(mdev->dev == NULL || mdev->model[0] == 0))
return -EINVAL;
INIT_LIST_HEAD(&mdev->entities); INIT_LIST_HEAD(&mdev->entities);
INIT_LIST_HEAD(&mdev->interfaces); INIT_LIST_HEAD(&mdev->interfaces);
INIT_LIST_HEAD(&mdev->pads); INIT_LIST_HEAD(&mdev->pads);
...@@ -640,6 +634,33 @@ int __must_check __media_device_register(struct media_device *mdev, ...@@ -640,6 +634,33 @@ int __must_check __media_device_register(struct media_device *mdev,
spin_lock_init(&mdev->lock); spin_lock_init(&mdev->lock);
mutex_init(&mdev->graph_mutex); mutex_init(&mdev->graph_mutex);
dev_dbg(mdev->dev, "Media device initialized\n");
}
EXPORT_SYMBOL_GPL(media_device_init);
/**
* media_device_cleanup() - Cleanup a media device
* @mdev: The media device
*
*/
void media_device_cleanup(struct media_device *mdev)
{
mutex_destroy(&mdev->graph_mutex);
}
EXPORT_SYMBOL_GPL(media_device_cleanup);
/**
* __media_device_register() - register a media device
* @mdev: The media device
* @owner: The module owner
*
* returns zero on success or a negative error code.
*/
int __must_check __media_device_register(struct media_device *mdev,
struct module *owner)
{
int ret;
/* Register the device node. */ /* Register the device node. */
mdev->devnode.fops = &media_device_fops; mdev->devnode.fops = &media_device_fops;
mdev->devnode.parent = mdev->dev; mdev->devnode.parent = mdev->dev;
......
...@@ -1313,7 +1313,10 @@ static int subdev_notifier_complete(struct v4l2_async_notifier *notifier) ...@@ -1313,7 +1313,10 @@ static int subdev_notifier_complete(struct v4l2_async_notifier *notifier)
ret = v4l2_device_register_subdev_nodes(&fmd->v4l2_dev); ret = v4l2_device_register_subdev_nodes(&fmd->v4l2_dev);
unlock: unlock:
mutex_unlock(&fmd->media_dev.graph_mutex); mutex_unlock(&fmd->media_dev.graph_mutex);
if (ret < 0)
return ret; return ret;
return media_device_register(&fmd->media_dev);
} }
static int fimc_md_probe(struct platform_device *pdev) static int fimc_md_probe(struct platform_device *pdev)
...@@ -1350,11 +1353,7 @@ static int fimc_md_probe(struct platform_device *pdev) ...@@ -1350,11 +1353,7 @@ static int fimc_md_probe(struct platform_device *pdev)
return ret; return ret;
} }
ret = media_device_register(&fmd->media_dev); media_device_init(&fmd->media_dev);
if (ret < 0) {
v4l2_err(v4l2_dev, "Failed to register media device: %d\n", ret);
goto err_v4l2_dev;
}
ret = fimc_md_get_clocks(fmd); ret = fimc_md_get_clocks(fmd);
if (ret) if (ret)
...@@ -1424,8 +1423,7 @@ static int fimc_md_probe(struct platform_device *pdev) ...@@ -1424,8 +1423,7 @@ static int fimc_md_probe(struct platform_device *pdev)
err_m_ent: err_m_ent:
fimc_md_unregister_entities(fmd); fimc_md_unregister_entities(fmd);
err_md: err_md:
media_device_unregister(&fmd->media_dev); media_device_cleanup(&fmd->media_dev);
err_v4l2_dev:
v4l2_device_unregister(&fmd->v4l2_dev); v4l2_device_unregister(&fmd->v4l2_dev);
return ret; return ret;
} }
...@@ -1445,6 +1443,7 @@ static int fimc_md_remove(struct platform_device *pdev) ...@@ -1445,6 +1443,7 @@ static int fimc_md_remove(struct platform_device *pdev)
fimc_md_unregister_entities(fmd); fimc_md_unregister_entities(fmd);
fimc_md_pipelines_free(fmd); fimc_md_pipelines_free(fmd);
media_device_unregister(&fmd->media_dev); media_device_unregister(&fmd->media_dev);
media_device_cleanup(&fmd->media_dev);
fimc_md_put_clocks(fmd); fimc_md_put_clocks(fmd);
return 0; return 0;
......
...@@ -1793,6 +1793,7 @@ static void isp_unregister_entities(struct isp_device *isp) ...@@ -1793,6 +1793,7 @@ static void isp_unregister_entities(struct isp_device *isp)
v4l2_device_unregister(&isp->v4l2_dev); v4l2_device_unregister(&isp->v4l2_dev);
media_device_unregister(&isp->media_dev); media_device_unregister(&isp->media_dev);
media_device_cleanup(&isp->media_dev);
} }
static int isp_link_entity( static int isp_link_entity(
...@@ -1875,12 +1876,7 @@ static int isp_register_entities(struct isp_device *isp) ...@@ -1875,12 +1876,7 @@ static int isp_register_entities(struct isp_device *isp)
sizeof(isp->media_dev.model)); sizeof(isp->media_dev.model));
isp->media_dev.hw_revision = isp->revision; isp->media_dev.hw_revision = isp->revision;
isp->media_dev.link_notify = isp_pipeline_link_notify; isp->media_dev.link_notify = isp_pipeline_link_notify;
ret = media_device_register(&isp->media_dev); media_device_init(&isp->media_dev);
if (ret < 0) {
dev_err(isp->dev, "%s: Media device registration failed (%d)\n",
__func__, ret);
return ret;
}
isp->v4l2_dev.mdev = &isp->media_dev; isp->v4l2_dev.mdev = &isp->media_dev;
ret = v4l2_device_register(isp->dev, &isp->v4l2_dev); ret = v4l2_device_register(isp->dev, &isp->v4l2_dev);
...@@ -2365,7 +2361,11 @@ static int isp_subdev_notifier_complete(struct v4l2_async_notifier *async) ...@@ -2365,7 +2361,11 @@ static int isp_subdev_notifier_complete(struct v4l2_async_notifier *async)
} }
} }
return v4l2_device_register_subdev_nodes(v4l2_dev); ret = v4l2_device_register_subdev_nodes(&isp->v4l2_dev);
if (ret < 0)
return ret;
return media_device_register(&isp->media_dev);
} }
/* /*
......
...@@ -305,7 +305,7 @@ static void camif_unregister_media_entities(struct camif_dev *camif) ...@@ -305,7 +305,7 @@ static void camif_unregister_media_entities(struct camif_dev *camif)
/* /*
* Media device * Media device
*/ */
static int camif_media_dev_register(struct camif_dev *camif) static int camif_media_dev_init(struct camif_dev *camif)
{ {
struct media_device *md = &camif->media_dev; struct media_device *md = &camif->media_dev;
struct v4l2_device *v4l2_dev = &camif->v4l2_dev; struct v4l2_device *v4l2_dev = &camif->v4l2_dev;
...@@ -328,9 +328,7 @@ static int camif_media_dev_register(struct camif_dev *camif) ...@@ -328,9 +328,7 @@ static int camif_media_dev_register(struct camif_dev *camif)
if (ret < 0) if (ret < 0)
return ret; return ret;
ret = media_device_register(md); media_device_init(md);
if (ret < 0)
v4l2_device_unregister(v4l2_dev);
return ret; return ret;
} }
...@@ -483,7 +481,7 @@ static int s3c_camif_probe(struct platform_device *pdev) ...@@ -483,7 +481,7 @@ static int s3c_camif_probe(struct platform_device *pdev)
goto err_alloc; goto err_alloc;
} }
ret = camif_media_dev_register(camif); ret = camif_media_dev_init(camif);
if (ret < 0) if (ret < 0)
goto err_mdev; goto err_mdev;
...@@ -510,6 +508,11 @@ static int s3c_camif_probe(struct platform_device *pdev) ...@@ -510,6 +508,11 @@ static int s3c_camif_probe(struct platform_device *pdev)
goto err_unlock; goto err_unlock;
mutex_unlock(&camif->media_dev.graph_mutex); mutex_unlock(&camif->media_dev.graph_mutex);
ret = media_device_register(&camif->media_dev);
if (ret < 0)
goto err_sens;
pm_runtime_put(dev); pm_runtime_put(dev);
return 0; return 0;
...@@ -518,6 +521,7 @@ static int s3c_camif_probe(struct platform_device *pdev) ...@@ -518,6 +521,7 @@ static int s3c_camif_probe(struct platform_device *pdev)
err_sens: err_sens:
v4l2_device_unregister(&camif->v4l2_dev); v4l2_device_unregister(&camif->v4l2_dev);
media_device_unregister(&camif->media_dev); media_device_unregister(&camif->media_dev);
media_device_cleanup(&camif->media_dev);
camif_unregister_media_entities(camif); camif_unregister_media_entities(camif);
err_mdev: err_mdev:
vb2_dma_contig_cleanup_ctx(camif->alloc_ctx); vb2_dma_contig_cleanup_ctx(camif->alloc_ctx);
...@@ -539,6 +543,7 @@ static int s3c_camif_remove(struct platform_device *pdev) ...@@ -539,6 +543,7 @@ static int s3c_camif_remove(struct platform_device *pdev)
struct s3c_camif_plat_data *pdata = &camif->pdata; struct s3c_camif_plat_data *pdata = &camif->pdata;
media_device_unregister(&camif->media_dev); media_device_unregister(&camif->media_dev);
media_device_cleanup(&camif->media_dev);
camif_unregister_media_entities(camif); camif_unregister_media_entities(camif);
v4l2_device_unregister(&camif->v4l2_dev); v4l2_device_unregister(&camif->v4l2_dev);
......
...@@ -127,6 +127,7 @@ static void vsp1_destroy_entities(struct vsp1_device *vsp1) ...@@ -127,6 +127,7 @@ static void vsp1_destroy_entities(struct vsp1_device *vsp1)
v4l2_device_unregister(&vsp1->v4l2_dev); v4l2_device_unregister(&vsp1->v4l2_dev);
media_device_unregister(&vsp1->media_dev); media_device_unregister(&vsp1->media_dev);
media_device_cleanup(&vsp1->media_dev);
} }
static int vsp1_create_entities(struct vsp1_device *vsp1) static int vsp1_create_entities(struct vsp1_device *vsp1)
...@@ -141,12 +142,7 @@ static int vsp1_create_entities(struct vsp1_device *vsp1) ...@@ -141,12 +142,7 @@ static int vsp1_create_entities(struct vsp1_device *vsp1)
strlcpy(mdev->model, "VSP1", sizeof(mdev->model)); strlcpy(mdev->model, "VSP1", sizeof(mdev->model));
snprintf(mdev->bus_info, sizeof(mdev->bus_info), "platform:%s", snprintf(mdev->bus_info, sizeof(mdev->bus_info), "platform:%s",
dev_name(mdev->dev)); dev_name(mdev->dev));
ret = media_device_register(mdev); media_device_init(mdev);
if (ret < 0) {
dev_err(vsp1->dev, "media device registration failed (%d)\n",
ret);
return ret;
}
vdev->mdev = mdev; vdev->mdev = mdev;
ret = v4l2_device_register(vsp1->dev, vdev); ret = v4l2_device_register(vsp1->dev, vdev);
...@@ -284,6 +280,10 @@ static int vsp1_create_entities(struct vsp1_device *vsp1) ...@@ -284,6 +280,10 @@ static int vsp1_create_entities(struct vsp1_device *vsp1)
} }
ret = v4l2_device_register_subdev_nodes(&vsp1->v4l2_dev); ret = v4l2_device_register_subdev_nodes(&vsp1->v4l2_dev);
if (ret < 0)
goto done;
ret = media_device_register(mdev);
done: done:
if (ret < 0) if (ret < 0)
......
...@@ -311,7 +311,7 @@ static int xvip_graph_notify_complete(struct v4l2_async_notifier *notifier) ...@@ -311,7 +311,7 @@ static int xvip_graph_notify_complete(struct v4l2_async_notifier *notifier)
if (ret < 0) if (ret < 0)
dev_err(xdev->dev, "failed to register subdev nodes\n"); dev_err(xdev->dev, "failed to register subdev nodes\n");
return ret; return media_device_register(&xdev->media_dev);
} }
static int xvip_graph_notify_bound(struct v4l2_async_notifier *notifier, static int xvip_graph_notify_bound(struct v4l2_async_notifier *notifier,
...@@ -573,6 +573,7 @@ static void xvip_composite_v4l2_cleanup(struct xvip_composite_device *xdev) ...@@ -573,6 +573,7 @@ static void xvip_composite_v4l2_cleanup(struct xvip_composite_device *xdev)
{ {
v4l2_device_unregister(&xdev->v4l2_dev); v4l2_device_unregister(&xdev->v4l2_dev);
media_device_unregister(&xdev->media_dev); media_device_unregister(&xdev->media_dev);
media_device_cleanup(&xdev->media_dev);
} }
static int xvip_composite_v4l2_init(struct xvip_composite_device *xdev) static int xvip_composite_v4l2_init(struct xvip_composite_device *xdev)
...@@ -584,19 +585,14 @@ static int xvip_composite_v4l2_init(struct xvip_composite_device *xdev) ...@@ -584,19 +585,14 @@ static int xvip_composite_v4l2_init(struct xvip_composite_device *xdev)
sizeof(xdev->media_dev.model)); sizeof(xdev->media_dev.model));
xdev->media_dev.hw_revision = 0; xdev->media_dev.hw_revision = 0;
ret = media_device_register(&xdev->media_dev); media_device_init(&xdev->media_dev);
if (ret < 0) {
dev_err(xdev->dev, "media device registration failed (%d)\n",
ret);
return ret;
}
xdev->v4l2_dev.mdev = &xdev->media_dev; xdev->v4l2_dev.mdev = &xdev->media_dev;
ret = v4l2_device_register(xdev->dev, &xdev->v4l2_dev); ret = v4l2_device_register(xdev->dev, &xdev->v4l2_dev);
if (ret < 0) { if (ret < 0) {
dev_err(xdev->dev, "V4L2 device registration failed (%d)\n", dev_err(xdev->dev, "V4L2 device registration failed (%d)\n",
ret); ret);
media_device_unregister(&xdev->media_dev); media_device_cleanup(&xdev->media_dev);
return ret; return ret;
} }
......
...@@ -136,6 +136,7 @@ static void au0828_unregister_media_device(struct au0828_dev *dev) ...@@ -136,6 +136,7 @@ static void au0828_unregister_media_device(struct au0828_dev *dev)
#ifdef CONFIG_MEDIA_CONTROLLER #ifdef CONFIG_MEDIA_CONTROLLER
if (dev->media_dev) { if (dev->media_dev) {
media_device_unregister(dev->media_dev); media_device_unregister(dev->media_dev);
media_device_cleanup(dev->media_dev);
kfree(dev->media_dev); kfree(dev->media_dev);
dev->media_dev = NULL; dev->media_dev = NULL;
} }
...@@ -216,12 +217,11 @@ static void au0828_usb_disconnect(struct usb_interface *interface) ...@@ -216,12 +217,11 @@ static void au0828_usb_disconnect(struct usb_interface *interface)
au0828_usb_release(dev); au0828_usb_release(dev);
} }
static void au0828_media_device_register(struct au0828_dev *dev, static void au0828_media_device_init(struct au0828_dev *dev,
struct usb_device *udev) struct usb_device *udev)
{ {
#ifdef CONFIG_MEDIA_CONTROLLER #ifdef CONFIG_MEDIA_CONTROLLER
struct media_device *mdev; struct media_device *mdev;
int ret;
mdev = kzalloc(sizeof(*mdev), GFP_KERNEL); mdev = kzalloc(sizeof(*mdev), GFP_KERNEL);
if (!mdev) if (!mdev)
...@@ -239,14 +239,7 @@ static void au0828_media_device_register(struct au0828_dev *dev, ...@@ -239,14 +239,7 @@ static void au0828_media_device_register(struct au0828_dev *dev,
mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice); mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice);
mdev->driver_version = LINUX_VERSION_CODE; mdev->driver_version = LINUX_VERSION_CODE;
ret = media_device_register(mdev); media_device_init(mdev);
if (ret) {
pr_err(
"Couldn't create a media device. Error: %d\n",
ret);
kfree(mdev);
return;
}
dev->media_dev = mdev; dev->media_dev = mdev;
#endif #endif
...@@ -374,8 +367,8 @@ static int au0828_usb_probe(struct usb_interface *interface, ...@@ -374,8 +367,8 @@ static int au0828_usb_probe(struct usb_interface *interface,
dev->boardnr = id->driver_info; dev->boardnr = id->driver_info;
dev->board = au0828_boards[dev->boardnr]; dev->board = au0828_boards[dev->boardnr];
/* Register the media controller */ /* Initialize the media controller */
au0828_media_device_register(dev, usbdev); au0828_media_device_init(dev, usbdev);
#ifdef CONFIG_VIDEO_AU0828_V4L2 #ifdef CONFIG_VIDEO_AU0828_V4L2
dev->v4l2_dev.release = au0828_usb_v4l2_release; dev->v4l2_dev.release = au0828_usb_v4l2_release;
...@@ -446,9 +439,17 @@ static int au0828_usb_probe(struct usb_interface *interface, ...@@ -446,9 +439,17 @@ static int au0828_usb_probe(struct usb_interface *interface,
if (retval) { if (retval) {
pr_err("%s() au0282_dev_register failed to create graph\n", pr_err("%s() au0282_dev_register failed to create graph\n",
__func__); __func__);
au0828_usb_disconnect(interface); goto done;
} }
#ifdef CONFIG_MEDIA_CONTROLLER
retval = media_device_register(dev->media_dev);
#endif
done:
if (retval < 0)
au0828_usb_disconnect(interface);
return retval; return retval;
} }
......
...@@ -1172,6 +1172,7 @@ static void cx231xx_unregister_media_device(struct cx231xx *dev) ...@@ -1172,6 +1172,7 @@ static void cx231xx_unregister_media_device(struct cx231xx *dev)
#ifdef CONFIG_MEDIA_CONTROLLER #ifdef CONFIG_MEDIA_CONTROLLER
if (dev->media_dev) { if (dev->media_dev) {
media_device_unregister(dev->media_dev); media_device_unregister(dev->media_dev);
media_device_cleanup(dev->media_dev);
kfree(dev->media_dev); kfree(dev->media_dev);
dev->media_dev = NULL; dev->media_dev = NULL;
} }
...@@ -1205,12 +1206,11 @@ void cx231xx_release_resources(struct cx231xx *dev) ...@@ -1205,12 +1206,11 @@ void cx231xx_release_resources(struct cx231xx *dev)
clear_bit(dev->devno, &cx231xx_devused); clear_bit(dev->devno, &cx231xx_devused);
} }
static void cx231xx_media_device_register(struct cx231xx *dev, static void cx231xx_media_device_init(struct cx231xx *dev,
struct usb_device *udev) struct usb_device *udev)
{ {
#ifdef CONFIG_MEDIA_CONTROLLER #ifdef CONFIG_MEDIA_CONTROLLER
struct media_device *mdev; struct media_device *mdev;
int ret;
mdev = kzalloc(sizeof(*mdev), GFP_KERNEL); mdev = kzalloc(sizeof(*mdev), GFP_KERNEL);
if (!mdev) if (!mdev)
...@@ -1224,14 +1224,7 @@ static void cx231xx_media_device_register(struct cx231xx *dev, ...@@ -1224,14 +1224,7 @@ static void cx231xx_media_device_register(struct cx231xx *dev,
mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice); mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice);
mdev->driver_version = LINUX_VERSION_CODE; mdev->driver_version = LINUX_VERSION_CODE;
ret = media_device_register(mdev); media_device_init(mdev);
if (ret) {
dev_err(dev->dev,
"Couldn't create a media device. Error: %d\n",
ret);
kfree(mdev);
return;
}
dev->media_dev = mdev; dev->media_dev = mdev;
#endif #endif
...@@ -1669,8 +1662,8 @@ static int cx231xx_usb_probe(struct usb_interface *interface, ...@@ -1669,8 +1662,8 @@ static int cx231xx_usb_probe(struct usb_interface *interface,
/* save our data pointer in this interface device */ /* save our data pointer in this interface device */
usb_set_intfdata(interface, dev); usb_set_intfdata(interface, dev);
/* Register the media controller */ /* Initialize the media controller */
cx231xx_media_device_register(dev, udev); cx231xx_media_device_init(dev, udev);
/* Create v4l2 device */ /* Create v4l2 device */
#ifdef CONFIG_MEDIA_CONTROLLER #ifdef CONFIG_MEDIA_CONTROLLER
...@@ -1742,11 +1735,18 @@ static int cx231xx_usb_probe(struct usb_interface *interface, ...@@ -1742,11 +1735,18 @@ static int cx231xx_usb_probe(struct usb_interface *interface,
request_modules(dev); request_modules(dev);
retval = cx231xx_create_media_graph(dev); retval = cx231xx_create_media_graph(dev);
if (retval < 0) { if (retval < 0)
cx231xx_release_resources(dev); goto done;
}
#ifdef CONFIG_MEDIA_CONTROLLER
retval = media_device_register(dev->media_dev);
#endif
done:
if (retval < 0)
cx231xx_release_resources(dev);
return retval; return retval;
err_video_alt: err_video_alt:
/* cx231xx_uninit_dev: */ /* cx231xx_uninit_dev: */
cx231xx_close_extension(dev); cx231xx_close_extension(dev);
......
...@@ -400,13 +400,12 @@ static int dvb_usb_stop_feed(struct dvb_demux_feed *dvbdmxfeed) ...@@ -400,13 +400,12 @@ static int dvb_usb_stop_feed(struct dvb_demux_feed *dvbdmxfeed)
return ret; return ret;
} }
static void dvb_usbv2_media_device_register(struct dvb_usb_adapter *adap) static void dvb_usbv2_media_device_init(struct dvb_usb_adapter *adap)
{ {
#ifdef CONFIG_MEDIA_CONTROLLER_DVB #ifdef CONFIG_MEDIA_CONTROLLER_DVB
struct media_device *mdev; struct media_device *mdev;
struct dvb_usb_device *d = adap_to_d(adap); struct dvb_usb_device *d = adap_to_d(adap);
struct usb_device *udev = d->udev; struct usb_device *udev = d->udev;
int ret;
mdev = kzalloc(sizeof(*mdev), GFP_KERNEL); mdev = kzalloc(sizeof(*mdev), GFP_KERNEL);
if (!mdev) if (!mdev)
...@@ -420,19 +419,18 @@ static void dvb_usbv2_media_device_register(struct dvb_usb_adapter *adap) ...@@ -420,19 +419,18 @@ static void dvb_usbv2_media_device_register(struct dvb_usb_adapter *adap)
mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice); mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice);
mdev->driver_version = LINUX_VERSION_CODE; mdev->driver_version = LINUX_VERSION_CODE;
ret = media_device_register(mdev); media_device_init(mdev);
if (ret) {
dev_err(&d->udev->dev,
"Couldn't create a media device. Error: %d\n",
ret);
kfree(mdev);
return;
}
dvb_register_media_controller(&adap->dvb_adap, mdev); dvb_register_media_controller(&adap->dvb_adap, mdev);
dev_info(&d->udev->dev, "media controller created\n"); dev_info(&d->udev->dev, "media controller created\n");
#endif
}
static void dvb_usbv2_media_device_register(struct dvb_usb_adapter *adap)
{
#ifdef CONFIG_MEDIA_CONTROLLER_DVB
media_device_register(adap->dvb_adap.mdev);
#endif #endif
} }
...@@ -444,6 +442,7 @@ static void dvb_usbv2_media_device_unregister(struct dvb_usb_adapter *adap) ...@@ -444,6 +442,7 @@ static void dvb_usbv2_media_device_unregister(struct dvb_usb_adapter *adap)
return; return;
media_device_unregister(adap->dvb_adap.mdev); media_device_unregister(adap->dvb_adap.mdev);
media_device_cleanup(adap->dvb_adap.mdev);
kfree(adap->dvb_adap.mdev); kfree(adap->dvb_adap.mdev);
adap->dvb_adap.mdev = NULL; adap->dvb_adap.mdev = NULL;
...@@ -467,7 +466,7 @@ static int dvb_usbv2_adapter_dvb_init(struct dvb_usb_adapter *adap) ...@@ -467,7 +466,7 @@ static int dvb_usbv2_adapter_dvb_init(struct dvb_usb_adapter *adap)
adap->dvb_adap.priv = adap; adap->dvb_adap.priv = adap;
dvb_usbv2_media_device_register(adap); dvb_usbv2_media_device_init(adap);
if (d->props->read_mac_address) { if (d->props->read_mac_address) {
ret = d->props->read_mac_address(adap, ret = d->props->read_mac_address(adap,
...@@ -702,6 +701,8 @@ static int dvb_usbv2_adapter_frontend_init(struct dvb_usb_adapter *adap) ...@@ -702,6 +701,8 @@ static int dvb_usbv2_adapter_frontend_init(struct dvb_usb_adapter *adap)
if (ret < 0) if (ret < 0)
goto err_dvb_unregister_frontend; goto err_dvb_unregister_frontend;
dvb_usbv2_media_device_register(adap);
return 0; return 0;
err_dvb_unregister_frontend: err_dvb_unregister_frontend:
......
...@@ -95,13 +95,12 @@ static int dvb_usb_stop_feed(struct dvb_demux_feed *dvbdmxfeed) ...@@ -95,13 +95,12 @@ static int dvb_usb_stop_feed(struct dvb_demux_feed *dvbdmxfeed)
return dvb_usb_ctrl_feed(dvbdmxfeed, 0); return dvb_usb_ctrl_feed(dvbdmxfeed, 0);
} }
static void dvb_usb_media_device_register(struct dvb_usb_adapter *adap) static void dvb_usb_media_device_init(struct dvb_usb_adapter *adap)
{ {
#ifdef CONFIG_MEDIA_CONTROLLER_DVB #ifdef CONFIG_MEDIA_CONTROLLER_DVB
struct media_device *mdev; struct media_device *mdev;
struct dvb_usb_device *d = adap->dev; struct dvb_usb_device *d = adap->dev;
struct usb_device *udev = d->udev; struct usb_device *udev = d->udev;
int ret;
mdev = kzalloc(sizeof(*mdev), GFP_KERNEL); mdev = kzalloc(sizeof(*mdev), GFP_KERNEL);
if (!mdev) if (!mdev)
...@@ -115,20 +114,21 @@ static void dvb_usb_media_device_register(struct dvb_usb_adapter *adap) ...@@ -115,20 +114,21 @@ static void dvb_usb_media_device_register(struct dvb_usb_adapter *adap)
mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice); mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice);
mdev->driver_version = LINUX_VERSION_CODE; mdev->driver_version = LINUX_VERSION_CODE;
ret = media_device_register(mdev); media_device_init(mdev);
if (ret) {
dev_err(&d->udev->dev,
"Couldn't create a media device. Error: %d\n",
ret);
kfree(mdev);
return;
}
dvb_register_media_controller(&adap->dvb_adap, mdev); dvb_register_media_controller(&adap->dvb_adap, mdev);
dev_info(&d->udev->dev, "media controller created\n"); dev_info(&d->udev->dev, "media controller created\n");
#endif #endif
} }
static void dvb_usb_media_device_register(struct dvb_usb_adapter *adap)
{
#ifdef CONFIG_MEDIA_CONTROLLER_DVB
media_device_register(adap->dvb_adap.mdev);
#endif
}
static void dvb_usb_media_device_unregister(struct dvb_usb_adapter *adap) static void dvb_usb_media_device_unregister(struct dvb_usb_adapter *adap)
{ {
#ifdef CONFIG_MEDIA_CONTROLLER_DVB #ifdef CONFIG_MEDIA_CONTROLLER_DVB
...@@ -136,6 +136,7 @@ static void dvb_usb_media_device_unregister(struct dvb_usb_adapter *adap) ...@@ -136,6 +136,7 @@ static void dvb_usb_media_device_unregister(struct dvb_usb_adapter *adap)
return; return;
media_device_unregister(adap->dvb_adap.mdev); media_device_unregister(adap->dvb_adap.mdev);
media_device_cleanup(adap->dvb_adap.mdev);
kfree(adap->dvb_adap.mdev); kfree(adap->dvb_adap.mdev);
adap->dvb_adap.mdev = NULL; adap->dvb_adap.mdev = NULL;
#endif #endif
...@@ -154,7 +155,7 @@ int dvb_usb_adapter_dvb_init(struct dvb_usb_adapter *adap, short *adapter_nums) ...@@ -154,7 +155,7 @@ int dvb_usb_adapter_dvb_init(struct dvb_usb_adapter *adap, short *adapter_nums)
} }
adap->dvb_adap.priv = adap; adap->dvb_adap.priv = adap;
dvb_usb_media_device_register(adap); dvb_usb_media_device_init(adap);
if (adap->dev->props.read_mac_address) { if (adap->dev->props.read_mac_address) {
if (adap->dev->props.read_mac_address(adap->dev, adap->dvb_adap.proposed_mac) == 0) if (adap->dev->props.read_mac_address(adap->dev, adap->dvb_adap.proposed_mac) == 0)
...@@ -323,6 +324,8 @@ int dvb_usb_adapter_frontend_init(struct dvb_usb_adapter *adap) ...@@ -323,6 +324,8 @@ int dvb_usb_adapter_frontend_init(struct dvb_usb_adapter *adap)
ret = dvb_create_media_graph(&adap->dvb_adap); ret = dvb_create_media_graph(&adap->dvb_adap);
dvb_usb_media_device_register(adap);
return ret; return ret;
} }
......
...@@ -361,10 +361,11 @@ static void *siano_media_device_register(struct smsusb_device_t *dev, ...@@ -361,10 +361,11 @@ static void *siano_media_device_register(struct smsusb_device_t *dev,
mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice); mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice);
mdev->driver_version = LINUX_VERSION_CODE; mdev->driver_version = LINUX_VERSION_CODE;
media_device_init(mdev);
ret = media_device_register(mdev); ret = media_device_register(mdev);
if (ret) { if (ret) {
pr_err("Couldn't create a media device. Error: %d\n", media_device_cleanup(mdev);
ret);
kfree(mdev); kfree(mdev);
return NULL; return NULL;
} }
......
...@@ -1656,6 +1656,7 @@ static void uvc_delete(struct uvc_device *dev) ...@@ -1656,6 +1656,7 @@ static void uvc_delete(struct uvc_device *dev)
#ifdef CONFIG_MEDIA_CONTROLLER #ifdef CONFIG_MEDIA_CONTROLLER
if (media_devnode_is_registered(&dev->mdev.devnode)) if (media_devnode_is_registered(&dev->mdev.devnode))
media_device_unregister(&dev->mdev); media_device_unregister(&dev->mdev);
media_device_cleanup(&dev->mdev);
#endif #endif
list_for_each_safe(p, n, &dev->chains) { list_for_each_safe(p, n, &dev->chains) {
...@@ -1906,7 +1907,7 @@ static int uvc_probe(struct usb_interface *intf, ...@@ -1906,7 +1907,7 @@ static int uvc_probe(struct usb_interface *intf,
"linux-uvc-devel mailing list.\n"); "linux-uvc-devel mailing list.\n");
} }
/* Register the media and V4L2 devices. */ /* Initialize the media device and register the V4L2 device. */
#ifdef CONFIG_MEDIA_CONTROLLER #ifdef CONFIG_MEDIA_CONTROLLER
dev->mdev.dev = &intf->dev; dev->mdev.dev = &intf->dev;
strlcpy(dev->mdev.model, dev->name, sizeof(dev->mdev.model)); strlcpy(dev->mdev.model, dev->name, sizeof(dev->mdev.model));
...@@ -1916,8 +1917,7 @@ static int uvc_probe(struct usb_interface *intf, ...@@ -1916,8 +1917,7 @@ static int uvc_probe(struct usb_interface *intf,
strcpy(dev->mdev.bus_info, udev->devpath); strcpy(dev->mdev.bus_info, udev->devpath);
dev->mdev.hw_revision = le16_to_cpu(udev->descriptor.bcdDevice); dev->mdev.hw_revision = le16_to_cpu(udev->descriptor.bcdDevice);
dev->mdev.driver_version = LINUX_VERSION_CODE; dev->mdev.driver_version = LINUX_VERSION_CODE;
if (media_device_register(&dev->mdev) < 0) media_device_init(&dev->mdev);
goto error;
dev->vdev.mdev = &dev->mdev; dev->vdev.mdev = &dev->mdev;
#endif #endif
...@@ -1936,6 +1936,11 @@ static int uvc_probe(struct usb_interface *intf, ...@@ -1936,6 +1936,11 @@ static int uvc_probe(struct usb_interface *intf,
if (uvc_register_chains(dev) < 0) if (uvc_register_chains(dev) < 0)
goto error; goto error;
#ifdef CONFIG_MEDIA_CONTROLLER
/* Register the media device node */
if (media_device_register(&dev->mdev) < 0)
goto error;
#endif
/* Save our data pointer in the interface data. */ /* Save our data pointer in the interface data. */
usb_set_intfdata(intf, dev); usb_set_intfdata(intf, dev);
......
...@@ -337,6 +337,32 @@ struct media_device { ...@@ -337,6 +337,32 @@ struct media_device {
/* media_devnode to media_device */ /* media_devnode to media_device */
#define to_media_device(node) container_of(node, struct media_device, devnode) #define to_media_device(node) container_of(node, struct media_device, devnode)
/**
* media_device_init() - Initializes a media device element
*
* @mdev: pointer to struct &media_device
*
* This function initializes the media device prior to its registration.
* The media device initialization and registration is split in two functions
* to avoid race conditions and make the media device available to user-space
* before the media graph has been completed.
*
* So drivers need to first initialize the media device, register any entity
* within the media device, create pad to pad links and then finally register
* the media device by calling media_device_register() as a final step.
*/
void media_device_init(struct media_device *mdev);
/**
* media_device_cleanup() - Cleanups a media device element
*
* @mdev: pointer to struct &media_device
*
* This function that will destroy the graph_mutex that is
* initialized in media_device_init().
*/
void media_device_cleanup(struct media_device *mdev);
/** /**
* __media_device_register() - Registers a media device element * __media_device_register() - Registers a media device element
* *
......
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