Skip to content

boards: renesas: Add support for EK-RA2L1 board#85688

Merged
fabiobaltieri merged 2 commits intozephyrproject-rtos:mainfrom
renesas:support-ra2-PR2.0
Feb 14, 2025
Merged

boards: renesas: Add support for EK-RA2L1 board#85688
fabiobaltieri merged 2 commits intozephyrproject-rtos:mainfrom
renesas:support-ra2-PR2.0

Conversation

@thaoluonguw
Copy link
Contributor

Add minimal support for EK-RA2L1 board

  • Add support for the RA2L1 SoC
  • Add support for the EK-RA2L1 boards

This PR refer #77589 and update to adapt with current implementation as other RA boards.

@zephyrbot zephyrbot added the platform: Renesas RA Renesas Electronics Corporation, RA label Feb 12, 2025
* Copyright (c) 2021-2024 MUNIC SA
* Copyright (c) 2024 Renesas Electronics Corporation
*
* Renesas RA2AL1 MCU series device tree
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Renesas RA2AL1 MCU series device tree
* Renesas RA2L1 MCU series device tree
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed it, please recheck. Thank you for your feedback.

Comment on lines 37 to 39
uint32_t key;

key = irq_lock();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need irq lock and unlock here. Please help to remove them.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed it, please recheck. Thank you for your feedback.

SystemCoreClock = BSP_MOCO_HZ;
g_protect_pfswe_counter = 0;

irq_unlock(key);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

diito

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed it, please recheck. Thank you for your feedback.

Supported Features
==================

The Renesas ``ek_ra2l1`` board configuration supports the following
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The Renesas ``ek_ra2l1`` board configuration supports the following
The Renesas ``ek_ra2l1`` board supports the following
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I updated it. Thank you for your suggestion.

duynguyenxa
duynguyenxa previously approved these changes Feb 13, 2025
KhiemNguyenT
KhiemNguyenT previously approved these changes Feb 13, 2025
Copy link

@KhiemNguyenT KhiemNguyenT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look good to me.

@KhiemNguyenT KhiemNguyenT added this to the v4.1.0 milestone Feb 13, 2025
@thaoluonguw
Copy link
Contributor Author

@nordicjm: I updated all as your points out. Could you please take a look again. Thank you so much.
@soburi : Could you please take a look? Thank you so much.

Add basic support for Renesas RA2L1 SOC series.

Signed-off-by: Khoa Nguyen <khoa.nguyen.xh@renesas.com>
Signed-off-by: Thao Luong <thao.luong.uw@renesas.com>
Add inital support for the Renesas EK-RA2L1 evaluation board.

Signed-off-by: Khoa Nguyen <khoa.nguyen.xh@renesas.com>
Signed-off-by: Thao Luong <thao.luong.uw@renesas.com>
Comment on lines +37 to +40

SystemCoreClock = BSP_MOCO_HZ;
g_protect_pfswe_counter = 0;

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

random spacing here

@fabiobaltieri fabiobaltieri merged commit 493ba5c into zephyrproject-rtos:main Feb 14, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

platform: Renesas RA Renesas Electronics Corporation, RA

9 participants