Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

boards: renesas: Add support for Renesas ek_ra4l1 board #85722

Merged
merged 9 commits into from
Feb 14, 2025

Conversation

thaoluonguw
Copy link
Collaborator

  • Add support for the RA4L1 SoC
  • Add support for the EK-RA4L1 board
  • Update GPIO driver: Only configs for VBATT pin when RA MCU support.
  • Provide test and sample support for the EK-RA4L1 board.

@zephyrbot zephyrbot added area: I2C area: UART Universal Asynchronous Receiver-Transmitter area: PWM Pulse Width Modulation area: GPIO platform: Renesas RA Renesas Electronics Corporation, RA area: SPI SPI bus area: ADC Analog-to-Digital Converter (ADC) labels Feb 13, 2025
@zephyrbot
Copy link
Collaborator

zephyrbot commented Feb 13, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_renesas zephyrproject-rtos/hal_renesas@3237c5d zephyrproject-rtos/hal_renesas@bcbd9fe zephyrproject-rtos/[email protected]

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added manifest manifest-hal_renesas DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Feb 13, 2025
@zephyrbot zephyrbot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Feb 13, 2025
@KhiemNguyenT KhiemNguyenT self-assigned this Feb 13, 2025
west.yml Show resolved Hide resolved
dts/arm/renesas/ra/ra4/r7fa4l1bd4cfp.dtsi Show resolved Hide resolved
dts/arm/renesas/ra/ra4/r7fa4l1bx.dtsi Show resolved Hide resolved
boards/renesas/ek_ra4l1/doc/ek_ra4l1.webp Outdated Show resolved Hide resolved
Supported Features
==================

The below features are currently supported on Zephyr for EK-RA4L1 board:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
The below features are currently supported on Zephyr for EK-RA4L1 board:
The below features are currently supported on Zephyr for the ``ek_ra4l1`` board:

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I updated. Thank you for your suggestion.

Programming and Debugging
*************************

Applications for the ``ek_ra4l1`` board target configuration can be
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Applications for the ``ek_ra4l1`` board target configuration can be
Applications for the ``ek_ra4l1`` board configuration can be

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I updated. Thank you for your suggestion.

boards/renesas/ek_ra4l1/ek_ra4l1.dts Show resolved Hide resolved
max-bitrate = <5000000>;
#phy-cells = <0>;
};

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I fixed it. Thank you for your feedback.

@thaoluonguw thaoluonguw force-pushed the ra_support_ek_ra4l1 branch 2 times, most recently from 68b3e47 to 65b469e Compare February 14, 2025 05:15
@thaoluonguw
Copy link
Collaborator Author

Remove compatible "renesas,ra-flash-hp-controller" for flash due to not support yet.

@thaoluonguw
Copy link
Collaborator Author

@nordicjm : Could you please help me recheck again? Thank you so much.

KhiemNguyenT
KhiemNguyenT previously approved these changes Feb 14, 2025
Copy link
Collaborator

@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.

LGTM. This is a new RA MCU in 2025.

@KhiemNguyenT KhiemNguyenT added this to the v4.1.0 milestone Feb 14, 2025
nordicjm
nordicjm previously approved these changes Feb 14, 2025
duynguyenxa
duynguyenxa previously approved these changes Feb 14, 2025
Update commit ID for hal_renesas to support RA4L1

Signed-off-by: Thao Luong <[email protected]>
Add minimal support for RA4L1 SoC and devicetree

Signed-off-by: Thao Luong <[email protected]>
Add minimal support for EK-RA4L1

Signed-off-by: Thao Luong <[email protected]>
Update GPIO driver for RA: Only configs for VBATT pin when RA
MCU support.

Signed-off-by: Thao Luong <[email protected]>
Add configs and dts node for ek_ra4l1 run on spi_loopback

Signed-off-by: Thao Luong <[email protected]>
Add dts nodes for ek_ra4l1 run on adc_accuracy_test and adc_api

Signed-off-by: Thao Luong <[email protected]>
Add dts nodes for ek_ra4l1 run on pwm_loopback

Signed-off-by: Thao Luong <[email protected]>
Add config and dts node for ek_ra4l1 run on i2c_api

Signed-off-by: Thao Luong <[email protected]>
Add dts node for ek_ra4l1 run on uart_async_api

Signed-off-by: Thao Luong <[email protected]>
@thaoluonguw
Copy link
Collaborator Author

Rebase to solve conflict in west.yml.

@KhiemNguyenT KhiemNguyenT self-requested a review February 14, 2025 09:32
Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

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

looks fine - please address documentation comments in a follow-up commit

Comment on lines +2 to +4

RA4L1 Evaluation Kit
####################
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
RA4L1 Evaluation Kit
####################


.. code-block:: console

west flash -r jlink
Copy link
Collaborator

Choose a reason for hiding this comment

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

jlink is already the default (and only) runner

Suggested change
west flash -r jlink
west flash

@kartben kartben merged commit e44a768 into zephyrproject-rtos:main Feb 14, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ADC Analog-to-Digital Converter (ADC) area: GPIO area: I2C area: PWM Pulse Width Modulation area: SPI SPI bus area: UART Universal Asynchronous Receiver-Transmitter manifest manifest-hal_renesas platform: Renesas RA Renesas Electronics Corporation, RA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants