Hi Sakari, Thank you for the patch. On Wed, Dec 20, 2023 at 12:36:52PM +0200, Sakari Ailus wrote: > The release callback is only used to print a debug message. Drop it. (It > will be re-introduced later in a different form.) > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > Acked-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > drivers/media/mc/mc-device.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/drivers/media/mc/mc-device.c b/drivers/media/mc/mc-device.c > index d4553a3705f5..c0ea08a8fc31 100644 > --- a/drivers/media/mc/mc-device.c > +++ b/drivers/media/mc/mc-device.c > @@ -566,11 +566,6 @@ static DEVICE_ATTR_RO(model); > * Registration/unregistration > */ > > -static void media_device_release(struct media_devnode *devnode) > -{ > - dev_dbg(devnode->parent, "Media device released\n"); > -} > - > static void __media_device_unregister_entity(struct media_entity *entity) > { > struct media_device *mdev = entity->graph_obj.mdev; > @@ -718,7 +713,6 @@ int __must_check __media_device_register(struct media_device *mdev, > /* Register the device node. */ > mdev->devnode.fops = &media_device_fops; > mdev->devnode.parent = mdev->dev; > - mdev->devnode.release = media_device_release; > > /* Set version 0 to indicate user-space that the graph is static */ > mdev->topology_version = 0; -- Regards, Laurent Pinchart