Commit f9e86b5e authored by Hans Verkuil's avatar Hans Verkuil Committed by Mauro Carvalho Chehab
Browse files

V4L/DVB (8782): v4l2-dev: add video_device_release_empty 



Add a second release function: video_device_release_empty
It can be used by drivers that have statically allocated
video_device structs.

Its use usually, but not always, indicates laziness on the
part of the driver programmer.

Signed-off-by: default avatarHans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent cb353588

drivers/media/video/v4l2-dev.c

+7 −0
Original line number Diff line number Diff line
@@ -73,6 +73,13 @@ void video_device_release(struct video_device *vfd) 
}

EXPORT_SYMBOL(video_device_release);



void video_device_release_empty(struct video_device *vfd)

{

	/* Do nothing */

	/* Only valid when the video_device struct is a static. */

}

EXPORT_SYMBOL(video_device_release_empty);



static void video_release(struct device *cd)

{

	struct video_device *vfd = container_of(cd, struct video_device, dev);

include/media/v4l2-dev.h

+5 −0
Original line number Diff line number Diff line
@@ -91,7 +91,12 @@ void video_unregister_device(struct video_device *); 
/* helper functions to alloc / release struct video_device, the

   later can be used for video_device->release() */

struct video_device *video_device_alloc(void);

/* this release function frees the vfd pointer */

void video_device_release(struct video_device *vfd);

/* this release function does nothing, use when the video_device is a

   static global struct. Note that having a static video_device is

   a dubious construction at best. */

void video_device_release_empty(struct video_device *vfd);



#ifdef OBSOLETE_DEVDATA /* to be removed soon */

/* helper functions to access driver private data. */