[coreboot-gerrit] Patch merged into coreboot/master: 69813fe sb/amd/amd8111/acpi.c: Remove set but unused variable `dword`

gerrit at coreboot.org gerrit at coreboot.org
Wed Apr 9 13:31:38 CEST 2014


the following patch was just integrated into master:
commit 69813febbc91571f9126f980b97fd7b37a45f125
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Sat Mar 1 12:49:57 2014 +0100

    sb/amd/amd8111/acpi.c: Remove set but unused variable `dword`
    
    Removing `-Wno-unused-but-set-variable` from `CFLAGS` results in the error
    below, when building for example the HP DL145 GL1.
    
    	    CC         southbridge/amd/amd8111/acpi.ramstage.o
    	src/southbridge/amd/amd8111/acpi.c: In function 'acpi_init':
    	src/southbridge/amd/amd8111/acpi.c:100:11: error: variable 'dword' set but not used [-Werror=unused-but-set-variable]
    
    Removing the variable `dword` fixes this error.
    
    The read is left in the code, as I do not know if it has an effect or
    not.
    
    Change-Id: I9957cef3a996c5974c275423c9de63ccf230974e
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-on: http://review.coreboot.org/5315
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <patrick at georgi-clan.de>


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

-gerrit



More information about the coreboot-gerrit mailing list