From: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
To: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>,
Shaik Ameer Basha <shaik.ameer@samsung.com>
Cc: linux-media@vger.kernel.org, sw0312.kim@samsung.com
Subject: Re: [PATCH] exynos-gsc: Add missing video device vfl_dir flag initialization
Date: Mon, 19 Nov 2012 21:06:53 +0100 [thread overview]
Message-ID: <50AA915D.5020304@gmail.com> (raw)
In-Reply-To: <1352588276-16260-1-git-send-email-sylvester.nawrocki@gmail.com>
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
>
next prev parent reply other threads:[~2012-11-19 20:06 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-10 22:57 [PATCH] exynos-gsc: Add missing video device vfl_dir flag initialization Sylwester Nawrocki
2012-11-19 20:06 ` Sylwester Nawrocki [this message]
2012-11-21 11:59 ` Shaik Ameer Basha
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=50AA915D.5020304@gmail.com \
--to=sylvester.nawrocki@gmail.com \
--cc=linux-media@vger.kernel.org \
--cc=shaik.ameer@samsung.com \
--cc=sw0312.kim@samsung.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox