From 92c7532d8e461f3fd9ea44f4d024ee3ce156e5e8 Mon Sep 17 00:00:00 2001
From: Takeo Takahashi <takeo.takahashi.xv@renesas.com>
Date: Thu, 1 Jun 2023 17:56:28 +0900
Subject: [PATCH] renesas-ra: Support changing baudrate for UART.

* Use R_SCI_UART_BaudCalculate() of fsp/src/r_sci_uart/r_sci_uart.c
* Support UART.init(baudrate)

Signed-off-by: Takeo Takahashi <takeo.takahashi.xv@renesas.com>
---
 ports/renesas-ra/Makefile                 |  4 +++
 ports/renesas-ra/fsp_cfg/r_sci_uart_cfg.h | 25 +++++++++++++++
 ports/renesas-ra/ra/ra_sci.c              | 37 +++++++++++++----------
 3 files changed, 50 insertions(+), 16 deletions(-)
 create mode 100644 ports/renesas-ra/fsp_cfg/r_sci_uart_cfg.h

diff --git a/ports/renesas-ra/Makefile b/ports/renesas-ra/Makefile
index 13be532cb2..7ffb3b1d0e 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 0000000000..bbc6dc5522
--- /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 3f8bf24444..5d14067248 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);