[coreboot-gerrit] Patch merged into coreboot/master: 0deb355 vendorcode/amd/agesa: unsigned enum is strictly positive

gerrit at coreboot.org gerrit at coreboot.org
Wed May 21 02:17:24 CEST 2014


the following patch was just integrated into master:
commit 0deb355d0e7f042cbcf6d74937b39729c7eaac0f
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Thu May 15 21:07:16 2014 +1000

    vendorcode/amd/agesa: unsigned enum is strictly positive
    
    The typedef'ed BIT_FIELD_NAME enum is type unsigned. The parameter
    'FieldName' is decleared with type BIT_FIELD_NAME and thus the redudant
    comparison of unsigned enum expression >= 0 is always true.
    
    BIT_FIELD_NAME is declared in vendorcode/amd/agesa/f14/Proc/Mem/mm.h
    
    Change-Id: Id2f03596c44b68e861e939f3528256d4b08c45ce
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
    Reviewed-on: http://review.coreboot.org/5757
    Tested-by: build bot (Jenkins)
    Reviewed-by: Marc Jones <marc.jones at se-eng.com>


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

-gerrit



More information about the coreboot-gerrit mailing list