From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Ezequiel Garcia <elezegarcia@gmail.com>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [GIT PULL for 3.7-rc1] media updates - part 1
Date: Tue, 9 Oct 2012 19:21:20 -0300 [thread overview]
Message-ID: <20121009192120.3d071497@redhat.com> (raw)
In-Reply-To: <CALF0-+XQW1RrHQN4=5fzLLsdJESrLnhSwW3yD9rK+huP5dUOqQ@mail.gmail.com>
Em Tue, 9 Oct 2012 14:43:46 -0300
Ezequiel Garcia <elezegarcia@gmail.com> escreveu:
> On Fri, Oct 5, 2012 at 10:42 AM, Mauro Carvalho Chehab
...
> > Ezequiel García (13):
> > [media] em28xx: Remove useless runtime->private_data usage
> > [media] media: Add stk1160 new driver (easycap replacement)
> > [media] staging: media: Remove easycap driver
>
> Hi Mauro,
>
> We've replaced easycap staging driver with stk1160.
> However, stk1160 still misses s-video input support, which I believe
> easycap had.
>
> This feature was missing because I couldn't get s-video devices to test with,
> but now a couple users have provided the test and the patch is ready.
> It's a tiny patch routing saa7115 properly.
Ok. Let's add it them.
> I think we should include this feature in v3.7 to complete easycap ->
> stk1160 replacement.
Yes, that's make sense for me. As easycap is a new driver, there's no
regression on adding a patch like that, even after the merge window,
especially since input selection is typically a trivial change that
won't cause bad effects.
Please send the patch. I'll likely submit it upstream by next week,
after giving some time for people to test it via the media tree.
> Also, it seems to me there's no point in keeping the driver until v3.8,
> since there aren't much users out there testing it.
>
> On the other side, I don't want to mess with the flow, so it's
> completely up to you.
> If you think it's okey, then I can send the patch tonight and
> hopefully you can pick
> it for your second pull request.
>
> Please let me know.
>
> Ezequiel
Regards,
Mauro
prev parent reply other threads:[~2012-10-09 22:21 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-05 13:42 [GIT PULL for 3.7-rc1] media updates - part 1 Mauro Carvalho Chehab
2012-10-09 17:43 ` Ezequiel Garcia
2012-10-09 22:21 ` Mauro Carvalho Chehab [this message]
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=20121009192120.3d071497@redhat.com \
--to=mchehab@redhat.com \
--cc=elezegarcia@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
/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;
as well as URLs for NNTP newsgroup(s).