(mynewt-core) branch master updated: mcu/pic32: Fix memset_func placement

[je...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jerzy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-core.git


The following commit(s) were added to refs/heads/master by this push:
     new 7a9dd95b9 mcu/pic32: Fix memset_func placement
7a9dd95b9 is described below

commit 7a9dd95b9e421fffbcd8c1dc9f6eaad556f9dbf0
Author: Jerzy Kasenberg <je...@codecoup.pl>
AuthorDate: Tue Mar 12 16:19:21 2024 +0100

    mcu/pic32: Fix memset_func placement
    
    By mistake sdata2 section was put in kseg0_program_mem.
    It should be left alone with no redirection to region
    xc32 linker will put it in correct place.
    
    memset_func put in this place resulted in crash during
    boot since mbedtls was was updated.
---
 hw/mcu/microchip/pic32mz/p32mz_boot.ld | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/hw/mcu/microchip/pic32mz/p32mz_boot.ld b/hw/mcu/microchip/pic32mz/p32mz_boot.ld
index b1f86553c..f50bb438a 100644
--- a/hw/mcu/microchip/pic32mz/p32mz_boot.ld
+++ b/hw/mcu/microchip/pic32mz/p32mz_boot.ld
@@ -1613,9 +1613,8 @@ SECTIONS
   .sdata2 ALIGN(4) :
   {
     *(.sdata2 .sdata2.* .gnu.linkonce.s2.*)
-    *(.data.memset_func.memset_func)
     . = ALIGN(4) ;
-  } >kseg0_program_mem
+  }
   /*
    * Uninitialized constant global and static data (i.e., variables which will
    * always be zero).  Again, this is different from .sbss, which contains