Skip to content

Add CC-RH port for Renesas F1Kx devices #1100

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

Merged
merged 4 commits into from
Aug 9, 2024
Merged

Conversation

TrongNguyenR
Copy link
Contributor

Add CC-RH port for Renesas F1Kx devices

Description

Add port files (port.c, portasm.s, portmacro.h) for CC-RH core for Renesas F1Kx devices.

Test Steps

The test project can be found here (RH850_F1Kx_CCRH). This project contains example tasks and configurations to help you get started with FreeRTOS on the RH850/F1K and F1Kx.

Checklist:

  • I have tested my changes. No regression in existing tests.
  • I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

None

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@TrongNguyenR TrongNguyenR requested a review from a team as a code owner July 7, 2024 09:22
archigup
archigup previously approved these changes Jul 9, 2024
@chinglee-iot
Copy link
Member

@TrongNguyenR
Thank you for creating this PR. We are still reviewing this PR and trying to run the demo with this port if the hardware is available to us. We thank you for your patience.

@TrongNguyenR
Copy link
Contributor Author

Hello @chinglee-iot,

@TrongNguyenR Thank you for creating this PR. We are still reviewing this PR and trying to run the demo with this port if the hardware is available to us. We thank you for your patience.

Thanks for your notice. We are working on preparing device for your testing, please wait.
Meanwhile, I understand that source code review is on-going, is it correct?
Thanks for your support.

@chinglee-iot
Copy link
Member

@TrongNguyenR
Yes, the review is still ongoing. We would like to run the demo with this port.
If the hardware can't be available in a short time, we will discuss to merge this PR first, then verify the demo later.

@TrongNguyenR
Copy link
Contributor Author

Hello @chinglee-iot,
We are preparing to send board from Renesas India by Jul 31. Please take it for your information.

@TrongNguyenR
Copy link
Contributor Author

Hello @chinglee-iot -san,
I got information that the board has already arrived at your side (Receiver: Mr. Nikhil Kamath).
Please let me know if you got any issue.

@aggarg
Copy link
Member

aggarg commented Jul 31, 2024

Thank you @TrongNguyenR! We received the board. We will get back to you.

@aggarg
Copy link
Member

aggarg commented Aug 8, 2024

@TrongNguyenR Would you please rebase your branch so that it is up-to-date with main?

Copy link

sonarqubecloud bot commented Aug 9, 2024

@TrongNguyenR
Copy link
Contributor Author

@aggarg, we have synced our branch, it's up to date now.

Copy link

codecov bot commented Aug 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.31%. Comparing base (17dfd0f) to head (8689b37).
Report is 10 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1100   +/-   ##
=======================================
  Coverage   92.31%   92.31%           
=======================================
  Files           6        6           
  Lines        3226     3228    +2     
  Branches      885      885           
=======================================
+ Hits         2978     2980    +2     
  Misses        132      132           
  Partials      116      116           
Flag Coverage Δ
unittests 92.31% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aggarg aggarg merged commit da3c35a into FreeRTOS:main Aug 9, 2024
18 checks passed
@aggarg
Copy link
Member

aggarg commented Aug 9, 2024

Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants