1
0
Fork 0
alistair23-linux/drivers/counter
Fabrice Gasnier b14d72ac73 counter: stm32-timer-cnt: fix ceiling miss-alignment with reload register
Ceiling value may be miss-aligned with what's actually configured into the
ARR register. This is seen after probe as currently the ARR value is zero,
whereas ceiling value is set to the maximum. So:
- reading ceiling reports zero
- in case the counter gets enabled without any prior configuration,
  it won't count.
- in case the function gets set by the user 1st, (priv->ceiling) is used.

Fix it by getting rid of the cached "priv->ceiling" variable. Rather use
the ARR register value directly by using regmap read or write when needed.
There should be no drawback on performance as priv->ceiling isn't used in
performance critical path.
There's also no point in writing ARR while setting function (sms), so
it can be safely removed.

Fixes: ad29937e20 ("counter: Add STM32 Timer quadrature encoder")
Suggested-by: William Breathitt Gray <vilhelm.gray@gmail.com>
Signed-off-by: Fabrice Gasnier <fabrice.gasnier@foss.st.com>
Acked-by: William Breathitt Gray <vilhelm.gray@gmail.com>
Cc: <Stable@vger.kernel.org>
Link: https://lore.kernel.org/r/1614793789-10346-1-git-send-email-fabrice.gasnier@foss.st.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
2021-03-06 16:48:09 +00:00
..
104-quad-8.c Linux 5.8-rc6 2020-07-20 09:37:31 +02:00
Kconfig counter: Add microchip TCB capture counter 2020-07-20 13:04:40 +01:00
Makefile counter: Add microchip TCB capture counter 2020-07-20 13:04:40 +01:00
counter.c counter: Simplify the count_read and count_write callbacks 2019-10-18 19:47:27 +01:00
ftm-quaddec.c counter: Simplify the count_read and count_write callbacks 2019-10-18 19:47:27 +01:00
microchip-tcb-capture.c counter: microchip-tcb-capture: Fix CMR value check 2020-12-03 19:40:31 +00:00
stm32-lptimer-cnt.c iio: remove explicit IIO device parent assignment 2020-06-14 11:49:59 +01:00
stm32-timer-cnt.c counter: stm32-timer-cnt: fix ceiling miss-alignment with reload register 2021-03-06 16:48:09 +00:00
ti-eqep.c counter:ti-eqep: remove floor 2021-01-14 20:56:56 +00:00