Re: [PATCH v2 20/39] docs: ABI: testing: make the files compatible with ReST output

From: Fabrice Gasnier
Date: Mon Nov 02 2020 - 06:05:36 EST


On 10/30/20 11:09 AM, Mauro Carvalho Chehab wrote:
> Em Fri, 30 Oct 2020 10:19:12 +0100
> Fabrice Gasnier <fabrice.gasnier@xxxxxx> escreveu:
>
>> Hi Mauro,
>>
>> [...]
>>
>>>
>>> +What: /sys/bus/iio/devices/iio:deviceX/in_count_quadrature_mode_available
>>> +KernelVersion: 4.12
>>> +Contact: benjamin.gaignard@xxxxxx
>>> +Description:
>>> + Reading returns the list possible quadrature modes.
>>> +
>>> +What: /sys/bus/iio/devices/iio:deviceX/in_count0_quadrature_mode
>>> +KernelVersion: 4.12
>>> +Contact: benjamin.gaignard@xxxxxx
>>> +Description:
>>> + Configure the device counter quadrature modes:
>>> +
>>> + channel_A:
>>> + Encoder A input servers as the count input and B as
>>> + the UP/DOWN direction control input.
>>> +
>>> + channel_B:
>>> + Encoder B input serves as the count input and A as
>>> + the UP/DOWN direction control input.
>>> +
>>> + quadrature:
>>> + Encoder A and B inputs are mixed to get direction
>>> + and count with a scale of 0.25.
>>> +
>>
>
> Hi Fabrice,
>
>> I just noticed that since Jonathan question in v1.
>>
>> Above ABI has been moved in the past as discussed in [1]. You can take a
>> look at:
>> b299d00 IIO: stm32: Remove quadrature related functions from trigger driver
>>
>> Could you please remove the above chunk ?
>>
>> With that, for the stm32 part:
>> Acked-by: Fabrice Gasnier <fabrice.gasnier@xxxxxx>
>
>
> Hmm... probably those were re-introduced due to a rebase. This
> series were originally written about 1,5 years ago.
>
> I'll drop those hunks.

Hi Mauro, Greg,

I just figured out this patch has been applied with above hunk.

This should be dropped: is there a fix on its way already ?
(I may have missed it)

Please advise,
Fabrice
>
> Thanks!
> Mauro
>