[media] media/platform: fix querycap
Querycap shouldn't set the version field (the core does that for you), but it should set the device_caps field. In addition, remove the CAPTURE and OUTPUT caps for M2M devices. These were already slated for removal, so it's time to do so. Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> Acked-by: Kamil Debski <k.debski@samsung.com> Acked-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> Acked-by: Jacek Anaszewski <j.anaszewski@samsung.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
This commit is contained in:
parent
a020c747bf
commit
8c17e5e3b3
|
@ -639,7 +639,6 @@ static int vpbe_display_querycap(struct file *file, void *priv,
|
|||
struct vpbe_layer *layer = video_drvdata(file);
|
||||
struct vpbe_device *vpbe_dev = layer->disp_dev->vpbe_dev;
|
||||
|
||||
cap->version = VPBE_DISPLAY_VERSION_CODE;
|
||||
cap->device_caps = V4L2_CAP_VIDEO_OUTPUT | V4L2_CAP_STREAMING;
|
||||
cap->capabilities = cap->device_caps | V4L2_CAP_DEVICE_CAPS;
|
||||
snprintf(cap->driver, sizeof(cap->driver), "%s",
|
||||
|
|
|
@ -930,8 +930,8 @@ static int vpfe_querycap(struct file *file, void *priv,
|
|||
|
||||
v4l2_dbg(1, debug, &vpfe_dev->v4l2_dev, "vpfe_querycap\n");
|
||||
|
||||
cap->version = VPFE_CAPTURE_VERSION_CODE;
|
||||
cap->capabilities = V4L2_CAP_VIDEO_CAPTURE | V4L2_CAP_STREAMING;
|
||||
cap->device_caps = V4L2_CAP_VIDEO_CAPTURE | V4L2_CAP_STREAMING;
|
||||
cap->capabilities = cap->device_caps | V4L2_CAP_DEVICE_CAPS;
|
||||
strlcpy(cap->driver, CAPTURE_DRV_NAME, sizeof(cap->driver));
|
||||
strlcpy(cap->bus_info, "VPFE", sizeof(cap->bus_info));
|
||||
strlcpy(cap->card, vpfe_dev->cfg->card_name, sizeof(cap->card));
|
||||
|
|
|
@ -297,14 +297,8 @@ static int vidioc_querycap(struct file *file, void *priv,
|
|||
strncpy(cap->driver, G2D_NAME, sizeof(cap->driver) - 1);
|
||||
strncpy(cap->card, G2D_NAME, sizeof(cap->card) - 1);
|
||||
cap->bus_info[0] = 0;
|
||||
cap->version = KERNEL_VERSION(1, 0, 0);
|
||||
/*
|
||||
* This is only a mem-to-mem video device. The capture and output
|
||||
* device capability flags are left only for backward compatibility
|
||||
* and are scheduled for removal.
|
||||
*/
|
||||
cap->capabilities = V4L2_CAP_VIDEO_CAPTURE | V4L2_CAP_VIDEO_OUTPUT |
|
||||
V4L2_CAP_VIDEO_M2M | V4L2_CAP_STREAMING;
|
||||
cap->device_caps = V4L2_CAP_VIDEO_M2M | V4L2_CAP_STREAMING;
|
||||
cap->capabilities = cap->device_caps | V4L2_CAP_DEVICE_CAPS;
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -1001,13 +1001,8 @@ static int s5p_jpeg_querycap(struct file *file, void *priv,
|
|||
sizeof(cap->card));
|
||||
}
|
||||
cap->bus_info[0] = 0;
|
||||
/*
|
||||
* This is only a mem-to-mem video device. The capture and output
|
||||
* device capability flags are left only for backward compatibility
|
||||
* and are scheduled for removal.
|
||||
*/
|
||||
cap->capabilities = V4L2_CAP_STREAMING | V4L2_CAP_VIDEO_M2M |
|
||||
V4L2_CAP_VIDEO_CAPTURE | V4L2_CAP_VIDEO_OUTPUT;
|
||||
cap->device_caps = V4L2_CAP_STREAMING | V4L2_CAP_VIDEO_M2M;
|
||||
cap->capabilities = cap->device_caps | V4L2_CAP_DEVICE_CAPS;
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -269,15 +269,13 @@ static int vidioc_querycap(struct file *file, void *priv,
|
|||
strncpy(cap->driver, dev->plat_dev->name, sizeof(cap->driver) - 1);
|
||||
strncpy(cap->card, dev->plat_dev->name, sizeof(cap->card) - 1);
|
||||
cap->bus_info[0] = 0;
|
||||
cap->version = KERNEL_VERSION(1, 0, 0);
|
||||
/*
|
||||
* This is only a mem-to-mem video device. The capture and output
|
||||
* device capability flags are left only for backward compatibility
|
||||
* and are scheduled for removal.
|
||||
*/
|
||||
cap->capabilities = V4L2_CAP_VIDEO_M2M_MPLANE | V4L2_CAP_STREAMING |
|
||||
V4L2_CAP_VIDEO_CAPTURE_MPLANE |
|
||||
V4L2_CAP_VIDEO_OUTPUT_MPLANE;
|
||||
cap->device_caps = V4L2_CAP_VIDEO_M2M_MPLANE | V4L2_CAP_STREAMING;
|
||||
cap->capabilities = cap->device_caps | V4L2_CAP_DEVICE_CAPS;
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -947,15 +947,13 @@ static int vidioc_querycap(struct file *file, void *priv,
|
|||
strncpy(cap->driver, dev->plat_dev->name, sizeof(cap->driver) - 1);
|
||||
strncpy(cap->card, dev->plat_dev->name, sizeof(cap->card) - 1);
|
||||
cap->bus_info[0] = 0;
|
||||
cap->version = KERNEL_VERSION(1, 0, 0);
|
||||
/*
|
||||
* This is only a mem-to-mem video device. The capture and output
|
||||
* device capability flags are left only for backward compatibility
|
||||
* and are scheduled for removal.
|
||||
*/
|
||||
cap->capabilities = V4L2_CAP_VIDEO_M2M_MPLANE | V4L2_CAP_STREAMING |
|
||||
V4L2_CAP_VIDEO_CAPTURE_MPLANE |
|
||||
V4L2_CAP_VIDEO_OUTPUT_MPLANE;
|
||||
cap->device_caps = V4L2_CAP_VIDEO_M2M_MPLANE | V4L2_CAP_STREAMING;
|
||||
cap->capabilities = cap->device_caps | V4L2_CAP_DEVICE_CAPS;
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue