platform: move clock enable from light driver to platform.c

Might as well keep all clock config in one place until we have a good
reason to do otherwise.

Author: dlech

lib/pbio/drv/city_hub/light.c

@@ -14,8 +14,6 @@
 
 // setup LED PWMs and pins
 void _pbdrv_light_init(void) {
-    RCC->APB2ENR |= RCC_APB2ENR_TIM16EN | RCC_APB2ENR_TIM15EN;
-
     // RGB values are 0-255, so multiplying by 5 here to limit brightness to
     // 1/5 of max possible without having to do division later. It should also
     // give use smoother steps than the official LEGO firmware since we aren't

lib/pbio/drv/debug/light.c

@@ -14,8 +14,6 @@
 
 // setup LED PWMs and pins
 void _pbdrv_light_init(void) {
-    RCC->APB1ENR |= RCC_APB1ENR_TIM3EN | RCC_APB1ENR_TIM4EN | RCC_APB1ENR_TIM12EN;
-
     // RGB values are 0-255, so setup timer to allow using those values
     // directly in TIMx->CCRx
     TIM12->PSC = 187;

lib/pbio/drv/move_hub/light.c

@@ -14,8 +14,6 @@
 
 // setup LED PWMs and pins
 void _pbdrv_light_init(void) {
-    RCC->APB2ENR |= RCC_APB2ENR_TIM16EN | RCC_APB2ENR_TIM15EN;
-
     // RGB values are 0-255, so multiplying by 5 here to limit brightness to
     // 1/5 of max possible without having to do division later. It should also
     // give use smoother steps than the official LEGO firmware since we aren't

lib/pbio/platform/city_hub/platform.c

@@ -101,7 +101,7 @@ void SystemInit(void) {
     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->APB2ENR |= RCC_APB2ENR_SYSCFGCOMPEN;
+    RCC->APB2ENR |= RCC_APB2ENR_SYSCFGCOMPEN | RCC_APB2ENR_TIM16EN | RCC_APB2ENR_TIM15EN;
 
 
     // Keep main power on

lib/pbio/platform/debug/platform.c

@@ -214,11 +214,12 @@ void SystemInit(void) {
     // dpgeorge: enable 8-byte stack alignment for IRQ handlers, in accord with EABI
     SCB->CCR |= SCB_CCR_STKALIGN_Msk;
 
-    // enable GPIO clocks
+    // enable all of the hardware modules we are using
     RCC->AHB1ENR |= RCC_AHB1ENR_GPIOAEN | RCC_AHB1ENR_GPIOBEN | RCC_AHB1ENR_GPIOCEN |
         RCC_AHB1ENR_GPIODEN | RCC_AHB1ENR_GPIOEEN | RCC_AHB1ENR_GPIOFEN |
         RCC_AHB1ENR_GPIOGEN | RCC_AHB1ENR_DMA2EN;
-    RCC->APB1ENR |= RCC_APB1ENR_TIM2EN | RCC_APB1ENR_USART2EN;
+    RCC->APB1ENR |= RCC_APB1ENR_TIM2EN | RCC_APB1ENR_TIM3EN | RCC_APB1ENR_TIM4EN |
+        RCC_APB1ENR_TIM12EN | RCC_APB1ENR_USART2EN;
     RCC->APB2ENR |= RCC_APB2ENR_ADC3EN | RCC_APB2ENR_USART6EN;
 
     // UART for terminal

lib/pbio/platform/move_hub/platform.c

@@ -124,8 +124,7 @@ void SystemInit(void) {
     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 |= RCC_APB2ENR_SYSCFGCOMPEN | RCC_APB2ENR_TIM16EN | RCC_APB2ENR_TIM15EN;
 
     // Keep BOOST alive
     GPIOB->BSRR = GPIO_BSRR_BS_11;