Re: [PATCH v5 2/2] iio: ad5755: Add DT binding documentation

[Jonathan Cameron <jic23@kernel.org>]

On 26/06/16 18:42, Jonathan Cameron wrote:
> On 26/06/16 18:30, Sean Nyekjær wrote:
>>
>>
>>> On 26. jun. 2016, at 18.51, Jonathan Cameron <jic23@kernel.org> wrote:
>>>
>>>> On 21/06/16 06:04, Sean Nyekjær wrote:
>>>>
>>>>
>>>>> On 2016-06-11 19:26, Jonathan Cameron wrote:
>>>>>> On 29/05/16 15:12, Lars-Peter Clausen wrote:
>>>>>>> On 05/29/2016 03:57 PM, Jonathan Cameron wrote:
>>>>>>>> On 09/05/16 09:16, Sean Nyekjær wrote:
>>>>>>>>
>>>>>>>>> On 2016-03-28 17:13, Jonathan Cameron wrote:
>>>>>>>>>> On 12/03/16 09:28, Jonathan Cameron wrote:
>>>>>>>>>>> On 11/03/16 14:12, Sean Nyekjaer wrote:
>>>>>>>>>>> Signed-off-by: Sean Nyekjaer <sean.nyekjaer@prevas.dk>
>>>>>>>>>> Looks much better to me.
>>>>>>>>>>
>>>>>>>>>> Looking for Acks from Lars and a device tree binding maintainer
>>>>>>>>>> on this one though.
>>>>>>>>>> (added CCs)
>>>>>>>>> Lars?
>>>>>>>> No response...
>>>>>>>> Is there anything wrong with this?
>>>>>>> Given we are early in cycle there is no real rush for this...
>>>>>>>
>>>>>>> Lars, let me know if you are going to a while getting to this / any other
>>>>>>> patches I want responses on from you.
>>>>>> I don't think this solution is the correct approach, but since I'm not able
>>>>>> to provide an alternative and everybody else is OK with it, I'm not going to
>>>>>> block it either. So go ahead and apply it.
>>>>> Hmm. I've applied this now.  Perhaps we will revisit in the future and
>>>>> eventually deprecate this binding in favour of something more generic.
>>>>>
>>>>> Thanks for your hard work on this Sean. It certainly wasn't a case
>>>>> with obvious 'right' answers!
>>>>>
>>>>> Jonathan
>>>> Hi Jonathan
>>>>
>>>> Where is this applied, i can't seen to find it? :-)
>>>> If it failed some test, please let me know...
>>> Sorry should have said. Applied to the togreg branch of iio.git on
>>> kernel.org, but... That gets initially pushed out as testing
>>> for the autobuilders to play with it.
>>>
>>> Basically it gives me a window with it as an 'unofficial' branch
>>> so I can fix up an dumb stuff the autobuilders find.  There
>>> is rather less of that these days as the autobuilders try patches
>>> off the list.
>>>
>>> Jonathan
>>>>
>>>> /Sean
>>>
>> Hi
>>
>> I see the builder gives some warnings with me not being consistent enough with the #ifdef CONFIG_OF
>> I'll submit v6 tomorrow with this corrected
> Cool.  I clearly failed to push out at all last week given these
> have only just come up.  Sorry about that, have some recollection
> of a garden crisis of some type!
> 
Gah, forgot to say - dropped the two patches for now.  Will pick up the
v6 ones later in the week.

Thanks,

Jonathan
> Jonathan
>>
>> /Sean--
>> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>