Re: [PATCH] sur40: drop unnecessary format description

2016-06-27 Thread Nick Dyer 
On 27/06/2016 11:25, Hans Verkuil wrote:
> Don't fill in the format description. This is now done in the V4L2 core to 
> ensure
> consistent descriptions.
> 
> Signed-off-by: Hans Verkuil 

Acked-by: Nick Dyer 

> 
> diff --git a/drivers/input/touchscreen/sur40.c 
> b/drivers/input/touchscreen/sur40.c
> index 880c40b..5f1617b 100644
> --- a/drivers/input/touchscreen/sur40.c
> +++ b/drivers/input/touchscreen/sur40.c
> @@ -793,7 +793,6 @@ static int sur40_vidioc_enum_fmt(struct file *file, void 
> *priv,
>  {
>   if (f->index != 0)
>   return -EINVAL;
> - strlcpy(f->description, "8-bit greyscale", sizeof(f->description));
>   f->pixelformat = V4L2_PIX_FMT_GREY;
>   f->flags = 0;
>   return 0;
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] sur40: drop unnecessary format description

2016-06-27 Thread Hans Verkuil 
Don't fill in the format description. This is now done in the V4L2 core to 
ensure
consistent descriptions.

Signed-off-by: Hans Verkuil 

diff --git a/drivers/input/touchscreen/sur40.c 
b/drivers/input/touchscreen/sur40.c
index 880c40b..5f1617b 100644
--- a/drivers/input/touchscreen/sur40.c
+++ b/drivers/input/touchscreen/sur40.c
@@ -793,7 +793,6 @@ static int sur40_vidioc_enum_fmt(struct file *file, void 
*priv,
 {
if (f->index != 0)
return -EINVAL;
-   strlcpy(f->description, "8-bit greyscale", sizeof(f->description));
f->pixelformat = V4L2_PIX_FMT_GREY;
f->flags = 0;
return 0;
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html