Re: [PATCH V2] media: i2c: Add ADV761X support

Lars-Peter Clausen
2013-11-29T20:16:48Z

On 11/26/2013 11:06 PM, Lars-Peter Clausen wrote:
> On 11/26/2013 11:03 PM, Laurent Pinchart wrote:
>> On Tuesday 26 November 2013 23:03:19 Lars-Peter Clausen wrote:
>>> On 11/26/2013 11:00 PM, Laurent Pinchart wrote:
>>>> On Tuesday 26 November 2013 22:43:32 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/610b9d5de22ae7c0047c65a07e
>>>>>>>> 4afa42af2daa12
>>>>>>>
>>>>>>> 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.
>>>>>
>>>>>> 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.
>>>>
>>>> Is it publicly available already ?
>>>
>>> Just started working on it the other day.
>>
>> I'm working on the same chip, how can we avoid effort duplication ?
>
> I'll have something by the end of the week, latest.

Sorry, this has to wait until next week. Didn't had the time to do further
work on it this week.

- Lars