[coreboot-gerrit] New patch to review for coreboot: c699ad6 rk3288: Increase PD_BUS_ACLK (SRAM clock) to improve boot speed

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon Apr 13 14:09:00 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9600

-gerrit

commit c699ad69086949021d8926d86c29af91cbe0f1e6
Author: Julius Werner <jwerner at chromium.org>
Date:   Mon Oct 20 18:04:29 2014 -0700

    rk3288: Increase PD_BUS_ACLK (SRAM clock) to improve boot speed
    
    This patch doubles the ACLK peripheral clock for the PD_BUS power domain
    to 297MHz, which is the closest to the maximum of 300MHz we can reach by
    dividing GPLL. This frequency directly translates into SRAM speed, so
    maximizing it has a huge impact on boot speed (especially with the lack
    of SRAM caching).
    
    BUG=chrome-os-partner:32987
    TEST=Booted Veyron_Pinky. Hacked timestamps into vboot and confirmed
    that the (visibly) long signature verification times are nearly halved.
    
    Change-Id: Iafa3044854a4058a7f885c775119d964a6295de4
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: c230585f4344d0eab4f8eeaa761869965f2da08a
    Original-Change-Id: I3f19eaa3d97dcc6235d820c71eb5edf2ae87d647
    Original-Signed-off-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/224524
    Original-Trybot-Ready: Doug Anderson <dianders at chromium.org>
    Original-Reviewed-by: David Hendricks <dhendrix at chromium.org>
---
 src/soc/rockchip/rk3288/include/soc/clock.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/soc/rockchip/rk3288/include/soc/clock.h b/src/soc/rockchip/rk3288/include/soc/clock.h
index f17428d..b8f892b 100644
--- a/src/soc/rockchip/rk3288/include/soc/clock.h
+++ b/src/soc/rockchip/rk3288/include/soc/clock.h
@@ -29,7 +29,8 @@
 #define CPLL_HZ		(384*MHz)
 #define NPLL_HZ		(384*MHz)
 
-#define PD_BUS_ACLK_HZ	(148500*KHz)
+/* The SRAM is clocked off aclk_bus, so we want to max it out for boot speed. */
+#define PD_BUS_ACLK_HZ	(297000*KHz)
 #define PD_BUS_HCLK_HZ	(148500*KHz)
 #define PD_BUS_PCLK_HZ	(74250*KHz)
 



More information about the coreboot-gerrit mailing list