From: Hans Verkuil <hverkuil@xs4all.nl>
To: Jean-Michel Hautbois <jean-michel.hautbois@veo-labs.com>,
Linux Media Mailing List <linux-media@vger.kernel.org>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Subject: Re: [RFC] ADV7604: VGA support
Date: Mon, 12 Oct 2015 12:22:53 +0200 [thread overview]
Message-ID: <561B89FD.4010802@xs4all.nl> (raw)
In-Reply-To: <CAH-u=81zwkTxjYEsO8rNLf687-nGuj3DdJNeF6bmnxSUSVYQQg@mail.gmail.com>
On 10/04/2015 06:17 PM, Jean-Michel Hautbois wrote:
> Hi,
>
> I had another look into the ADV7604 HW manual, and I understand that
> in automatic mode, there is 4 AIN_SEL values possible, determining the
> connection on AIN pins.
> Now, having a look at the current ADV76xx files, I can see that two
> pads are there :
> ADV7604_PAD_VGA_RGB and ADV7604_PAD_VGA_COMP.
>
> According to the manual, my understanding is that we should have four
> HDMI pads and four analog pads. The latter would be configured as RGB
> or component, which allows four analog inputs as described in the HW
> manual.
When I wrote the driver we only needed one VGA input receiving either RGB
or YCbCr. Hence there is only one analog input and two pads. I wouldn't have
been able to test the additional analog inputs anyway.
I chose to use pads to select between the two modes, but that's something
that can be changed (it's not something you can autodetect, unfortunately).
If you want to add support for all four analog inputs, then feel free to
do so.
Regards,
Hans
>
> I don't know if you agree with that or if you had something else in
> mind when designing it in the first place, I may have missed something
> (Lars :) ?).
>
> Thanks,
> JM
> --
> 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
>
next prev parent reply other threads:[~2015-10-12 10:24 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-04 16:17 [RFC] ADV7604: VGA support Jean-Michel Hautbois
2015-10-12 10:22 ` Hans Verkuil [this message]
2016-01-07 17:20 ` Jean-Michel Hautbois
2016-01-11 9:15 ` Hans Verkuil
2016-01-13 13:18 ` Jean-Michel Hautbois
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=561B89FD.4010802@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=jean-michel.hautbois@veo-labs.com \
--cc=lars@metafoo.de \
--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).