[coreboot-gerrit] Patch merged into coreboot/master: 6758c68 ASRock E350M1: mptable.c: Remove unused variable `dev`

gerrit at coreboot.org gerrit at coreboot.org
Mon Apr 1 21:07:47 CEST 2013


the following patch was just integrated into master:
commit 6758c6887ec7daea9d14229ae674d899ba219166
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Fri Mar 29 20:37:42 2013 +0100

    ASRock E350M1: mptable.c: Remove unused variable `dev`
    
    When building the ASRock E350M1, the following warning is shown.
    
        $ make # on Jenkins (build server)
        […]
            CC         mainboard/asrock/e350m1/mptable.ramstage.o
        src/mainboard/asrock/e350m1/mptable.c:64:12: warning: unused variable 'dev' [-Wunused-variable]
        […]
    
    Removing the variable `dev` addresses the warning.
    
    The same change was done in the following commit for the
    AMD Persimmon board.
    
        commit d7a696d0f229abccc95ff411f28d91b9b796ab74
        Author: efdesign98 <efdesign98 at gmail.com>
        Date:   Thu Sep 15 15:24:26 2011 -0600
    
            Persimmon updates for AMD F14 rev C0
    
            Reviewed-on: http://review.coreboot.org/137
    
    Change-Id: I83f4630cb6ab1e4c95d04b4e8423850ed1858e45
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-on: http://review.coreboot.org/2965
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>

Build-Tested: build bot (Jenkins) at Fri Mar 29 21:24:46 2013, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer at coreboot.org> at Mon Apr  1 21:07:46 2013, giving +2
See http://review.coreboot.org/2965 for details.

-gerrit



More information about the coreboot-gerrit mailing list