On 02/03/2015 04:24 PM, Hans Verkuil wrote:
On 02/03/15 16:22, Laurent Pinchart wrote:
Hi Hans,
On Monday 02 February 2015 11:09:27 Hans Verkuil wrote:
On 02/02/2015 11:01 AM, Laurent Pinchart wrote:
On Sunday 01 February 2015 12:26:11 Guennadi Liakhovetski wrote:
On a second thought:
On Sun, 1 Feb 2015, Guennadi Liakhovetski wrote:
Hi Wills,
Thanks for the patch. First and foremost, the title of the patch is
wrong. This patch does more than just adding some "adv7604
compatibility." It's adding pad-level API to soc-camera.
This is just a rough review. I'm not an expert in media-controller /
pad-level API, I hope someone with a better knowledge of those areas
will help me reviewing this.
Another general comment: it has been discussed since a long time,
whether a wrapper wouldn't be desired to enable a seamless use of both
subdev drivers using and not using the pad-level API. Maybe it's the
right time now?..
This would be a considerable change and would most probably take a rather
long time, given how busy everyone is.
If I understood correctly Hans Verkuil told me over the weekend that he
wanted to address this problem in the near future. Hans, could you detail
your plans ?
That's correct. This patch series makes all the necessary changes.
https://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg83415.html
Patches 1-4 have been merged already, but I need to do more testing for the
remainder. The Renesas SH7724 board is ideal for that, but unfortunately I
can't get it to work with the current kernel.
I can't help you much with that, but I could test changes using the rcar-vin
driver with the adv7180 if needed (does the adv7180 generate an image if no
analog source is connected ?).
I expect so, most SDTV receivers do that.
It has a freerun mode, in which it can output various kinds of patterns, see
https://patchwork.linuxtv.org/patch/27894/
- Lars
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
