diff mbox

exynos-gsc: Add missing video device vfl_dir flag initialization

Message ID 1352588276-16260-1-git-send-email-sylvester.nawrocki@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Sylwester Nawrocki Nov. 10, 2012, 10:57 p.m. UTC
vfl_dir should be set to VFL_DIR_M2M so valid ioctls for this
mem-to-mem device can be properly determined in the v4l2 core.

Cc: Shaik Ameer Basha <shaik.ameer@samsung.com>
Signed-off-by: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
---
I didn't run-time test this patch.

 drivers/media/platform/exynos-gsc/gsc-m2m.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

--
1.7.4.1

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

Comments

Sylwester Nawrocki Nov. 19, 2012, 8:06 p.m. UTC | #1
Hi Shaik,

Could you let us know if the driver is working fine with this patch
applied ? I have no exynos5 based board to test it. And this patch
qualifies as an important fix that should be applied for v3.7, where
the driver's first appeared.

Thanks,
Sylwester

On 11/10/2012 11:57 PM, Sylwester Nawrocki wrote:
> vfl_dir should be set to VFL_DIR_M2M so valid ioctls for this
> mem-to-mem device can be properly determined in the v4l2 core.
>
> Cc: Shaik Ameer Basha<shaik.ameer@samsung.com>
> Signed-off-by: Sylwester Nawrocki<sylvester.nawrocki@gmail.com>
> ---
> I didn't run-time test this patch.
>
>   drivers/media/platform/exynos-gsc/gsc-m2m.c |    1 +
>   1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/media/platform/exynos-gsc/gsc-m2m.c b/drivers/media/platform/exynos-gsc/gsc-m2m.c
> index 3c7f005..88642a8 100644
> --- a/drivers/media/platform/exynos-gsc/gsc-m2m.c
> +++ b/drivers/media/platform/exynos-gsc/gsc-m2m.c
> @@ -732,6 +732,7 @@ int gsc_register_m2m_device(struct gsc_dev *gsc)
>   	gsc->vdev.ioctl_ops	=&gsc_m2m_ioctl_ops;
>   	gsc->vdev.release	= video_device_release_empty;
>   	gsc->vdev.lock		=&gsc->lock;
> +	gsc->vdev.vfl_dir	= VFL_DIR_M2M;
>   	snprintf(gsc->vdev.name, sizeof(gsc->vdev.name), "%s.%d:m2m",
>   					GSC_MODULE_NAME, gsc->id);
>
> --
> 1.7.4.1
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Shaik Ameer Basha Nov. 21, 2012, 11:59 a.m. UTC | #2
Hi Sylwester,

I applied this patch and tested exynos-gsc driver basic features on exynos5250.
It's working fine for me.

Thanks,
Shaik Ameer Basha

On Tue, Nov 20, 2012 at 1:36 AM, Sylwester Nawrocki
<sylvester.nawrocki@gmail.com> wrote:
> Hi Shaik,
>
> Could you let us know if the driver is working fine with this patch
> applied ? I have no exynos5 based board to test it. And this patch
> qualifies as an important fix that should be applied for v3.7, where
> the driver's first appeared.
>
> Thanks,
> Sylwester
>
>
> On 11/10/2012 11:57 PM, Sylwester Nawrocki wrote:
>>
>> vfl_dir should be set to VFL_DIR_M2M so valid ioctls for this
>> mem-to-mem device can be properly determined in the v4l2 core.
>>
>> Cc: Shaik Ameer Basha<shaik.ameer@samsung.com>
>> Signed-off-by: Sylwester Nawrocki<sylvester.nawrocki@gmail.com>
>> ---
>> I didn't run-time test this patch.
>>
>>   drivers/media/platform/exynos-gsc/gsc-m2m.c |    1 +
>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/media/platform/exynos-gsc/gsc-m2m.c
>> b/drivers/media/platform/exynos-gsc/gsc-m2m.c
>> index 3c7f005..88642a8 100644
>> --- a/drivers/media/platform/exynos-gsc/gsc-m2m.c
>> +++ b/drivers/media/platform/exynos-gsc/gsc-m2m.c
>> @@ -732,6 +732,7 @@ int gsc_register_m2m_device(struct gsc_dev *gsc)
>>         gsc->vdev.ioctl_ops     =&gsc_m2m_ioctl_ops;
>>         gsc->vdev.release       = video_device_release_empty;
>>         gsc->vdev.lock          =&gsc->lock;
>>
>> +       gsc->vdev.vfl_dir       = VFL_DIR_M2M;
>>         snprintf(gsc->vdev.name, sizeof(gsc->vdev.name), "%s.%d:m2m",
>>                                         GSC_MODULE_NAME, gsc->id);
>>
>> --
>> 1.7.4.1
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/media/platform/exynos-gsc/gsc-m2m.c b/drivers/media/platform/exynos-gsc/gsc-m2m.c
index 3c7f005..88642a8 100644
--- a/drivers/media/platform/exynos-gsc/gsc-m2m.c
+++ b/drivers/media/platform/exynos-gsc/gsc-m2m.c
@@ -732,6 +732,7 @@  int gsc_register_m2m_device(struct gsc_dev *gsc)
 	gsc->vdev.ioctl_ops	= &gsc_m2m_ioctl_ops;
 	gsc->vdev.release	= video_device_release_empty;
 	gsc->vdev.lock		= &gsc->lock;
+	gsc->vdev.vfl_dir	= VFL_DIR_M2M;
 	snprintf(gsc->vdev.name, sizeof(gsc->vdev.name), "%s.%d:m2m",
 					GSC_MODULE_NAME, gsc->id);