camkes-vm-apps: use empty list #36
Conversation
| @@ -87,7 +87,7 @@ assembly { | |||
| "0x48000000:27" //RAM | |||
| ]; | |||
| vm1.irqs = []; | |||
| vm1.dtb = dtb([{}]); | |||
There was a problem hiding this comment.
Could this instead be addressed by allowing the template that uses this config option to accept an unset .dtb config setting?
There was a problem hiding this comment.
Could this instead be addressed by allowing the template that uses this config option to accept an unset .dtb config setting?
I'd make this a separate PR then. Maybe we could just set dtb([]) in the defaults, so this does not have to be set explicitly.
For now, this PR here is just for some more test coverage for seL4/camkes-tool#129, it does not have to get merged.
I'd really like to see the empty list support from seL4/camkes-tool#129 getting merged. It is quite convenient for development, when adding an removing entries there. You don't run into an error, if the list turns out to be empty because of commented out elements.
axel-h
force-pushed
the
patch-axel-1
branch
8 times, most recently
from
d36dcf8 to
d617b48
Compare
axel-h
force-pushed
the
patch-axel-1
branch
2 times, most recently
from
61f46d8 to
507fd1d
Compare
Signed-off-by: Axel Heider <[email protected]>
Test with: seL4/camkes-tool#129