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.

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

Add PWM driver support for Renesas RZ/G3S #83251

Merged
merged 4 commits into from
Feb 5, 2025
Merged

Add PWM driver support for Renesas RZ/G3S #83251

merged 4 commits into from
Feb 5, 2025

Conversation

binhnguyen2434
Copy link
Collaborator

@binhnguyen2434 binhnguyen2434 commented Dec 20, 2024 •
edited
Loading

Add PWM driver support for Renesas RZ/G3S.

@zephyrbot
Copy link
Collaborator

zephyrbot commented Dec 20, 2024 •
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.

@zephyrbot zephyrbot added manifest manifest-hal_renesas DNM This PR should not be merged (Do Not Merge) labels Dec 20, 2024
@binhnguyen2434 binhnguyen2434 force-pushed the rzg3s_support_pwm branch 3 times, most recently from ae4d286 to f4f0a55 Compare January 3, 2025 03:42
@zephyrbot zephyrbot removed the DNM This PR should not be merged (Do Not Merge) label Jan 14, 2025
@kartben
Copy link
Collaborator

kartben commented Jan 21, 2025

@anangl please take a look or re-assign as you see fit. Thanks!

@binhnguyen2434 binhnguyen2434 mentioned this pull request Feb 4, 2025
Closed
Hieu Nguyen added 4 commits February 4, 2025 16:54
@binhnguyen2434
drivers: pwm: Initial support for RZ/G3S
c6ca76a 
Add PWM driver support for Renesas RZ/G3S

Signed-off-by: Hieu Nguyen <hieu.nguyen.ym@bp.renesas.com>
Signed-off-by: Binh Nguyen <binh.nguyen.xw@renesas.com>
@binhnguyen2434
dts: renesas: Add PWM support for RZ/G3S
e38a46a 
Add PWM nodes to Renesas RZ/G3S devicetree

Signed-off-by: Hieu Nguyen <hieu.nguyen.ym@bp.renesas.com>
Signed-off-by: Binh Nguyen <binh.nguyen.xw@renesas.com>
@binhnguyen2434
boards: renesas: rzg3s_smarc: Add PWM support
9e04dcf 
Add PWM support for board RZ/G3S-SMARC

Signed-off-by: Hieu Nguyen <hieu.nguyen.ym@bp.renesas.com>
Signed-off-by: Binh Nguyen <binh.nguyen.xw@renesas.com>
@binhnguyen2434
tests: drivers: pwm: Add support for RZ/G3S-SMARC
76c607b 
Enable PWM driver tests for RZ/G3S

Signed-off-by: Hieu Nguyen <hieu.nguyen.ym@bp.renesas.com>
Signed-off-by: Binh Nguyen <binh.nguyen.xw@renesas.com>
@binhnguyen2434
Copy link
Collaborator Author

Hi @henrikbrixandersen,
I just rebased to resolve conflict.
Could you please approve again?

@binhnguyen2434
Copy link
Collaborator Author

Hi @kartben,
Please help to proceed next step to merge.
Thank you.

@kartben
Copy link
Collaborator

kartben commented Feb 5, 2025 •
edited
Loading

Hi @kartben, Please help to proceed next step to merge. Thank you.

@binhnguyen2434 no need to ping :) this will be picked up during an upcoming merge batch now that you have the required approvals

@binhnguyen2434
Copy link
Collaborator Author

Hi @kartben, Please help to proceed next step to merge. Thank you.

@binhnguyen2434 no need to ping :) this will be picked up during an upcoming merge batch now that you have the required approvals

@kartben Noted, thank you for the update! I appreciate your support.

@KhiemNguyenT KhiemNguyenT self-requested a review February 5, 2025 16:30
@kartben kartben merged commit a93a302 into zephyrproject-rtos:main Feb 5, 2025
27 checks passed
@binhnguyen2434 binhnguyen2434 mentioned this pull request Feb 6, 2025
Open
@binhnguyen2434 binhnguyen2434 deleted the rzg3s_support_pwm branch February 6, 2025 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@henrikbrixandersen henrikbrixandersen henrikbrixandersen approved these changes

@anangl anangl anangl approved these changes

@KhiemNguyenT KhiemNguyenT KhiemNguyenT approved these changes

@andyross andyross Awaiting requested review from andyross

@teburd teburd Awaiting requested review from teburd

@tgorochowik tgorochowik Awaiting requested review from tgorochowik

Assignees

@anangl anangl

Labels
area: PWM Pulse Width Modulation area: Timer Timer platforms: Renesas RZ
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@binhnguyen2434 @zephyrbot @kartben @henrikbrixandersen @anangl @KhiemNguyenT