You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by GitBox <gi...@apache.org> on 2022/06/27 08:34:06 UTC
[GitHub] [mynewt-core] kasjer opened a new pull request, #2855: mcu/nrf5340: Trust zone RAM clean update
kasjer opened a new pull request, #2855:
URL: https://github.com/apache/mynewt-core/pull/2855
RAM_SIZE was always taken from bps.h.
With trust zone in place it may be that the bootloader
had limited RAM assigned in linker scripts.
This may happen when MCU_BOOT_MEASURED_STARTUP is used.
This change clears only RAM that bootloader used.
If nothing special was done to linker scripts RAM size
will be the same as it used to be.
Signed-off-by: Jerzy Kasenberg <je...@codecoup.pl>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2855: mcu/nrf5340: Trust zone RAM clean update
Posted by GitBox <gi...@apache.org>.
apache-mynewt-bot commented on PR #2855:
URL: https://github.com/apache/mynewt-core/pull/2855#issuecomment-1167053315
<!-- style-bot -->
## Style check summary
### Our coding style is [here!](https://github.com/apache/mynewt-core/blob/master/CODING_STANDARDS.md)
#### hw/mcu/nordic/nrf5340/src/hal_system_start.c
<details>
```diff
@@ -144,13 +146,13 @@
* Normal loop here is inlined by GCC to call to memset hence asm version of
* memset that does not use stack (that just get erased).
*/
- asm volatile(" mov r0, #0 \n"
- "1: stmia %0!, {r0} \n"
- " cmp %0, %1 \n"
- " blt 1b \n"
- :
- : "r" (&_ram_start), "r" (&__StackTop)
- : "r0");
+ asm volatile (" mov r0, #0 \n"
+ "1: stmia %0!, {r0} \n"
+ " cmp %0, %1 \n"
+ " blt 1b \n"
+ :
+ : "r" (&_ram_start), "r" (&__StackTop)
+ : "r0");
/* Application startup code expects interrupts to be enabled */
__enable_irq();
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [mynewt-core] kasjer merged pull request #2855: mcu/nrf5340: Trust zone RAM clean update
Posted by GitBox <gi...@apache.org>.
kasjer merged PR #2855:
URL: https://github.com/apache/mynewt-core/pull/2855
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org