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 ?). That will need to wait for two weeks though as I don't have access to the hardware right now. -- Regards, Laurent Pinchart -- 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
