[coreboot-gerrit] Patch merged into coreboot/master: arm64: declare do_dcsw_op as function

gerrit at coreboot.org gerrit at coreboot.org
Fri Aug 28 08:43:57 CEST 2015


the following patch was just integrated into master:
commit 46502c9e374c30018773f09d0be78577cbcf3d0e
Author: Jimmy Huang <jimmy.huang at mediatek.com>
Date:   Thu Aug 13 10:48:49 2015 +0800

    arm64: declare do_dcsw_op as function
    
    do_dcsw_op is coded as a label, it's possible that linker will place
    do_dcsw_op on unaligned address. To avoid this situation, we declare
    do_dcsw_op as a function. Also explicitly set the 2nd argument of
    ENTRY_WITH_ALIGN(name, bits) to 2.
    
    do_dcsw_op:
    	cbz     x3, exit
       c103d:       b40003e3        cbz     x3, c10b9 <exit>
    	mov     x10, xzr
       c1041:       aa1f03ea        mov     x10, xzr
    	adr     x14, dcsw_loop_table    // compute inner loop address
    
    BRANCH=none
    BUG=none
    TEST=build and check do_dcsw_op in elf file
    
    Change-Id: Ieb5f4188d6126ac9f6ddb0bfcc67452f79de94ad
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
    Original-Commit-Id: 4ee26b76089fab82cf4fb9b21c9f15b29e57b453
    Original-Change-Id: Id331e8ecab7ea8782e97c10b13e8810955747a51
    Original-Signed-off-by: Jimmy Huang <jimmy.huang at mediatek.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/293660
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
    Original-Commit-Queue: Yidi Lin <yidi.lin at mediatek.com>
    Original-Tested-by: Yidi Lin <yidi.lin at mediatek.com>
    Reviewed-on: http://review.coreboot.org/11395
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


See http://review.coreboot.org/11395 for details.

-gerrit



More information about the coreboot-gerrit mailing list