[coreboot-gerrit] Patch merged into coreboot/master: src/lib: Clean up general issues found by checkpatch.pl

gerrit at coreboot.org gerrit at coreboot.org
Wed Mar 15 05:05:10 CET 2017


the following patch was just integrated into master:
commit 36984d85e758bd7b280a835c5c6e5c68b10e82b6
Author: Lee Leahy <leroy.p.leahy at intel.com>
Date:   Fri Mar 10 17:56:44 2017 -0800

    src/lib: Clean up general issues found by checkpatch.pl
    
    Fix the following errors and warnings detected by checkpatch.pl:
    
    ERROR: Bad function definition - void init_timer() should probably be void init_timer(void)
    ERROR: Prefixing 0x with decimal output is defective
    WARNING: Comparisons should place the constant on the right side of the test
    WARNING: char * array declaration might be better as static const
    
    TEST=Build and run on Galileo Gen2
    
    Change-Id: I9f618eea95e1f92fa34f4f89da27c0b16ae7f4ee
    Signed-off-by: Lee Leahy <Leroy.P.Leahy at intel.com>
    Reviewed-on: https://review.coreboot.org/18763
    Tested-by: build bot (Jenkins)
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>


See https://review.coreboot.org/18763 for details.

-gerrit



More information about the coreboot-gerrit mailing list