[coreboot-gerrit] Patch merged into coreboot/master: 8253bd9 tegra124: fix the dangerous VPR write order

gerrit at coreboot.org gerrit at coreboot.org
Fri Nov 14 07:28:11 CET 2014


the following patch was just integrated into master:
commit 8253bd912ae4cd65ac0aa9ecaebb3aa7efd46cb0
Author: Joseph Lo <josephl at nvidia.com>
Date:   Fri Mar 28 19:13:51 2014 +0800

    tegra124: fix the dangerous VPR write order
    
    Currently we put the VPR write code just right before the AVP is going
    to freeze. We have no idea does the write operation successful or not
    before halting the AVP. And the power_on_main_cpu should be the last step
    of that. So we make a fix to change the order.
    
    BUG=none
    BRANCH=none
    TEST=LP0 suspend stress test and check the VPR is correct;
         LP0 suspend stress test with video playback
    
    Original-Change-Id: Ia62dde2a020910de39796d1cf62c1bf185cdb372
    Original-Signed-off-by: Joseph Lo <josephl at nvidia.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/192029
    Original-Reviewed-by: Tom Warren <twarren at nvidia.com>
    Original-Reviewed-by: Andrew Bresticker <abrestic at chromium.org>
    Original-Commit-Queue: Tom Warren <twarren at nvidia.com>
    Original-Tested-by: Tom Warren <twarren at nvidia.com>
    (cherry picked from commit 51473811fa477cca9ad9cbafdaad4fd4a2309234)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: Ia28329e38fcf12994594b73c805d061804aa01c4
    Reviewed-on: http://review.coreboot.org/7459
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
    Tested-by: build bot (Jenkins)


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

-gerrit



More information about the coreboot-gerrit mailing list