hal: renesas: rx: Add HAL support for Renesas RX26T #121
Conversation
quytranpzz
force-pushed
the
rx_zephyr_support_soc_rx26t
branch
from
c55a752 to
b83b211
Compare
quytranpzz
force-pushed
the
rx_zephyr_support_soc_rx26t
branch
from
b83b211 to
bfa3135
Compare
| #include "r_bsp_cpu.h" | ||
| #include "mcu_clocks.h" |
There was a problem hiding this comment.
@quytranpzz: It seems that lacked of include for r_bsp_locking.h?
| #include "mcu/rx26t/mcu_mapped_interrupts_private.h" | ||
| #include "mcu/rx26t/mcu_mapped_interrupts.h" |
There was a problem hiding this comment.
@quytranpzz : Why we need full path for these includes?
There was a problem hiding this comment.
I updated the above lines to use full-path includes as here, for consistency with FIT and other MCUs.
There was a problem hiding this comment.
Incorrect DISCLAIMER.
There was a problem hiding this comment.
Sorry for the mistake, I have updated this file into latest version to correct the disclaimer.
| #endif | ||
|
|
||
| /* Set PLL Multiplier. */ | ||
| SYSTEM.PLLCR.BIT.STC = ((uint8_t)((float)BSP_CFG_PLL_MUL * 2.0f)) - 1; |
There was a problem hiding this comment.
Different from RDPv1.47. Please separate your update for Zephyr into one commit and record it in the readme file.
There was a problem hiding this comment.
I have seperated it in another commit for this change. Thanks you
| error = (((float)pclk / ((baud * divisor) * tmp)) - 1) * 100; | ||
| abs_error = (error < 0) ? (-error) : error; | ||
|
|
||
| if ((abs_error <= 1.0f) || (SCI_MODE_ASYNC != hdl->mode)) |
There was a problem hiding this comment.
Different from RDPv1.47.
There was a problem hiding this comment.
I have seperated it in another commit for this change. Thanks you
Add HAL support for RX26T using RDP Signed-off-by: Hau Ho <[email protected]> Signed-off-by: Phi Tran <[email protected]>
Fix the implicit conversion from float to double warnings in RX26T source files Signed-off-by: Quy Tran <[email protected]>
quytranpzz
force-pushed
the
rx_zephyr_support_soc_rx26t
branch
from
bfa3135 to
b5e59be
Compare
Uh oh!
There was an error while loading. Please reload this page.