[coreboot-gerrit] Patch set updated for coreboot: bd56e35 vendorcode/amd/agesa/f16kb/*/PcieComplexDataKB.c: Implicit truncation

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Tue Dec 9 11:12:31 CET 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7719

-gerrit

commit bd56e3551c24cd081b0976e61bedf1f7adf893e5
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Tue Dec 9 03:39:13 2014 +1100

    vendorcode/amd/agesa/f16kb/*/PcieComplexDataKB.c: Implicit truncation
    
    Clang complains:
     "implicit truncation from 'int' to bitfield changes value from -1 to 15"
    
    This is "0xFF" on Mullins, so use a fictitious type-cast to get the intended
    behaviour without warnings.
    
    Change-Id: Ib7476d453ffd932bb911e638117cf9f56f71f269
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 .../amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/PcieComplexDataKB.c      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/PcieComplexDataKB.c b/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/PcieComplexDataKB.c
index 5fd3cca..b1bc3bf 100644
--- a/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/PcieComplexDataKB.c
+++ b/src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbInitKB/PcieComplexDataKB.c
@@ -145,7 +145,7 @@ KB_COMPLEX_CONFIG ComplexDataKB = {
     0,
     DDI_START_PHY_LANE,
     DDI_END_PHY_LANE,
-    -1,
+    (UINT8)(-1u),
     0,
     0,
     {



More information about the coreboot-gerrit mailing list