the following patch was just integrated into master: commit 0deb355d0e7f042cbcf6d74937b39729c7eaac0f Author: Edward O'Callaghan eocallaghan@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@alterapraxis.com Reviewed-on: http://review.coreboot.org/5757 Tested-by: build bot (Jenkins) Reviewed-by: Marc Jones marc.jones@se-eng.com
See http://review.coreboot.org/5757 for details.
-gerrit