[coreboot] Patch merged into coreboot/master: 7675d8a Supermicro H8SCM & H8QGI: Fix printk warnings

gerrit at coreboot.org gerrit at coreboot.org
Mon Feb 25 19:02:21 CET 2013


the following patch was just integrated into master:
commit 7675d8a481c6cbeba418f00f2eb733d904171a41
Author: Martin Roth <martin.roth at se-eng.com>
Date:   Sun Feb 24 15:09:11 2013 -0700

    Supermicro H8SCM & H8QGI: Fix printk warnings
    
    Changes:
     - Fix printk warnings for these two platforms by getting rid of the
       l length specifier and casting to unsigned int.
       This gets rid of a bunch of warnings like this one:
         agesawrapper.c:279, GNU Compiler 4 (gcc), Priority: Normal
         format '%lu' expects argument of type 'long unsigned int',
           but argument 3 has type 'UINT32' [-Wformat]
    
    Notes:
     - This is the same change that was done for Tyan s8226 in change:
       ddff32eb - http://review.coreboot.org/#/c/2451/
       Tyan S8226: Fix printk warnings
    
     - I have not tested this change on either of these platforms, I have
       just compiled it.
    
    Change-Id: I46b4c13fde7473cd2a084c7c7cb5c893f1731b02
    Signed-off-by: Martin Roth <martin.roth at se-eng.com>
    Reviewed-on: http://review.coreboot.org/2502
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>

Build-Tested: build bot (Jenkins) at Sun Feb 24 23:25:56 2013, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer at coreboot.org> at Mon Feb 25 19:02:17 2013, giving +2
See http://review.coreboot.org/2502 for details.

-gerrit



More information about the coreboot mailing list