-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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
base: main
Are you sure you want to change the base?
Conversation
60d8734
to
c83329e
Compare
|
||
.. zephyr:board-supported-runners:: | ||
|
||
Applications for the ``ra8d1_vision_board`` board configuration can be |
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.
Applications for the ``ra8d1_vision_board`` board configuration can be | |
Applications for the ``ra8d1_vision_board`` board can be |
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.
done, thanks
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.
why is this needed?
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, thanks
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.
remove empty space around image, then reduce resolution since this is larger than my whole screen, then put through https://tinypng.com/
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.
done, thanks
psels = <RA_PSEL(RA_PSEL_SCI_9, 2, 9)>; | ||
drive-strength = "medium"; | ||
}; | ||
group2 { |
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.
newline gap between nodes
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.
done
bus-width = "16-bit"; | ||
|
||
bank@0 { | ||
reg = <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.
double indent?
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.
done, thanks
<RA_PSEL(RA_PSEL_SDHI, 5, 2)>; /* SDDATA3 */ | ||
drive-strength = "high"; | ||
}; | ||
group2 { |
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.
and here
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.
done, thaks
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.
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> |
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.
It seem you not using adc on the board dts. Should we remove this adc #include
?
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.
not used, removed, thanks
zephyr,console = &uart9; | ||
zephyr,shell-uart = &uart9; | ||
zephyr,entropy = &trng; | ||
zephyr,flash-controller = &flash1; |
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.
It should be the label of flash-controller
zephyr,flash-controller = &flash1; | |
zephyr,flash-controller = &flash; |
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.
done, thanks
&port_irq12 { | ||
interrupts = <88 12>; | ||
status = "okay"; | ||
}; | ||
|
||
&port_irq13 { | ||
interrupts = <89 12>; | ||
status = "okay"; | ||
}; |
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.
Can I ask the purpose of the external interrupt you enabled? Currently, we only enable the external interrupt corresponding to the on-board button.
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.
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 |
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.
Should be changed to RA8D1 Vision Board instead of EK-RA8M1
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.
done, thanks
- 480MHz frequency,on-chip 2Mb Flash,1Mb SRAM | ||
- 32Mb-SDRAM;8Mb-QSPI Flash | ||
- MIPI-DSI;RGB666;8bit Camera |
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.
Small nit, adding space after "," and ";" for better documentation
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.
done, thanks
this is another board from ruiside, tested uart, button, led, sdram, sd card Signed-off-by: Shan Pen <bricle031@gmail.com>
|
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
sd card test result
with
west build tests/drivers/disk/disk_access -b ra8d1_vision_board/r7fa8d1bhecbd -d build -p always && west flash