-
Notifications
You must be signed in to change notification settings - Fork 8k
samples: boards: nordic: spis_wakeup: Run sample on APP+PPR #97113
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?
samples: boards: nordic: spis_wakeup: Run sample on APP+PPR #97113
Conversation
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.
Now, there are two sample configurations: APP+Radio and APP+PPR.
I moved common DTS part on APP to a separate file.
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.
Include common and delete references to led1
. Otherwise Radio core will not be able to take ownership of the led1
.
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.
PPR inherits permissions from its owner (APP). Therefore, APP must take ownership of led1
(so PPR could use it). Moreover, APP must claim ownership of other resources that PPR will use (spi131
, uart135
, route interrupts to PPR interrupt handler).
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.
This is only to limit code size. Otherwise, sample doesn't fit in available ROM.
Extend sample with possibility to use PPR core instead of Radio core. Signed-off-by: Sebastian Głąb <[email protected]>
7d7c3e4
to
74a8bde
Compare
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.
Move common part for "remote" core to a separate file.
|
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 from remote core prj.conf because PPR doesn't support CONFIG_PM and binary file is too big when ASSERTS are enabled.
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.
Common file plus set memory-regions property in spi131.
Works in internal CI:
|
Extend sample with possibility to use PPR core instead of Radio core.