Skip to content

Add minimal support for Renesas RZ/G2UL SMARC #90710

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Jun 9, 2025
Merged

Add minimal support for Renesas RZ/G2UL SMARC #90710

merged 7 commits into from
Jun 9, 2025

Conversation

nhutnguyenkc
Copy link
Collaborator

@nhutnguyenkc nhutnguyenkc commented May 28, 2025 •
edited
Loading

Add minimal support for RZ/G2UL SMARC as following:

Interface Controller Driver/Component
GPIO on-chip gpio
Pinctrl on-chip pinctrl
UART on-chip serial

Need: zephyrproject-rtos/hal_renesas#102

Doc: https://builds.zephyrproject.io/zephyr/pr/90710/docs/boards/renesas/rzg2ul_smarc/doc/index.html

@github-actions GitHub Actions
Copy link

github-actions bot commented May 28, 2025 •
edited
Loading

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

Name Old Revision New Revision Diff

✅ All manifest checks OK

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

@github-actions github-actions bot added manifest manifest-hal_renesas DNM (manifest) This PR should not be merged (controlled by action-manifest) labels May 28, 2025
@nhutnguyenkc nhutnguyenkc marked this pull request as ready for review May 28, 2025 05:44
@github-actions github-actions bot added area: GPIO area: Pinctrl area: UART Universal Asynchronous Receiver-Transmitter platforms: Renesas RZ labels May 28, 2025
nordicjm
nordicjm reviewed May 28, 2025
View reviewed changes
@nhutnguyenkc nhutnguyenkc force-pushed the rzg2ul_minimal_support branch from 8a8de44 to 1cdcf2c Compare May 29, 2025 07:25
@binhnguyen2434 binhnguyen2434 requested a review from nordicjm June 4, 2025 01:48
nordicjm
nordicjm previously approved these changes Jun 4, 2025
View reviewed changes
@nordicjm nordicjm requested a review from kartben June 4, 2025 06:55
kartben
kartben previously approved these changes Jun 4, 2025
View reviewed changes
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.

+1 for docs, great work!

@nhutnguyenkc nhutnguyenkc dismissed stale reviews from kartben and nordicjm via 3c60af1 June 4, 2025 07:14
@nhutnguyenkc nhutnguyenkc force-pushed the rzg2ul_minimal_support branch from 1cdcf2c to 3c60af1 Compare June 4, 2025 07:14
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Jun 4, 2025
@nhutnguyenkc nhutnguyenkc reopened this Jun 4, 2025
kartben
kartben previously approved these changes Jun 4, 2025
View reviewed changes
Phuc Pham added 5 commits June 5, 2025 09:46
@nhutnguyenkc
soc: renesas: Add initial support for Renesas RZ/G2UL
d950c64 
Add initial support for Renesas RZ/G2UL

Signed-off-by: Phuc Pham <[email protected]>
Signed-off-by: Nhut Nguyen <[email protected]>
@nhutnguyenkc
dts: arm: renesas: Add support for Renesas RZ/G2UL
ff29e62 
Add devicetree to support for Renesas RZ/G2UL

Signed-off-by: Phuc Pham <[email protected]>
Signed-off-by: Nhut Nguyen <[email protected]>
@nhutnguyenkc
boards: renesas: Add minimal support for Renesas RZ/G2UL-SMARC
f4aa92a 
Add minimal support for board Renesas RZ/G2UL-SMARC

Signed-off-by: Phuc Pham <[email protected]>
Signed-off-by: Nhut Nguyen <[email protected]>
@nhutnguyenkc
drivers: pinctrl: Add support for Renesas RZ/G2UL
2c8256f 
Add pinctrl support for Renesas RZ/G2UL

Signed-off-by: Phuc Pham <[email protected]>
Signed-off-by: Nhut Nguyen <[email protected]>
@nhutnguyenkc
tests: drivers: serial: Add support for Renesas RZ/G2UL-SMARC
d2b9d8b 
Enable serial driver tests for RZ/G2UL-SMARC

