drivers: serial: stm32: simplify logic

yishai1999 · kartben · commit 7fe85f8cb48a · 2025-06-06T10:11:29.000+02:00
Simplify the logic for the `uart_stm32_cfg2ll_databits` function
regarding `LL_USART_DATAWIDTH_9B` being defined or not.

Signed-off-by: Yishai Jaffe &lt;yishai1999@gmail.com&gt;
diff --git a/drivers/serial/uart_stm32.c b/drivers/serial/uart_stm32.c
@@ -411,13 +411,11 @@ static inline uint32_t uart_stm32_cfg2ll_databits(enum uart_config_data_bits db,
 #endif	/* LL_USART_DATAWIDTH_9B */
 	case UART_CFG_DATA_BITS_8:
 	default:
-		if (p == UART_CFG_PARITY_NONE) {
-			return LL_USART_DATAWIDTH_8B;
 #ifdef LL_USART_DATAWIDTH_9B
-		} else {
+		if (p != UART_CFG_PARITY_NONE) {
 			return LL_USART_DATAWIDTH_9B;
-#endif
 		}
+#endif
 		return LL_USART_DATAWIDTH_8B;
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -411,13 +411,11 @@ static inline uint32_t uart_stm32_cfg2ll_databits(enum uart_config_data_bits db,`
`411`	`411`	`#endif /* LL_USART_DATAWIDTH_9B */`
`412`	`412`	`case UART_CFG_DATA_BITS_8:`
`413`	`413`	`default:`
`414`		`- if (p == UART_CFG_PARITY_NONE) {`
`415`		`- return LL_USART_DATAWIDTH_8B;`
`416`	`414`	`#ifdef LL_USART_DATAWIDTH_9B`
`417`		`- } else {`
	`415`	`+ if (p != UART_CFG_PARITY_NONE) {`
`418`	`416`	`return LL_USART_DATAWIDTH_9B;`
`419`		`-#endif`
`420`	`417`	`}`
	`418`	`+#endif`
`421`	`419`	`return LL_USART_DATAWIDTH_8B;`
`422`	`420`	`}`
`423`	`421`	`}`