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

Added initial Renesas Cortex-R52 support. #148

Conversation

RICCIARDI-Adrien
Copy link
Contributor

No description provided.

fsylvestre and others added 30 commits November 8, 2023 10:36
* Remove unused rpi dependent header from common cortex-a file
* Cleanup inclusion of 'CallTerminateISR2' and 'CallTerminateTask' depending of their existence
Not all cortex-a platforms support secure mode, so the vector table is platform
dependent: then move its inclusion to board directory.
Basic support for Spider platform: without interrupt support
[Adrien Ricciardi: Added .gitignore]
New data is taken from Zephyr's "rcar_gen4_cr52.dtsi" file.
A correct placement of RAM data is very important otherwise external
perihperals (such as Rswitch) will not be able to access data structures
placed in RAM.

Signed-off-by: Valerio Setti <[email protected]>
…esas.

Only the addresses that have been used and translated to be used from the
Cortex-R52 Application Domain are currently uncommented.

Signed-off-by: Adrien Ricciardi <[email protected]>
It is currently able to receive and transmit CAN 2.0 frames at a bus speed
of 125 Kbit/s.

Signed-off-by: Adrien Ricciardi <[email protected]>
…, increments the ID and the payload bytes, then send it back.

Signed-off-by: Adrien Ricciardi <[email protected]>
…ated to CAN-FD introduction.

Signed-off-by: Adrien Ricciardi <[email protected]>
…ed to CAN-FD introduction.

Signed-off-by: Adrien Ricciardi <[email protected]>
… the CAN driver.

The increased frequency allows for more accurate baud rates and also to reach bus speeds up to 8Mbit/s.
The current Tseg and SJW values are aligned on the test setup Kvaser probe ones.

Signed-off-by: Adrien Ricciardi <[email protected]>
…gered by a 7-byte payload frame.

Receiving a CAN 2.0 frame with a 7-byte payload triggered a data abort exception.
This exception was due to the usage of memcpy() to copy the data from and to the
CAN controller registers.

Maybe the memcpy() has some internal optimization that copies 4 bytes at a time,
then single bytes, and these non uniform accesses were messing up the hardware.

Signed-off-by: Adrien Ricciardi <[email protected]>
…bility.

The function now waits for the frame being fully transmitted and checks
a status bit that seems to be more accurate.

Signed-off-by: Adrien Ricciardi <[email protected]>
Added support for CAN-FD without the Bit Rate Switch feature.

Signed-off-by: Adrien Ricciardi <[email protected]>
…fer in the CAN driver to improve performances.

This also allows to receive another CAN frame while the current received one
is being read without altering the current frame buffer content.

Signed-off-by: Adrien Ricciardi <[email protected]>
… the CAN driver.

This saves some waiting time for a transmission to finish when a program wants
to send several frames in a row.

Signed-off-by: Adrien Ricciardi <[email protected]>
Copy link
Member

@jlbirccyn jlbirccyn left a comment

Choose a reason for hiding this comment

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

There is no file listed in the object PLATFORM_FILES of templates/config/cortex-a-r/armv8/config.oil. Is it normal or is it a mistake ?

@RICCIARDI-Adrien
Copy link
Contributor Author

Yes, this is normal. ARMV8-generic files will be added here when a new ARMV8 machine will be supported.

@jlbirccyn jlbirccyn self-requested a review November 9, 2023 14:18
@jlbirccyn
Copy link
Member

Maybe add the new target in the general README

@RICCIARDI-Adrien
Copy link
Contributor Author

Sure, I add it right now.

@jlbirccyn
Copy link
Member

Ok to merge 👍

@RICCIARDI-Adrien RICCIARDI-Adrien merged commit 9c67435 into TrampolineRTOS:master Nov 9, 2023
39 checks passed
@RICCIARDI-Adrien RICCIARDI-Adrien deleted the initial_renesas_cr52_support branch November 9, 2023 15:18
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.

4 participants