driver: uart stm32: Check irq enabled in API calls

nvincent-vossloh · nashif · commit eb534d39d21d · 2021-01-11T16:55:29.000-05:00
When calling irq_rx_ready or irq_tx_ready API, return the logical AND
between the irq status and the enable of that irq.

Signed-off-by: Nicolas VINCENT &lt;nicolas.vincent@vossloh.com&gt;
diff --git a/drivers/serial/uart_stm32.c b/drivers/serial/uart_stm32.c
@@ -533,7 +533,8 @@ static int uart_stm32_irq_tx_ready(const struct device *dev)
 {
 	USART_TypeDef *UartInstance = UART_STRUCT(dev);
 
-	return LL_USART_IsActiveFlag_TXE(UartInstance);
+	return LL_USART_IsActiveFlag_TXE(UartInstance) &&
+		LL_USART_IsEnabledIT_TC(UartInstance);
 }
 
 static int uart_stm32_irq_tx_complete(const struct device *dev)
@@ -561,7 +562,8 @@ static int uart_stm32_irq_rx_ready(const struct device *dev)
 {
 	USART_TypeDef *UartInstance = UART_STRUCT(dev);
 
-	return LL_USART_IsActiveFlag_RXNE(UartInstance);
+	return LL_USART_IsActiveFlag_RXNE(UartInstance) &&
+		LL_USART_IsEnabledIT_RXNE(UartInstance);
 }
 
 static void uart_stm32_irq_err_enable(const struct device *dev)

Original file line number	Diff line number	Diff line change
`@@ -533,7 +533,8 @@ static int uart_stm32_irq_tx_ready(const struct device *dev)`
`533`	`533`	`{`
`534`	`534`	`USART_TypeDef *UartInstance = UART_STRUCT(dev);`
`535`	`535`
`536`		`- return LL_USART_IsActiveFlag_TXE(UartInstance);`
	`536`	`+ return LL_USART_IsActiveFlag_TXE(UartInstance) &&`
	`537`	`+ LL_USART_IsEnabledIT_TC(UartInstance);`
`537`	`538`	`}`
`538`	`539`
`539`	`540`	`static int uart_stm32_irq_tx_complete(const struct device *dev)`
`@@ -561,7 +562,8 @@ static int uart_stm32_irq_rx_ready(const struct device *dev)`
`561`	`562`	`{`
`562`	`563`	`USART_TypeDef *UartInstance = UART_STRUCT(dev);`
`563`	`564`
`564`		`- return LL_USART_IsActiveFlag_RXNE(UartInstance);`
	`565`	`+ return LL_USART_IsActiveFlag_RXNE(UartInstance) &&`
	`566`	`+ LL_USART_IsEnabledIT_RXNE(UartInstance);`
`565`	`567`	`}`
`566`	`568`
`567`	`569`	`static void uart_stm32_irq_err_enable(const struct device *dev)`