Skip to content

boards: ruiside: RA8D1 Vision Board: add support #90283

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bricle
Copy link
Contributor

@bricle bricle commented May 21, 2025

This is another board from Shanghai Ruiside Electronic Technology Co., Ltd..
This board is based on the Renesas Cortex-M85 architecture RA8D1 chip.

This board is available from mouser and aliexpress

currently tested uart, button, led, sdram, sd card

sdram test log

with west twister --device-testing --device-serial=/dev/ttyACM0 -v --device-serial-baud 115200 -p ra8d1_vision_board/r7fa8d1bhecbd -s tests/drivers/memc/ram/drivers.memc.renesas_ra_sdram --west-flash --west-runner=pyocd

INFO    - JOBS: 12
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - 1/1 ra8d1_vision_board/r7fa8d1bhecbd tests/drivers/memc/ram/drivers.memc.renesas_ra_sdram PASSED (device 7.994s <zephyr>)

INFO    - 1 test scenarios (1 configurations) selected, 0 configurations filtered (0 by static filter, 0 at runtime).
INFO    - 1 of 1 executed test configurations passed (100.00%), 0 built (not run), 0 failed, 0 errored, with no warnings in 31.15 seconds.
INFO    - 1 of 1 executed test cases passed (100.00%) on 1 out of total 1053 platforms (0.09%).
INFO    - 5 selected test cases not executed: 5 skipped.
INFO    - 1 test configurations executed on platforms, 0 test configurations were only built.

Hardware distribution summary:

| Board                            | ID   |   Counter |   Failures |
|----------------------------------|------|-----------|------------|
| ra8d1_vision_board/r7fa8d1bhecbd |      |         1 |          0 |

sd card test result

with west build tests/drivers/disk/disk_access -b ra8d1_vision_board/r7fa8d1bhecbd -d build -p always && west flash

*** Booting Zephyr OS build 60d873418870 ***
Running TESTSUITE disk_driver
===================================================================
Disk reports 57626624 sectors
Disk reports sector size 512
START - test_read
Testing reads of 8 sectors
Testing reads of 1 sectors
Testing reads of 29 sectors
Testing reads of 31 sectors
 PASS - test_read in 0.082 seconds
===================================================================
START - test_write
Testing writes of 8 sectors
Testing writes of 1 sectors
Testing writes of 29 sectors
Testing writes of 31 sectors
 PASS - test_write in 0.168 seconds
===================================================================
TESTSUITE disk_driver succeeded

------ TESTSUITE SUMMARY START ------                                                                       
                                                                                                            
SUITE PASS - 100.00% [disk_driver]: pass = 2, fail = 0, skip = 0, total = 2 duration = 0.250 seconds        
 - PASS - [disk_driver.test_read] duration = 0.082 seconds                                                  
 - PASS - [disk_driver.test_write] duration = 0.168 seconds                                                 
                                                                                                            
------ TESTSUITE SUMMARY END ------                                                                         
                                                                                                            
===================================================================                                         
PROJECT EXECUTION SUCCESSFUL                                                                                

@bricle bricle force-pushed the rtt_vision branch 2 times, most recently from 60d8734 to c83329e Compare May 21, 2025 14:18

.. zephyr:board-supported-runners::

Applications for the ``ra8d1_vision_board`` board configuration can be
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Applications for the ``ra8d1_vision_board`` board configuration can be
Applications for the ``ra8d1_vision_board`` board can be

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, thanks

Copy link
Collaborator

Choose a reason for hiding this comment

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

why is this needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed, thanks

Copy link
Collaborator

Choose a reason for hiding this comment

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

remove empty space around image, then reduce resolution since this is larger than my whole screen, then put through https://tinypng.com/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, thanks

psels = <RA_PSEL(RA_PSEL_SCI_9, 2, 9)>;
drive-strength = "medium";
};
group2 {
Copy link
Collaborator

Choose a reason for hiding this comment

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

newline gap between nodes

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

bus-width = "16-bit";

bank@0 {
reg = <0>;
Copy link
Collaborator

Choose a reason for hiding this comment

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

double indent?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, thanks

<RA_PSEL(RA_PSEL_SDHI, 5, 2)>; /* SDDATA3 */
drive-strength = "high";
};
group2 {
Copy link
Collaborator

Choose a reason for hiding this comment

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

and here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, thaks

@bricle bricle marked this pull request as ready for review May 21, 2025 15:04
Copy link
Contributor

@khoa-nguyen-18 khoa-nguyen-18 left a comment

Choose a reason for hiding this comment

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

Hi, I have some commends in the board dts

#include <dt-bindings/input/input-event-codes.h>
#include <zephyr/dt-bindings/memory-attr/memory-attr-arm.h>
#include <zephyr/dt-bindings/memory-controller/renesas,ra-sdram.h>
#include <zephyr/dt-bindings/adc/adc.h>
Copy link
Contributor

Choose a reason for hiding this comment

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

It seem you not using adc on the board dts. Should we remove this adc #include?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not used, removed, thanks

zephyr,console = &uart9;
zephyr,shell-uart = &uart9;
zephyr,entropy = &trng;
zephyr,flash-controller = &flash1;
Copy link
Contributor

Choose a reason for hiding this comment

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

It should be the label of flash-controller

Suggested change
zephyr,flash-controller = &flash1;
zephyr,flash-controller = &flash;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, thanks

Comment on lines 173 to 181
&port_irq12 {
interrupts = <88 12>;
status = "okay";
};

&port_irq13 {
interrupts = <89 12>;
status = "okay";
};
Copy link
Contributor

Choose a reason for hiding this comment

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

Can I ask the purpose of the external interrupt you enabled? Currently, we only enable the external interrupt corresponding to the on-board button.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not used, removed, thanks

building and running.

**Note:** Only support from SDK v0.16.6 in which GCC for Cortex Arm-M85 was available.
To build for EK-RA8M1 user need to get and install GNU Arm Embedded toolchain from https://github.com/zephyrproject-rtos/sdk-ng/releases/tag/v0.16.6
Copy link
Member

Choose a reason for hiding this comment

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

Should be changed to RA8D1 Vision Board instead of EK-RA8M1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, thanks

Comment on lines 13 to 15
- 480MHz frequency,on-chip 2Mb Flash,1Mb SRAM
- 32Mb-SDRAM;8Mb-QSPI Flash
- MIPI-DSI;RGB666;8bit Camera
Copy link
Member

Choose a reason for hiding this comment

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

Small nit, adding space after "," and ";" for better documentation

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, thanks

this is another board from ruiside, tested uart,
button, led, sdram, sd card

Signed-off-by: Shan Pen <bricle031@gmail.com>
Copy link

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

Successfully merging this pull request may close these issues.

5 participants