RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

bsp/tms570: ensure that linker symbol comparison to NULL is not optimized out.

Browse Source 
Signed-off-by: Pavel Pisa <pisa@cmp.felk.cvut.cz>
This commit is contained in:
Pavel Pisa
2015-11-28 12:01:36 +01:00
committed by Gedare Bloom
parent 4b8184f0ed
commit 4a02a74154
1 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
c/src/lib/libbsp/arm/tms570/startup/bspstart.c
View File

@@ -32,6 +32,9 @@
void bsp_start( void ) void bsp_start( void )
{ {
void *need_remap_ptr;
unsigned int need_remap_int;
#if BYTE_ORDER == BIG_ENDIAN #if BYTE_ORDER == BIG_ENDIAN
/* /*
* If CPU is big endian (TMS570 family variant) * If CPU is big endian (TMS570 family variant)
@@ -65,9 +68,16 @@ void bsp_start( void )
* SRAM then it leads to CPU error halt. * SRAM then it leads to CPU error halt.
* *
* So use of POM to replace jumps to vectors target * So use of POM to replace jumps to vectors target
* addresses seems to be the best option. * addresses seems to be the best option for now.
*
* The passing of linker symbol (represented as start address
* of global array) through dummy asm block ensures that C compiler
* cannot optimize comparison out on premise that reference cannot
* evaluate to NULL definition in standard.
*/ */
if ( (uintptr_t)bsp_start_vector_table_begin != 0 ) { need_remap_ptr = bsp_start_vector_table_begin;
asm volatile ("\n": "=r" (need_remap_int): "0" (need_remap_ptr));
if ( need_remap_int != 0 ) {
tms570_pom_remap(); tms570_pom_remap();
} }