From e1679927697a5182c2728616ea5c16e8fd79a9a4 Mon Sep 17 00:00:00 2001 From: grumpyengineer Date: Tue, 21 Feb 2023 19:32:32 +0000 Subject: [PATCH 1/2] Change temp to uint32_t in serial_format Fix bug where calling set_format in MBED (which calls serial_format) causes the CTRL register to become corrupt due to saving its initial value to a uint8_t when the register is 32 bit wide --- .../TARGET_MIMXRT1050/TARGET_EVK/serial_api.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/targets/TARGET_NXP/TARGET_MCUXpresso_MCUS/TARGET_MIMXRT1050/TARGET_EVK/serial_api.c b/targets/TARGET_NXP/TARGET_MCUXpresso_MCUS/TARGET_MIMXRT1050/TARGET_EVK/serial_api.c index 0af83e98d30..78debf71f87 100644 --- a/targets/TARGET_NXP/TARGET_MCUXpresso_MCUS/TARGET_MIMXRT1050/TARGET_EVK/serial_api.c +++ b/targets/TARGET_NXP/TARGET_MCUXpresso_MCUS/TARGET_MIMXRT1050/TARGET_EVK/serial_api.c @@ -91,7 +91,7 @@ void serial_baud(serial_t *obj, int baudrate) void serial_format(serial_t *obj, int data_bits, SerialParity parity, int stop_bits) { LPUART_Type *base = uart_addrs[obj->index]; - uint8_t temp; + uint32_t temp; /* Set bit count and parity mode. */ temp = base->CTRL & ~(LPUART_CTRL_PE_MASK | LPUART_CTRL_PT_MASK | LPUART_CTRL_M_MASK); if (parity != ParityNone) { From fbf4ac03d7623b50432c027e0a090a664608f2aa Mon Sep 17 00:00:00 2001 From: grumpyengineer Date: Tue, 21 Feb 2023 19:51:10 +0000 Subject: [PATCH 2/2] Change temp to uint32_t in serial_format Fix bug where calling set_format in MBED (which calls serial_format) causes the CTRL register to become corrupt due to saving its initial value to a uint8_t when the register is 32 bit wide --- .../TARGET_MIMXRT1170/TARGET_EVK/serial_api.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/targets/TARGET_NXP/TARGET_MCUXpresso_MCUS/TARGET_MIMXRT1170/TARGET_EVK/serial_api.c b/targets/TARGET_NXP/TARGET_MCUXpresso_MCUS/TARGET_MIMXRT1170/TARGET_EVK/serial_api.c index c479ade934e..e02ca5d7655 100644 --- a/targets/TARGET_NXP/TARGET_MCUXpresso_MCUS/TARGET_MIMXRT1170/TARGET_EVK/serial_api.c +++ b/targets/TARGET_NXP/TARGET_MCUXpresso_MCUS/TARGET_MIMXRT1170/TARGET_EVK/serial_api.c @@ -91,7 +91,7 @@ void serial_baud(serial_t *obj, int baudrate) void serial_format(serial_t *obj, int data_bits, SerialParity parity, int stop_bits) { LPUART_Type *base = uart_addrs[obj->index]; - uint8_t temp; + uint32_t temp; /* Set bit count and parity mode. */ temp = base->CTRL & ~(LPUART_CTRL_PE_MASK | LPUART_CTRL_PT_MASK | LPUART_CTRL_M_MASK); if (parity != ParityNone) {