On 02/02/2015 11:01 AM, Laurent Pinchart wrote: > Hi Guennadi, > > 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. Regards, Hans -- 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
