[coreboot-gerrit] Patch merged into coreboot/master: 9d32739 tegra124: More improvements to the clock initialization macros.

gerrit at coreboot.org gerrit at coreboot.org
Mon Dec 15 19:58:22 CET 2014


the following patch was just integrated into master:
commit 9d32739baaf86e845436aec6c43580c5626d3499
Author: Gabe Black <gabeblack at google.com>
Date:   Mon Apr 7 01:05:44 2014 -0700

    tegra124: More improvements to the clock initialization macros.
    
    Consolidate the register setting clrsetbits_le32 call to simplify the macros.
    Add a check for bits of the divisor being dropped. The clock source registers
    will throw away bits that aren't supported, so we can check for divisor
    overflow by checking for dropped bits.
    
    BUG=None
    TEST=Purposefully tried to set a clock to a rate which overflows its divisor.
    Verified that the check triggered. Booted on nyan. Verified the TPM i2c bus
    frequency was still correct.
    BRANCH=None
    
    Original-Change-Id: I3b1b6ba57f6b7729f303d15a16b685a48751d41f
    Original-Signed-off-by: Gabe Black <gabeblack at google.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/193348
    Original-Reviewed-by: Gabe Black <gabeblack at chromium.org>
    Original-Commit-Queue: Gabe Black <gabeblack at chromium.org>
    Original-Tested-by: Gabe Black <gabeblack at chromium.org>
    (cherry picked from commit 9cd79dd974d8a3c31398f8fbd62750b194867891)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: Id4d8ecfeff52737cdd68999028b37cbdedb0d116
    Reviewed-on: http://review.coreboot.org/7738
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list