summaryrefslogtreecommitdiff
path: root/common
diff options
context:
space:
mode:
authorMats Kärrman <mats.karrman@tritech.se>2013-04-09 17:10:59 +0200
committerTom Rini <trini@ti.com>2013-04-10 10:31:02 -0400
commitfac150e83f403933e9ffc9d01f858c4a6313874e (patch)
treebb4ba2875fd12abdc370040eb79af5f1125407be /common
parent645b271a6039e79b368f027a5624dc0820441733 (diff)
downloadu-boot-imx-fac150e83f403933e9ffc9d01f858c4a6313874e.zip
u-boot-imx-fac150e83f403933e9ffc9d01f858c4a6313874e.tar.gz
u-boot-imx-fac150e83f403933e9ffc9d01f858c4a6313874e.tar.bz2
powerpc/lib: fix unsafe register handling in wait_ticks
If watchdog is enabled, the arch/powerpc/lib/ticks.S::wait_ticks() function calls the function specified by the WATCHDOG_RESET macro. The wait_ticks function depends on the registers r0, r6 and r7 being preserved however that is not guaranteed, e.g. if the reset function is a C function this will probably overwrite r0 and cause an endless loop. The following patch changes to using r14+r15 instead of r6+r7 (to resemble what would have been generated by a C compiler) and saves all necessary registers on the stack. The patch has been tested on a custom MPC5125 based machine using the 512x powerpc architecture. Signed-off-by: Mats Karrman <mats.karrman@tritech.se> Cc: Wolfgang Denk <wd@denx.de> Acked-by: Joakim Tjernlund <joakim.tjernlund@transmode.se> Tested-by: Stefan Roese <sr@denx.de>
Diffstat (limited to 'common')
0 files changed, 0 insertions, 0 deletions