drv/motor: move RCC init to platform.c

Keeping things consistent by doing all RCC init in one place.

Author: dlech

lib/pbio/drv/city_hub/motor.c

@@ -16,7 +16,6 @@
 
 void _pbdrv_motor_init(void) {
     // TIM3 is used for port C PWM
-    RCC->APB1ENR |= RCC_APB1ENR_TIM3EN;
     TIM3->PSC = 3;      // divide by 4 (= 3 + 1), so ticks are 12MHz
     TIM3->ARR = 10000;  // 12MHz divided by 10k makes 1.2kHz PWM
     TIM3->BDTR |= TIM_BDTR_MOE;

lib/pbio/drv/move_hub/motor.c

@@ -16,7 +16,6 @@
 
 void _pbdrv_motor_init(void) {
     // TIM1 provides PWM for ports A/B
-    RCC->APB2ENR |= RCC_APB2ENR_TIM1EN;
     TIM1->PSC = 3;      // divide by 4 (= 3 + 1), so ticks are 12MHz
     TIM1->ARR = 10000;  // 12MHz divided by 10k makes 1.2kHz PWM
     TIM1->BDTR |= TIM_BDTR_MOE;
@@ -54,7 +53,6 @@ void _pbdrv_motor_init(void) {
     TIM1->EGR |= TIM_EGR_UG;
 
     // TIM3 is used for port C/D PWM
-    RCC->APB1ENR |= RCC_APB1ENR_TIM3EN;
     TIM3->PSC = 3;      // divide by 4 (= 3 + 1), so ticks are 12MHz
     TIM3->ARR = 10000;  // 12MHz divided by 10k makes 1.2kHz PWM
     TIM3->BDTR |= TIM_BDTR_MOE;

lib/pbio/platform/city_hub/platform.c

@@ -96,14 +96,13 @@ void SystemInit(void) {
     // dpgeorge: enable 8-byte stack alignment for IRQ handlers, in accord with EABI
     SCB->CCR |= SCB_CCR_STKALIGN_Msk;
 
-    // Enable all of the shared hardware modules we are using
-
+    // Enable all of the hardware modules we are using
     RCC->AHBENR |= RCC_AHBENR_DMAEN | RCC_AHBENR_GPIOAEN | RCC_AHBENR_GPIOBEN
         | RCC_AHBENR_GPIOCEN | RCC_AHBENR_GPIODEN | RCC_AHBENR_GPIOFEN;
-    RCC->APB1ENR |= RCC_APB1ENR_SPI2EN | RCC_APB1ENR_USART3EN | RCC_APB1ENR_USART4EN;
+    RCC->APB1ENR |= RCC_APB1ENR_TIM3EN | RCC_APB1ENR_SPI2EN | RCC_APB1ENR_USART3EN
+        | RCC_APB1ENR_USART4EN;
     RCC->APB2ENR |= RCC_APB2ENR_SYSCFGCOMPEN | RCC_APB2ENR_TIM16EN | RCC_APB2ENR_TIM15EN;
 
-
     // Keep main power on
     GPIOB->MODER = (GPIOB->MODER & ~GPIO_MODER_MODER11_Msk) | (1 << GPIO_MODER_MODER11_Pos);
     GPIOB->BSRR = GPIO_BSRR_BS_11;

lib/pbio/platform/move_hub/platform.c

@@ -119,12 +119,12 @@ void SystemInit(void) {
     // dpgeorge: enable 8-byte stack alignment for IRQ handlers, in accord with EABI
     SCB->CCR |= SCB_CCR_STKALIGN_Msk;
 
-    // Enable all of the shared hardware modules we are using
-
+    // Enable all of the hardware modules we are using
     RCC->AHBENR |= RCC_AHBENR_GPIOAEN | RCC_AHBENR_GPIOBEN | RCC_AHBENR_GPIOCEN
         | RCC_AHBENR_GPIODEN | RCC_AHBENR_GPIOFEN;
-    RCC->APB1ENR |= RCC_APB1ENR_USART3EN | RCC_APB1ENR_USART4EN;
-    RCC->APB2ENR |= RCC_APB2ENR_SYSCFGCOMPEN | RCC_APB2ENR_TIM16EN | RCC_APB2ENR_TIM15EN;
+    RCC->APB1ENR |= RCC_APB1ENR_TIM3EN | RCC_APB1ENR_USART3EN | RCC_APB1ENR_USART4EN;
+    RCC->APB2ENR |= RCC_APB2ENR_SYSCFGCOMPEN | RCC_APB2ENR_TIM1EN
+        | RCC_APB2ENR_TIM15EN | RCC_APB2ENR_TIM16EN;
 
     // Keep BOOST alive
     GPIOB->BSRR = GPIO_BSRR_BS_11;