Commit cf439d5f authored by Sakari Ailus's avatar Sakari Ailus Committed by Mauro Carvalho Chehab

[media] media: Determine early whether an IOCTL is supported

Preparation for refactoring media IOCTL handling to unify common parts.
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent 2950da48
...@@ -419,6 +419,22 @@ static long media_device_get_topology(struct media_device *mdev, ...@@ -419,6 +419,22 @@ static long media_device_get_topology(struct media_device *mdev,
return 0; return 0;
} }
#define MEDIA_IOC(__cmd) \
[_IOC_NR(MEDIA_IOC_##__cmd)] = { .cmd = MEDIA_IOC_##__cmd }
/* the table is indexed by _IOC_NR(cmd) */
struct media_ioctl_info {
unsigned int cmd;
};
static const struct media_ioctl_info ioctl_info[] = {
MEDIA_IOC(DEVICE_INFO),
MEDIA_IOC(ENUM_ENTITIES),
MEDIA_IOC(ENUM_LINKS),
MEDIA_IOC(SETUP_LINK),
MEDIA_IOC(G_TOPOLOGY),
};
static long media_device_ioctl(struct file *filp, unsigned int cmd, static long media_device_ioctl(struct file *filp, unsigned int cmd,
unsigned long arg) unsigned long arg)
{ {
...@@ -426,6 +442,10 @@ static long media_device_ioctl(struct file *filp, unsigned int cmd, ...@@ -426,6 +442,10 @@ static long media_device_ioctl(struct file *filp, unsigned int cmd,
struct media_device *dev = devnode->media_dev; struct media_device *dev = devnode->media_dev;
long ret; long ret;
if (_IOC_NR(cmd) >= ARRAY_SIZE(ioctl_info)
|| ioctl_info[_IOC_NR(cmd)].cmd != cmd)
return -ENOIOCTLCMD;
mutex_lock(&dev->graph_mutex); mutex_lock(&dev->graph_mutex);
switch (cmd) { switch (cmd) {
case MEDIA_IOC_DEVICE_INFO: case MEDIA_IOC_DEVICE_INFO:
......
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