drivers: counter: Add __fallthrough in STM32 counter driver
STM32 counter driver uses switch statement in which cases don't end
with break or return intentionally.
Affected switches in counter driver check status of all timer channels
(maximum 4 channels), but the number of channels is not determined
during compilation. In switch, we jump to channel with highest number
and then check other channels with lower numbers.
Compiler can warn about it, so this patch adds information that it was
intentional.
Signed-off-by: Patryk Duda <pdk@semihalf.com>
diff --git a/drivers/counter/counter_ll_stm32_timer.c b/drivers/counter/counter_ll_stm32_timer.c
index 629c0f3..65146bd 100644
--- a/drivers/counter/counter_ll_stm32_timer.c
+++ b/drivers/counter/counter_ll_stm32_timer.c
@@ -339,10 +339,13 @@
switch (counter_get_num_of_channels(dev)) {
case 4U:
pending |= LL_TIM_IsActiveFlag_CC4(cfg->timer);
+ __fallthrough;
case 3U:
pending |= LL_TIM_IsActiveFlag_CC3(cfg->timer);
+ __fallthrough;
case 2U:
pending |= LL_TIM_IsActiveFlag_CC2(cfg->timer);
+ __fallthrough;
case 1U:
pending |= LL_TIM_IsActiveFlag_CC1(cfg->timer);
}
@@ -585,10 +588,13 @@
switch (counter_get_num_of_channels(dev)) {
case 4U:
TIM_IRQ_HANDLE_CC(timer, 4);
+ __fallthrough;
case 3U:
TIM_IRQ_HANDLE_CC(timer, 3);
+ __fallthrough;
case 2U:
TIM_IRQ_HANDLE_CC(timer, 2);
+ __fallthrough;
case 1U:
TIM_IRQ_HANDLE_CC(timer, 1);
}
