[coreboot-gerrit] Patch merged into coreboot/master: northbridge/amd/{lx, gx2}: remove immediate accesses of 0

gerrit at coreboot.org gerrit at coreboot.org
Sat Apr 16 01:51:05 CEST 2016


the following patch was just integrated into master:
commit 59493717ad7aee5e4d6179b00c66b21af79e0376
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Wed Apr 13 21:00:12 2016 +0200

    northbridge/amd/{lx,gx2}: remove immediate accesses of 0
    
    gcc doesn't like these because they're undefined behavior, so use
    zeroptr instead. For the loop that just does a number of writes (0..4),
    use zeroptr + i.
    
    Checked the disassembly (AMD_RUMBA and PCENGINES_ALIX2D) to not contain
    ud2 anymore and to look reasonable where zeroptr was used.
    
    Change-Id: I4a58220ec9a10c465909ca4ecbe5366d0a8cc0df
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Reviewed-on: https://review.coreboot.org/14345
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
    Tested-by: build bot (Jenkins)


See https://review.coreboot.org/14345 for details.

-gerrit



More information about the coreboot-gerrit mailing list