stop hardcoding everything except for clock

Author: GabeMillikan

BLINKY/G4HELLO/Core/Src/main.c

@@ -117,8 +117,28 @@ int main(void)
 	USARTConfig config = {
 	    .instance = USART1,
 	    .tx_queue_length = 16,
-	    .baud_rate = 115200,
 	    .on_rx_byte = received_byte_callback,
+	    .gpio_port = GPIOB,
+	    .ll_gpio =
+		&(LL_GPIO_InitTypeDef){
+		    .Pin = LL_GPIO_PIN_6 | LL_GPIO_PIN_7,
+		    .Mode = LL_GPIO_MODE_ALTERNATE,
+		    .Alternate = LL_GPIO_AF_7,
+		    .OutputType = LL_GPIO_OUTPUT_PUSHPULL,
+		    .Pull = LL_GPIO_PULL_UP,
+		    .Speed = LL_GPIO_SPEED_FREQ_HIGH,
+		},
+	    .ll_usart =
+		&(LL_USART_InitTypeDef){
+		    .PrescalerValue = LL_USART_PRESCALER_DIV1,
+		    .BaudRate = 115200,
+		    .DataWidth = LL_USART_DATAWIDTH_8B,
+		    .StopBits = LL_USART_STOPBITS_1,
+		    .Parity = LL_USART_PARITY_NONE,
+		    .TransferDirection = LL_USART_DIRECTION_TX_RX,
+		    .HardwareFlowControl = LL_USART_HWCONTROL_NONE,
+		    .OverSampling = LL_USART_OVERSAMPLING_16,
+		},
 	};
 	USARTHandle *handle = usart_init_peripheral(&config);
 	for (uint8_t i = 0; i < 30; i++) {

Lib/Peripherals/USART/usart.c

@@ -97,27 +97,8 @@ void usart_init_hardware(USARTConfig *config, USARTHandle *handle)
 	// Select PCLK2 as USART1 clock source
 	LL_RCC_SetUSARTClockSource(LL_RCC_USART1_CLKSOURCE_PCLK2);
 
-	// Configure PB6 = TX, PB7 = RX
-	LL_GPIO_InitTypeDef gpio = {0};
-	gpio.Pin = LL_GPIO_PIN_6 | LL_GPIO_PIN_7;
-	gpio.Mode = LL_GPIO_MODE_ALTERNATE;
-	gpio.Alternate = LL_GPIO_AF_7;
-	gpio.OutputType = LL_GPIO_OUTPUT_PUSHPULL;
-	gpio.Pull = LL_GPIO_PULL_UP;
-	gpio.Speed = LL_GPIO_SPEED_FREQ_HIGH;
-	LL_GPIO_Init(GPIOB, &gpio);
-
-	// Configure USART
-	LL_USART_InitTypeDef us = {0};
-	us.PrescalerValue = LL_USART_PRESCALER_DIV1;
-	us.BaudRate = config->baud_rate;
-	us.DataWidth = LL_USART_DATAWIDTH_8B;
-	us.StopBits = LL_USART_STOPBITS_1;
-	us.Parity = LL_USART_PARITY_NONE;
-	us.TransferDirection = LL_USART_DIRECTION_TX_RX;
-	us.HardwareFlowControl = LL_USART_HWCONTROL_NONE;
-	us.OverSampling = LL_USART_OVERSAMPLING_16;
-	LL_USART_Init(handle->instance, &us);
+	LL_GPIO_Init(config->gpio_port, config->ll_gpio);
+	LL_USART_Init(handle->instance, config->ll_usart);
 
 	LL_USART_ConfigAsyncMode(handle->instance);
 

Lib/Peripherals/USART/usart.h

@@ -6,8 +6,10 @@ typedef void (*USART_RxByteCallback)(uint8_t byte);
 typedef struct {
 	USART_TypeDef *instance;  // i.e. USART1, USART2, etc.
 	uint32_t tx_queue_length; // in # of messages
-	uint32_t baud_rate;
 	USART_RxByteCallback on_rx_byte;
+	GPIO_TypeDef *gpio_port; // i.e. GPIOA, GPIOB, etc.
+	LL_GPIO_InitTypeDef *ll_gpio;
+	LL_USART_InitTypeDef *ll_usart;
 } USARTConfig;
 
 typedef struct usart_handle_st USARTHandle;