On 03/11/14 09:33, Archit Taneja wrote:
> The bus_info parameter in v4l2_capabilities expects a 'platform_' prefix. This
> wasn't done in the driver and hence was breaking compliance. Update the 
> bus_info
> parameter accordingly.
> 
> Signed-off-by: Archit Taneja <arc...@ti.com>

Reviewed-by: Hans Verkuil <hans.verk...@cisco.com>

> ---
>  drivers/media/platform/ti-vpe/vpe.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/media/platform/ti-vpe/vpe.c 
> b/drivers/media/platform/ti-vpe/vpe.c
> index 46b9d44..5591d04 100644
> --- a/drivers/media/platform/ti-vpe/vpe.c
> +++ b/drivers/media/platform/ti-vpe/vpe.c
> @@ -1346,7 +1346,8 @@ static int vpe_querycap(struct file *file, void *priv,
>  {
>       strncpy(cap->driver, VPE_MODULE_NAME, sizeof(cap->driver) - 1);
>       strncpy(cap->card, VPE_MODULE_NAME, sizeof(cap->card) - 1);
> -     strlcpy(cap->bus_info, VPE_MODULE_NAME, sizeof(cap->bus_info));
> +     snprintf(cap->bus_info, sizeof(cap->bus_info), "platform:%s",
> +             VPE_MODULE_NAME);
>       cap->device_caps  = V4L2_CAP_VIDEO_M2M_MPLANE | V4L2_CAP_STREAMING;
>       cap->capabilities = cap->device_caps | V4L2_CAP_DEVICE_CAPS;
>       return 0;
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to