diff --git a/ports/renesas-ra/Makefile b/ports/renesas-ra/Makefile index 13be532cb239..7ffb3b1d0ec7 100644 --- a/ports/renesas-ra/Makefile +++ b/ports/renesas-ra/Makefile @@ -359,8 +359,12 @@ HAL_SRC_C += $(addprefix $(HAL_DIR)/ra/fsp/src/bsp/mcu/all/,\ HAL_SRC_C += $(addprefix $(HAL_DIR)/ra/fsp/src/,\ r_ioport/r_ioport.c \ + r_sci_uart/r_sci_uart.c \ ) +CFLAGS_FSP = -Wno-unused-variable -Wno-unused-function +$(BUILD)/lib/fsp/ra/fsp/src/r_sci_uart/r_sci_uart.o: CFLAGS += $(CFLAGS_FSP) + ifeq ($(USE_FSP_LPM), 1) CFLAGS += -DUSE_FSP_LPM HAL_SRC_C += $(HAL_DIR)/ra/fsp/src/r_lpm/r_lpm.c diff --git a/ports/renesas-ra/fsp_cfg/r_sci_uart_cfg.h b/ports/renesas-ra/fsp_cfg/r_sci_uart_cfg.h new file mode 100644 index 000000000000..bbc6dc5522c6 --- /dev/null +++ b/ports/renesas-ra/fsp_cfg/r_sci_uart_cfg.h @@ -0,0 +1,25 @@ +#ifndef R_SCI_UART_CFG_H_ +#define R_SCI_UART_CFG_H_ +#ifdef __cplusplus +extern "C" { +#endif + +#define SCI_UART_CFG_PARAM_CHECKING_ENABLE (BSP_CFG_PARAM_CHECKING_ENABLE) +#define SCI_UART_CFG_FIFO_SUPPORT (0) +#define SCI_UART_CFG_DTC_SUPPORTED (0) +#define SCI_UART_CFG_FLOW_CONTROL_SUPPORT (0) +#define SCI_UART_CFG_RS485_SUPPORT (0) + +/* + * Disable r_sci_uart.c's interrupt handlers by defining followings + * because the renesas-ra has own interrupt handlers for SCI and + * symbol conflict happens. + */ + +#define SCI_UART_CFG_TX_ENABLE (0) +#define SCI_UART_CFG_RX_ENABLE (0) + +#ifdef __cplusplus +} +#endif +#endif /* R_SCI_UART_CFG_H_ */ diff --git a/ports/renesas-ra/ra/ra_sci.c b/ports/renesas-ra/ra/ra_sci.c index 3f8bf2444401..5d1406724868 100644 --- a/ports/renesas-ra/ra/ra_sci.c +++ b/ports/renesas-ra/ra/ra_sci.c @@ -28,6 +28,7 @@ #include "ra_int.h" #include "ra_utils.h" #include "ra_sci.h" +#include "r_sci_uart.h" #if !defined(RA_PRI_UART) #define RA_PRI_UART (1) @@ -1100,20 +1101,24 @@ static void ra_sci_fifo_init(uint32_t ch) { void ra_sci_set_baud(uint32_t ch, uint32_t baud) { uint32_t idx = ch_to_idx[ch]; R_SCI0_Type *sci_reg = sci_regs[idx]; - // Only works for 115200 bps - // ToDo: support other bps - if (baud == 0) { - /* ABCS=1 */ - sci_reg->SMR_b.CKS = 0; /* PCLKA */ - sci_reg->BRR = (uint8_t)((int)PCLK / SCI_DEF_BAUD / 16 - 1); - } else if (baud > 19200) { - /* ABCS=1 */ - sci_reg->SMR_b.CKS = 0; /* PCLKA */ - sci_reg->BRR = (uint8_t)((int)PCLK / baud / 16 - 1); - } else { - /* ABCS=1 */ - sci_reg->SMR_b.CKS = 2; /* PCLKA/16 */ - sci_reg->BRR = (uint8_t)((int)PCLK / baud / 256 - 1); + baud_setting_t baud_setting; + + // baudrate_modulation=1(enabled), baudrate_max_err=5%(general value for FSP example). + if (R_SCI_UART_BaudCalculate(baud, 1, 5000, &baud_setting) == FSP_SUCCESS) { + // Disable transmitter and receiver. + uint8_t scr = sci_reg->SCR; + sci_reg->SCR = scr & ~(0xF0); + + // Change baudrate + sci_reg->BRR = baud_setting.brr; + // Update CKS(b1:b0) + sci_reg->SMR_b.CKS = (uint8_t)(0x03U & baud_setting.cks); + sci_reg->MDDR = baud_setting.mddr; + // Update BGDM(b6)/ABCS(b4)/ABCSE(b3)/BRME(b2) bits + sci_reg->SEMR = (uint8_t)((sci_reg->SEMR & ~(0x5cU)) | (baud_setting.semr_baudrate_bits & 0x5cU)); + + // Restore SCR + sci_reg->SCR = scr; } } @@ -1131,8 +1136,8 @@ void ra_sci_init_with_flow(uint32_t ch, uint32_t tx_pin, uint32_t rx_pin, uint32 sci_cb[idx] = 0; ra_sci_init_flag[idx]++; } else { - ra_sci_init_flag[idx]++; - return; + ra_sci_irq_disable(ch); + ra_sci_module_stop(ch); } ra_sci_module_start(ch); ra_sci_tx_set_pin(tx_pin);