From 1a142cd2c51c6f51a1597c21ad513feb151e0938 Mon Sep 17 00:00:00 2001 From: Julius Werner Date: Thu, 6 Nov 2014 17:32:58 -0800 Subject: [PATCH] rk3288: Adjust CBFS header and ROM offsets Our CBFS header offset on rk3288 was very low and overlapped with the end of the bootblock on recent Pinky builds. This can create all kinds of fun effects like BSS variables suddenly being initialized to something else than zero, in an effect that jumps somewhere else for every slightest code size change. This patch moves the CBFS header offset up a bit and the CBFS ROM offset down (because there's really no point in leaving such a large gap). This resolves our immediate booting problems, and I'll also start on a patch to add further checks somewhere that catch these overlaps in the future. BRANCH=None BUG=None TEST=Created a Pinky image from the exact same commit version as the official 6443.0.0 build, with a KERNELREVISION string of the exact same length as the builder (which for some arcane reason is different than running emerge locally, shifting the whole bootblock around with it). Confirmed that I saw the same "Not enough room for another sub-pagetable!" hang, and that this patch fixes it. Change-Id: I8be5b7b7e87021cc1b3a91d336e8d233546ee188 Signed-off-by: Julius Werner Reviewed-on: https://chromium-review.googlesource.com/228326 Reviewed-by: Gediminas Ramanauskas Reviewed-by: David Hendricks --- src/soc/rockchip/rk3288/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/soc/rockchip/rk3288/Kconfig b/src/soc/rockchip/rk3288/Kconfig index 113f5e0cc3..a36a861dfc 100644 --- a/src/soc/rockchip/rk3288/Kconfig +++ b/src/soc/rockchip/rk3288/Kconfig @@ -53,10 +53,10 @@ config BOOTBLOCK_ROM_OFFSET config CBFS_HEADER_ROM_OFFSET hex - default 0x0008000 + default 0x0010000 config CBFS_ROM_OFFSET hex - default 0x0018000 + default 0x0010100 endif