drivers: misc: ethos_u: support nuvoton numaker m55m1x #88941
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
platform: Nuvoton NPCX
github-actions
bot
requested review from
alvsun,
ChiHuaL,
fabiobaltieri,
ithinuel,
jackrosenthal,
kartben,
keith-zephyr,
kristofer-jonsson-arm,
laurenmurphyx64,
MulinChao,
nashif,
sjg20,
TomChang19,
wearyzen and
XenuIsWatching
Comment on lines
+303
to
+321
| /* Address undefined _fini for the exit call | ||
| * | ||
| * With gcc command-line options -nostartfiles or -nostdlib, the gcc | ||
| * built-in object files crti.o/crtn.o which implement _init/_fini | ||
| * won't get linked in automatically and will cause undefined reference | ||
| * to _fini error when exit is invoked. | ||
| * | ||
| * This is fixed by providing one dummey _fini to let linker pass. | ||
| */ | ||
| __weak void _fini(void) | ||
| { | ||
| } |
|
Hi @ccli8, could you share your output log when running the |
With gcc command-line options -nostartfiles or -nostdlib, the gcc built-in object files crti.o/crtn.o which implement _init/_fini won't get linked in automatically and will cause undefined reference to _fini error when exit is invoked. This is fixed by providing one dummey _fini to let linker pass. Signed-off-by: Chun-Chieh Li <ccli8@nuvoton.com>
Add header file stdint.h so that fixed width integer type e.g. uint8_t can use without compile error. Signed-off-by: Chun-Chieh Li <ccli8@nuvoton.com>
It is unsafe to pass raw buffer as thread stack. Instead, use canonical K_THREAD_STACK_DEFINE or friends to define thread stack. Signed-off-by: Chun-Chieh Li <ccli8@nuvoton.com>
This addes frontend of arm ethos-u core driver for nuvoton numaker m55m1x. Special notes include: 1. Leaving application overriding dcache flush/invalidate weak functions for cacheable NPU buffer 2. Configuring macs_per_cc to 256 in arm ethos-u core driver to match m55m1x ethos-u RTL config Signed-off-by: Chun-Chieh Li <ccli8@nuvoton.com>
This supports nuvoton numaker_m55m1 board, including: 1. Choosing NPU buffer as cacheable or non-cacheable, dependent on CONFIG_NOCACHE_MEMORY. When cachealbe is chosen, data cache flush/ invalidate weak functions (ethos-u core driver) are also overridden. 2. Add Vela-compiled model which matches this targets's NPU, Ethos-U55 with 256 macs_per_cc Besides, re-organize for new board support, including: 1. Refactor Vela-compiled models which must match target Ethos-U RTL config 2. Add non-Vela-compiled model for evaluation of no-Ethos-U platform Signed-off-by: Chun-Chieh Li <ccli8@nuvoton.com>
|
|
Log of |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This supports Nuvoton's M55M1 Ethos-U NPU. Besides, the
tflm_ethosusample is refined for this target's verification, including:_finisymbol on invokingexit()for newlib libcstdint.hfor e.g.uint8_ttypeK_THREAD_STACK_DEFINEor friends instead of raw bufferNOTE: This PR covers #88218.