Signed-off-by: Phuc Pham <[email protected]>
Signed-off-by: Nhut Nguyen <[email protected]>
@nhutnguyenkc nhutnguyenkc dismissed stale reviews from binhnguyen2434 and kartben via 0687c14 June 5, 2025 02:58
@nhutnguyenkc nhutnguyenkc force-pushed the rzg2ul_minimal_support branch from 3c60af1 to 0687c14 Compare June 5, 2025 02:58
@nhutnguyenkc
Copy link
Collaborator Author

Hi, I just rebased to fix the Rust CI failure. Could you please refresh your reviews? Thanks

kartben
kartben previously approved these changes Jun 5, 2025
View reviewed changes
binhnguyen2434
binhnguyen2434 previously approved these changes Jun 5, 2025
View reviewed changes
Phuc Pham added 2 commits June 5, 2025 11:35
@nhutnguyenkc
drivers: gpio: Add support for RZ/G2UL
c2123a5 
Add GPIO support for RZ/G2UL

Signed-off-by: Phuc Pham <[email protected]>
Signed-off-by: Nhut Nguyen <[email protected]>
@nhutnguyenkc
tests: drivers: gpio: Add support for RZ/G2UL-SMARC
aa66f1a 
Enable GPIO driver tests for RZ/G2UL-SMARC

Signed-off-by: Phuc Pham <[email protected]>
Signed-off-by: Nhut Nguyen <[email protected]>
@nhutnguyenkc nhutnguyenkc dismissed stale reviews from binhnguyen2434 and kartben via aa66f1a June 5, 2025 05:39
@nhutnguyenkc nhutnguyenkc force-pushed the rzg2ul_minimal_support branch from 0687c14 to aa66f1a Compare June 5, 2025 05:39
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Jun 5, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@nhutnguyenkc
Copy link
Collaborator Author

Hi, sorry for the update after approvals. But the last push is to fix a CI failure due to the previous rebase. Could you please help me refresh your reviews. Thanks

Fix the below CI failure

INFO    - The following issues were found (showing the top 10 items):
INFO    - 1) drivers.gpio.1pin on rzg2ul_smarc/r9a07g043u11gbg/cm33 error (Build failure - error: implicit declaration of function 'GPIO_RZ_TINT_STATUS_REG_CLEAR'; did you mean 'GPIO_RZ_TINT_STATUS_REG_GET'? [-Werror=implicit-function-declaration])
INFO    - 2) drivers.gpio.2pin on rzg2ul_smarc/r9a07g043u11gbg/cm33 error (Build failure - error: implicit declaration of function 'GPIO_RZ_TINT_STATUS_REG_CLEAR'; did you mean 'GPIO_RZ_TINT_STATUS_REG_GET'? [-Werror=implicit-function-declaration])

@nhutnguyenkc
Copy link
Collaborator Author

nhutnguyenkc commented Jun 6, 2025 •
edited
Loading

@kartben , @nordicjm , could you please revisit it. It is just a small change in drivers/gpio/gpio_renesas_rz.h to resolve a conflict after rebasing. Thanks

@fabiobaltieri fabiobaltieri merged commit 8b438d2 into zephyrproject-rtos:main Jun 9, 2025
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kartben kartben kartben approved these changes

@nordicjm nordicjm nordicjm approved these changes

@binhnguyen2434 binhnguyen2434 binhnguyen2434 approved these changes

@dcpleung dcpleung Awaiting requested review from dcpleung

@gmarull gmarull Awaiting requested review from gmarull

@henrikbrixandersen henrikbrixandersen Awaiting requested review from henrikbrixandersen

@mnkp mnkp Awaiting requested review from mnkp

@tgorochowik tgorochowik Awaiting requested review from tgorochowik

Assignees

@dcpleung dcpleung

@binhnguyen2434 binhnguyen2434

Labels
area: GPIO area: Pinctrl area: UART Universal Asynchronous Receiver-Transmitter platforms: Renesas RZ
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@nhutnguyenkc @kartben @nordicjm @binhnguyen2434 @fabiobaltieri @dcpleung