Skip to content

Commit 1c47574

Browse files
author
Alain Volmat
committed
drivers: clock: stm32: use logical AND in preproc check
Replace bitwise AND (&) by logical AND (&&) within preprocessor if statements in clock_stm32_ll_common.c Signed-off-by: Alain Volmat <[email protected]>
1 parent 7820875 commit 1c47574

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed

drivers/clock_control/clock_stm32_ll_common.c

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -484,47 +484,47 @@ static int stm32_clock_control_get_subsys_rate(const struct device *clock,
484484
case STM32_SRC_SYSCLK:
485485
*rate = SystemCoreClock * STM32_CORE_PRESCALER;
486486
break;
487-
#if defined(STM32_SRC_PLLCLK) & defined(STM32_SYSCLK_SRC_PLL)
487+
#if defined(STM32_SRC_PLLCLK) && defined(STM32_SYSCLK_SRC_PLL)
488488
case STM32_SRC_PLLCLK:
489489
if (get_pllout_frequency() == 0) {
490490
return -EIO;
491491
}
492492
*rate = get_pllout_frequency();
493493
break;
494494
#endif
495-
#if defined(STM32_SRC_PLL_P) & STM32_PLL_P_ENABLED
495+
#if defined(STM32_SRC_PLL_P) && STM32_PLL_P_ENABLED
496496
case STM32_SRC_PLL_P:
497497
*rate = get_pll_div_frequency(get_pllsrc_frequency(),
498498
STM32_PLL_M_DIVISOR,
499499
STM32_PLL_N_MULTIPLIER,
500500
STM32_PLL_P_DIVISOR);
501501
break;
502502
#endif
503-
#if defined(STM32_SRC_PLL_Q) & STM32_PLL_Q_ENABLED
503+
#if defined(STM32_SRC_PLL_Q) && STM32_PLL_Q_ENABLED
504504
case STM32_SRC_PLL_Q:
505505
*rate = get_pll_div_frequency(get_pllsrc_frequency(),
506506
STM32_PLL_M_DIVISOR,
507507
STM32_PLL_N_MULTIPLIER,
508508
STM32_PLL_Q_DIVISOR);
509509
break;
510510
#endif
511-
#if defined(STM32_SRC_PLL_R) & STM32_PLL_R_ENABLED
511+
#if defined(STM32_SRC_PLL_R) && STM32_PLL_R_ENABLED
512512
case STM32_SRC_PLL_R:
513513
*rate = get_pll_div_frequency(get_pllsrc_frequency(),
514514
STM32_PLL_M_DIVISOR,
515515
STM32_PLL_N_MULTIPLIER,
516516
STM32_PLL_R_DIVISOR);
517517
break;
518518
#endif
519-
#if defined(STM32_SRC_PLLI2S_Q) & STM32_PLLI2S_Q_ENABLED & STM32_PLLI2S_ENABLED
519+
#if defined(STM32_SRC_PLLI2S_Q) && STM32_PLLI2S_Q_ENABLED && STM32_PLLI2S_ENABLED
520520
case STM32_SRC_PLLI2S_Q:
521521
*rate = get_pll_div_frequency(get_pllsrc_frequency(),
522522
STM32_PLLI2S_M_DIVISOR,
523523
STM32_PLLI2S_N_MULTIPLIER,
524524
STM32_PLLI2S_Q_DIVISOR);
525525
break;
526526
#endif /* STM32_SRC_PLLI2S_Q */
527-
#if defined(STM32_SRC_PLLI2S_R) & STM32_PLLI2S_ENABLED
527+
#if defined(STM32_SRC_PLLI2S_R) && STM32_PLLI2S_ENABLED
528528
case STM32_SRC_PLLI2S_R:
529529
*rate = get_pll_div_frequency(get_pllsrc_frequency(),
530530
STM32_PLLI2S_M_DIVISOR,
@@ -576,56 +576,56 @@ static int stm32_clock_control_get_subsys_rate(const struct device *clock,
576576
*rate /= STM32_PLLSAI_DIVR_DIVISOR;
577577
break;
578578
#endif /* STM32_SRC_PLLSAI_DIVR */
579-
#if defined(STM32_SRC_PLLSAI1_P) & STM32_PLLSAI1_P_ENABLED
579+
#if defined(STM32_SRC_PLLSAI1_P) && STM32_PLLSAI1_P_ENABLED
580580
case STM32_SRC_PLLSAI1_P:
581581
*rate = get_pll_div_frequency(get_pllsai1src_frequency(),
582582
STM32_PLLSAI1_M_DIVISOR,
583583
STM32_PLLSAI1_N_MULTIPLIER,
584584
STM32_PLLSAI1_P_DIVISOR);
585585
break;
586586
#endif /* STM32_SRC_PLLSAI1_P */
587-
#if defined(STM32_SRC_PLLSAI1_Q) & STM32_PLLSAI1_Q_ENABLED
587+
#if defined(STM32_SRC_PLLSAI1_Q) && STM32_PLLSAI1_Q_ENABLED
588588
case STM32_SRC_PLLSAI1_Q:
589589
*rate = get_pll_div_frequency(get_pllsai1src_frequency(),
590590
STM32_PLLSAI1_M_DIVISOR,
591591
STM32_PLLSAI1_N_MULTIPLIER,
592592
STM32_PLLSAI1_Q_DIVISOR);
593593
break;
594594
#endif /* STM32_SRC_PLLSAI1_Q */
595-
#if defined(STM32_SRC_PLLSAI1_R) & STM32_PLLSAI1_R_ENABLED
595+
#if defined(STM32_SRC_PLLSAI1_R) && STM32_PLLSAI1_R_ENABLED
596596
case STM32_SRC_PLLSAI1_R:
597597
*rate = get_pll_div_frequency(get_pllsai1src_frequency(),
598598
STM32_PLLSAI1_M_DIVISOR,
599599
STM32_PLLSAI1_N_MULTIPLIER,
600600
STM32_PLLSAI1_R_DIVISOR);
601601
break;
602602
#endif /* STM32_SRC_PLLSAI1_R */
603-
#if defined(STM32_SRC_PLLSAI2_P) & STM32_PLLSAI2_P_ENABLED
603+
#if defined(STM32_SRC_PLLSAI2_P) && STM32_PLLSAI2_P_ENABLED
604604
case STM32_SRC_PLLSAI2_P:
605605
*rate = get_pll_div_frequency(get_pllsai2src_frequency(),
606606
STM32_PLLSAI2_M_DIVISOR,
607607
STM32_PLLSAI2_N_MULTIPLIER,
608608
STM32_PLLSAI2_P_DIVISOR);
609609
break;
610610
#endif /* STM32_SRC_PLLSAI2_P */
611-
#if defined(STM32_SRC_PLLSAI2_Q) & STM32_PLLSAI2_Q_ENABLED
611+
#if defined(STM32_SRC_PLLSAI2_Q) && STM32_PLLSAI2_Q_ENABLED
612612
case STM32_SRC_PLLSAI2_Q:
613613
*rate = get_pll_div_frequency(get_pllsai2src_frequency(),
614614
STM32_PLLSAI2_M_DIVISOR,
615615
STM32_PLLSAI2_N_MULTIPLIER,
616616
STM32_PLLSAI2_Q_DIVISOR);
617617
break;
618618
#endif /* STM32_SRC_PLLSAI2_Q */
619-
#if defined(STM32_SRC_PLLSAI2_R) & STM32_PLLSAI2_R_ENABLED
619+
#if defined(STM32_SRC_PLLSAI2_R) && STM32_PLLSAI2_R_ENABLED
620620
case STM32_SRC_PLLSAI2_R:
621621
*rate = get_pll_div_frequency(get_pllsai2src_frequency(),
622622
STM32_PLLSAI2_M_DIVISOR,
623623
STM32_PLLSAI2_N_MULTIPLIER,
624624
STM32_PLLSAI2_R_DIVISOR);
625625
break;
626626
#endif /* STM32_SRC_PLLSAI2_R */
627-
#if defined(STM32_SRC_PLLSAI2_DIVR) & STM32_PLLSAI2_R_ENABLED & STM32_PLLSAI2_DIVR_ENABLED \
628-
& defined(STM32_PLLSAI2_DIVR_DIVISOR)
627+
#if defined(STM32_SRC_PLLSAI2_DIVR) && STM32_PLLSAI2_R_ENABLED && STM32_PLLSAI2_DIVR_ENABLED && \
628+
defined(STM32_PLLSAI2_DIVR_DIVISOR)
629629
case STM32_SRC_PLLSAI2_DIVR:
630630
*rate = get_pll_div_frequency(get_pllsai2src_frequency(),
631631
STM32_PLLSAI2_M_DIVISOR,
@@ -812,11 +812,11 @@ static void set_up_plls(void)
812812

813813
#if defined(STM32_PLL_ENABLED)
814814

815-
#if defined(STM32_SRC_PLL_P) & STM32_PLL_P_ENABLED
815+
#if defined(STM32_SRC_PLL_P) && STM32_PLL_P_ENABLED
816816
MODIFY_REG(RCC->PLLCFGR, RCC_PLLCFGR_PLLP, pllp(STM32_PLL_P_DIVISOR));
817817
RCC_PLLP_ENABLE();
818818
#endif
819-
#if defined(STM32_SRC_PLL_Q) & STM32_PLL_Q_ENABLED
819+
#if defined(STM32_SRC_PLL_Q) && STM32_PLL_Q_ENABLED
820820
MODIFY_REG(RCC->PLLCFGR, RCC_PLLCFGR_PLLQ, pllq(STM32_PLL_Q_DIVISOR));
821821
RCC_PLLQ_ENABLE();
822822
#endif

0 commit comments

Comments
 (0)