[coreboot-gerrit] Patch merged into coreboot/master: f6280bc tegra124: Fix some bugs in the clock configuration macros.

gerrit at coreboot.org gerrit at coreboot.org
Thu Nov 13 06:26:58 CET 2014


the following patch was just integrated into master:
commit f6280bc6501a0ea8f1226f9177d80d614f75853e
Author: Gabe Black <gabeblack at google.com>
Date:   Wed Mar 5 21:53:02 2014 -0800

    tegra124: Fix some bugs in the clock configuration macros.
    
    There were some missing parenthesis and some extra semicolons which this
    change adds and removes, respectively.
    
    BUG=chrome-os-partner:25467
    TEST=Built and booted on nyan rev1. Verified that the same frequency calculated
    differently results in the same settings. Before operator precedence would
    pull apart the frequency calculation and use the pieces in the wrong order.
    BRANCH=None
    
    Original-Change-Id: I843d4ae9f7a2ae362926d94b6b77ef31d350a329
    Original-Signed-off-by: Gabe Black <gabeblack at google.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/189013
    Original-Reviewed-by: Hung-Te Lin <hungte at chromium.org>
    Original-Reviewed-by: Tom Warren <twarren at nvidia.com>
    Original-Reviewed-by: Julius Werner <jwerner 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 462e61ad898a4d6a99c1d161d77bde245c5b1f5c)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: Ifce3aac262cf5e2ec0496c5b3ad894bf6f0f9a46
    Reviewed-on: http://review.coreboot.org/7416
    Tested-by: build bot (Jenkins)
    Reviewed-by: David Hendricks <dhendrix at chromium.org>


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

-gerrit



More information about the coreboot-gerrit mailing list