-
Notifications
You must be signed in to change notification settings - Fork 7.3k
Add minimal support for Renesas RZ/A2M EVK #88888
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
Add minimal support for Renesas RZ/A2M EVK #88888
Conversation
64f10b7
to
5e726e8
Compare
config SYS_CLOCK_HW_CYCLES_PER_SEC | ||
default 66000000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs to come from dts property
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nordicjm Thanks for reviewing.
Updated to get clock freq from dts.
dts/arm/renesas/rz/rza/r7s9210.dtsi
Outdated
/* Value must be set by board */ | ||
clock-frequency = <0>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so why is this here? If it needs to be set, omit it so it throws an error if a board does not have it set
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed redundant clock freq.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still here, remove from original commit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nordicjm
Thanks, I removed it from original commit.
supported: | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
??
supported: | ||
|
||
- uart |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so omit this empty part in the original commit and add the whole part in the commit adding UART
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved this change to board's commit.
5e726e8
to
723d0da
Compare
Add support for Renesas RZ/A2M Signed-off-by: Hoang Nguyen <[email protected]> Signed-off-by: Binh Nguyen <[email protected]>
Add minimal support for Renesas RZ/A2M EVK Signed-off-by: Hoang Nguyen <[email protected]> Signed-off-by: Binh Nguyen <[email protected]>
Add clock control support for RZ/A2M Signed-off-by: Hoang Nguyen <[email protected]> Signed-off-by: Binh Nguyen <[email protected]>
Add timer support for RZ/A2M Signed-off-by: Hoang Nguyen <[email protected]> Signed-off-by: Binh Nguyen <[email protected]>
Add pinctrl support for RZ/A2M Signed-off-by: Hieu Nguyen <[email protected]> Signed-off-by: Binh Nguyen <[email protected]> Signed-off-by: Hoang Nguyen <[email protected]>
Add serial support for RZ/A2M Signed-off-by: Hoang Nguyen <[email protected]> Signed-off-by: Binh Nguyen <[email protected]>
Add gpio support for RZ/A2M Signed-off-by: Hoang Nguyen <[email protected]> Signed-off-by: Binh Nguyen <[email protected]>
RZ/A2M requires MMU enabled. This sample disables MMU, so exclude the platform. Signed-off-by: Hoang Nguyen <[email protected]> Signed-off-by: Binh Nguyen <[email protected]>
723d0da
to
ddbebc3
Compare
Add minimal support for RZ/A2M EVK as following:
Document: https://builds.zephyrproject.io/zephyr/pr/88888/docs/boards/renesas/rza2m_evk/doc/index.html
This SoC does not utilize the FSP driver in hal_renesas because it is not supported; instead, the driver is developed directly within the Zephyr tree.