On 11/27/2013 01:43 AM, Lars-Peter Clausen wrote:
On 11/26/2013 10:28 PM, Valentine wrote:
On 11/20/2013 07:53 PM, Valentine wrote:
On 11/20/2013 07:42 PM, Hans Verkuil wrote:
Hi Valentine,
Hi Hans,
Did you ever look at this adv7611 driver:
https://github.com/Xilinx/linux-xlnx/commit/610b9d5de22ae7c0047c65a07e4afa42af2daa12
No, I missed that one somehow, although I did search for the adv7611/7612
before implementing this one.
I'm going to look closer at the patch and test it.
I've tried the patch and I doubt that it was ever tested on adv7611.
It was and it works.
That's good to hear.
I haven't been able to make it work so far. Here's the description of some
of the issues
I've encountered.
The patch does not apply cleanly so I had to make small adjustments just to
make it apply
without changing the functionality.
I have an updated version of the patch, which I intend to submit soon.
That's great, thanks!
[...]
I'd also appreciate your thoughts about the issues I've described,
which have been replaced by [...] here.
It adds adv761x support to the adv7604 in a pretty clean way.
Doesn't seem that clean to me after having a look at it.
It tries to handle both 7604 and 7611 chips in the same way, though,
I'm not exactly sure if it's a good idea since 7611/12 is a pure HDMI
receiver with no analog inputs.
It is the same HDMI core (with minor modifications) though. So you end end
up with largely the same code for the 7604 and the 7611.
Yes, but that's about half of the code while 7611 doesn't really need the
non-HDMI part.
Just thought that we may end up with a bigger mess if we keep adding adv76xx
support to 7604 driver.
- Lars
Thanks,
Val.
--
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
