[riot-notifications] [RIOT-OS/RIOT] dist/tools/renode: improve integration + additional boards (#15459)

Alexandre Abadie notifications at github.com
Sat Nov 21 12:45:20 CET 2020


@aabadie commented on this pull request.



> @@ -0,0 +1,25 @@
+:name: STK3700
+:description: Silicon Labs STK3700 starter kit
+
+$name?="STK3700"
+
+using sysbus
+mach create $name
+
+machine LoadPlatformDescription $RIOTCPU/efm32/families/efm32gg/dist/renode/efm32gg990f1024.repl
+machine LoadPlatformDescription $RIOTBOARD/stk3700/dist/renode/stk3700.repl

I was not talking about the .resc files as they depends on RIOT specific variables (the elffile at least). I agree that eventually the repl could be different because RIOT doesn't use the same default configuration. But for the CPU repl, let me share some doubts because the CPU definitions are not supposed to change (the register addresses, etc).
The problem with relative paths seems to be an issue on their side. TBH it's super simple to install Renode using the package manager of your system (the latest release provides OSX packages, .deb and .rpm). If you decide to build it locally, I guess setting the PATH variable accordingly shouldn't be an issue.

You are also mentioning the versions. I looked in detail at how they are handled by this PR and I don't like it: with this PR you would have to use 1.12 which is not release yet and although 1.11 still works with cc2538dk. I think this should be handed differently, with the boards having renode configuration defining a variable with the minimum version required.

I've also been working on improving the renode integration and could came up with something only using Makefiles (no need for a temporary resc file in the application directory) and I'm also able to use the debugger (but start should be called from the board.resc).
If you are interested, I can PR this work.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15459#discussion_r528186618
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201121/9eeeff5b/attachment.htm>


More information about the notifications mailing list