[coreboot-gerrit] Patch merged into coreboot/master: b3f08c6 cmos: Rename the CMOS related functions.

gerrit at coreboot.org gerrit at coreboot.org
Wed Oct 22 03:55:22 CEST 2014


the following patch was just integrated into master:
commit b3f08c61f15970ef3d9e197b02d6dedb8b2c5830
Author: Gabe Black <gabeblack at google.com>
Date:   Wed Apr 30 17:12:25 2014 -0700

    cmos: Rename the CMOS related functions.
    
    Most of the code related to the mc146818 is not related to the RTC and is
    really for managing the CMOS storage. Since we intend to add a generic API
    for RTC drivers it's inconvenient for those functions to have an rtc_ prefix.
    This CL renames those functions so they start with cmos_ instead. There are
    some places where rtc_init was called with a comment that says something about
    starting the RTC. That wasn't correct before (the RTC is always running), but
    it looks a little odd now that the function is called cmos_init.
    
    This CL also opportunistically cleans up some style problems in this file.
    
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Reviewed-on: https://chromium-review.googlesource.com/197794
    Reviewed-by: Gabe Black <gabeblack at chromium.org>
    Tested-by: Gabe Black <gabeblack at chromium.org>
    Commit-Queue: Gabe Black <gabeblack at chromium.org>
    (cherry picked from commit 9a9ad24888b185fb58965457704e326bb508d788)
    
    Removed the addition of stdint.h to mc146818rtc.h since
    types.h is now included. Changed rtc_init to cmos_init for
    fsp_bd82x6x, fsp_rangeley, fsp_baytrail, ibexpeak, vortex86ex.
    
    Change-Id: Id4b9f6bea93e8bd5eaef2cb17f296adb9697114c
    Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
    Reviewed-on: http://review.coreboot.org/6977
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


See http://review.coreboot.org/6977 for details.

-gerrit



More information about the coreboot-gerrit mailing list