This is an automated email from the ASF dual-hosted git repository. xiaoxiang781216 pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git
commit d780c7c1cb14df8d7614e215371aad33d2c25b35 Author: Junbo Zheng <[email protected]> AuthorDate: Wed Oct 14 23:35:35 2026 +0800 drivers/serial/uart_16550: Fix missing semicolon in u16550_dmarxfree() Commit ceb2921d796 ("drivers: prepare 16550 UART driver to support PCI") dropped the semicolon after the priv->ops->dmachan() assignment in u16550_dmarxfree(), so any build with 16550 UART DMA enabled (CONFIG_16550_UARTx_DMA=y) fails to compile: drivers/serial/uart_16550.c: In function 'u16550_dmarxfree': drivers/serial/uart_16550.c:1491: error: expected ',' or ';' before 'if' Restore the semicolon, matching the TX-side twin in u16550_dmatxavail(). Fixes: ceb2921d796 ("drivers: prepare 16550 UART driver to support PCI") Assisted-by: Claude Code (glm-5.3) <[email protected]> Signed-off-by: Junbo Zheng <[email protected]> --- drivers/serial/uart_16550.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/serial/uart_16550.c b/drivers/serial/uart_16550.c index d51574ad310..f2b7acbf7da 100644 --- a/drivers/serial/uart_16550.c +++ b/drivers/serial/uart_16550.c @@ -1487,7 +1487,7 @@ static void u16550_dmarxfree(FAR struct uart_dev_s *dev) if (priv->chanrx == NULL) { - priv->chanrx = priv->ops->dmachan(priv, priv->dmarx) + priv->chanrx = priv->ops->dmachan(priv, priv->dmarx); if (priv->chanrx == NULL) { return; /* Fail to get DMA channel */
