diff options
| author | Andrew Ruder <andrew.ruder@elecsyscorp.com> | 2015-10-05 13:51:10 -0500 |
|---|---|---|
| committer | Freddie Chopin <freddie.chopin@gmail.com> | 2015-10-30 23:41:44 +0000 |
| commit | 9413a7a814755f3fab92f460bfbdd518f24ffaf5 (patch) | |
| tree | 778b48c83cfa0a273898810eb93dbf3bc2ffc3ed /src/rtos/ThreadX.c | |
| parent | ddc3317c54c3f2327cdd07d87f997197200a385e (diff) | |
| download | riscv-openocd-9413a7a814755f3fab92f460bfbdd518f24ffaf5.zip riscv-openocd-9413a7a814755f3fab92f460bfbdd518f24ffaf5.tar.gz riscv-openocd-9413a7a814755f3fab92f460bfbdd518f24ffaf5.tar.bz2 | |
rtos: turn stack alignment into a function pointer
Some targets (Cortex M) require more complicated calculations for
turning the stored stack pointer back into a process stack pointer.
For example, the Cortex M stores a bit in the auto-stacked xPSR
indicating that alignment had to be performed and an additional 4
byte padding is present before the exception stacking. This change
only sets up the framework for Cortex-M unstacking and does not
add Cortex-M support.
Note: this also fixes the alignment calculation nearly addressed by
change #2301 entitled rtos/rtos.c: fix stack alignment calculation.
Updated calculation is in rtos_generic_stack_align.
Change-Id: I0f662cad0df81cbe5866219ad0fef980dcb3e44f
Signed-off-by: Andrew Ruder <andrew.ruder@elecsyscorp.com>
Cc: Paul Fertser <fercerpav@gmail.com>
Cc: Andreas Fritiofson <andreas.fritiofson@gmail.com>
Cc: Evan Hunter <evanhunter920@gmail.com>
Cc: Jon Burgess <jburgess777@gmail.com>
Reviewed-on: http://openocd.zylin.com/3002
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
Tested-by: jenkins
Diffstat (limited to 'src/rtos/ThreadX.c')
| -rw-r--r-- | src/rtos/ThreadX.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/rtos/ThreadX.c b/src/rtos/ThreadX.c index c033125..16df7a4 100644 --- a/src/rtos/ThreadX.c +++ b/src/rtos/ThreadX.c @@ -117,14 +117,14 @@ const struct rtos_register_stacking rtos_threadx_arm926ejs_stacking[] = { ARM926EJS_REGISTERS_SIZE_SOLICITED, /* stack_registers_size */ -1, /* stack_growth_direction */ 17, /* num_output_registers */ - 0, /* stack_alignment */ + NULL, /* stack_alignment */ rtos_threadx_arm926ejs_stack_offsets_solicited /* register_offsets */ }, { ARM926EJS_REGISTERS_SIZE_INTERRUPT, /* stack_registers_size */ -1, /* stack_growth_direction */ 17, /* num_output_registers */ - 0, /* stack_alignment */ + NULL, /* stack_alignment */ rtos_threadx_arm926ejs_stack_offsets_interrupt /* register_offsets */ }, }; |