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. > That will need to wait for two weeks though as > I don't have access to the hardware right now. > It's certainly appreciated (I'll see if I can rebase it), but I am not worried about that driver. It's soc_camera that is affected the most. 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
