[PATCH] counter: stm32-lptimer-cnt: remove iio counter abi

William Breathitt Gray vilhelm.gray at gmail.com
Fri Jan 29 06:02:29 EST 2021


On Fri, Jan 29, 2021 at 10:59:15AM +0100, Fabrice Gasnier wrote:
> On 1/29/21 9:35 AM, William Breathitt Gray wrote:
> > On Fri, Jan 22, 2021 at 04:03:23PM +0100, Fabrice Gasnier wrote:
> >> Currently, the STM32 LP Timer counter driver registers into both IIO and
> >> counter subsystems, which is redundant.
> >>
> >> Remove the IIO counter ABI and IIO registration from the STM32 LP Timer
> >> counter driver since it's been superseded by the Counter subsystem
> >> as discussed in [1].
> >>
> >> Keep only the counter subsystem related part.
> >> Move a part of the ABI documentation into a driver comment.
> >>
> >> This also removes a duplicate ABI warning
> >> $ scripts/get_abi.pl validate
> >> ...
> >> /sys/bus/iio/devices/iio:deviceX/in_count0_preset is defined 2 times:
> >>   ./Documentation/ABI/testing/sysfs-bus-iio-timer-stm32:100
> >>   ./Documentation/ABI/testing/sysfs-bus-iio-lptimer-stm32:0
> >>
> >> [1] https://lkml.org/lkml/2021/1/19/347
> >>
> >> Signed-off-by: Fabrice Gasnier <fabrice.gasnier at foss.st.com>
> >> ---
> >>  .../ABI/testing/sysfs-bus-iio-lptimer-stm32        |  62 -----
> >>  drivers/counter/stm32-lptimer-cnt.c                | 297 +++------------------
> >>  2 files changed, 36 insertions(+), 323 deletions(-)
> >>  delete mode 100644 Documentation/ABI/testing/sysfs-bus-iio-lptimer-stm32
> > 
> > Hi Fabrice,
> > 
> > I noticed this is missing a Kconfig change to remove the IIO dependency
> > from the STM32_LPTIMER_CNT config option. I assume that dependency is no
> > longer necessary with the IIO code is removed.
> 
> Hi William,
> 
> Thanks for spotting this.
> I'll send a v2 with that fixed.
> I assume I can add your ack'ed by for v2 (as commented earlier) ?
> 
> Best Regards,
> Fabrice

Yes that should be fine with me. :-)

William Breathitt Gray
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20210129/bad95f46/attachment.sig>


More information about the linux-arm-kernel mailing list