Re: [PATCH] IIO: stm32: Remove quadrature related functions from trigger driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 5/8/19 9:34 AM, William Breathitt Gray wrote:
> On Tue, May 07, 2019 at 02:37:07PM +0200, Benjamin Gaignard wrote:
>> Le mar. 7 mai 2019 à 12:19, William Breathitt Gray
>> <vilhelm.gray@xxxxxxxxx> a écrit :
>>> On Tue, May 07, 2019 at 11:12:24AM +0200, Benjamin Gaignard wrote:
>>>> Quadrature feature is now hosted on it own framework.
>>>> Remove quadrature related code from stm32-trigger driver to avoid
>>>> code duplication and simplify the ABI.
>>>>
>>>> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@xxxxxx>
>>> Although this functionality is now provided by the Counter subsystem, we
>>> should keep the IIO Counter interface for this driver intact since
>>> existing user applications may depend on it; this is the same reason why
>>> the IIO Counter code in the 104-QUAD-8 device driver was not removed
>>> despite moving to the Counter subsystem.
>>>
>>> Once user applications have had enough time to migrate to the Generic
>>> Counter interface, we can consider removing the deprecated IIO Counter
>>> interface.
>> Hi William,
>>
>> This SoC is not yet in production so their is no legacy on the old interface
>> and I would like to avoid to create one.
>>
>> Benjamin
> Ah, I see what you mean, this driver is for future devices. Do the
> earlier STM32 H7 series devices have a quadrature feature as well, or is
> this functionality only available with the new devices?

It is available on STM32 H7 but I have never see any Linux product on 
those SOC

and even less using quadratic encoder ;-)

Benjamin

>
> William Breathitt Gray
>
>>> William Breathitt Gray
>>>
>>> _______________________________________________
>>> linux-arm-kernel mailing list
>>> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
>>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux