From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Mon, 29 Jul 2013 09:24:49 +0200 Subject: [U-Boot] [PATCH] OpenRD: relocate environment to 640kB In-Reply-To: <20130715121957.GX13531@bill-the-cat> References: <1369647746-18120-1-git-send-email-t-uboot@infra-silbe.de> <20130611120019.182fa315@lilith> <20130627114159.59225f02@lilith> <20130715121957.GX13531@bill-the-cat> Message-ID: <20130729092449.1d5adf2b@lilith> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Sascha, On Mon, 15 Jul 2013 08:19:57 -0400, Tom Rini wrote: > On Mon, Jul 15, 2013 at 11:23:54AM +0200, Sascha Silbe wrote: > > Albert ARIBAUD writes: > > > > >> The situation has gotten better recently and U-Boot fits into the > > >> previous partition size of 384KiB again. So it isn't broken on OpenRD > > >> anymore and the above would seem like a good approach. > > > How well does it fit again, and do you have any idea what caused the > > > increase in size, and what caused the decrease? > > > > I had the same questions and tried a few buildman runs, but didn't get a > > clear picture. The size was going up and down for various slices of > > commits. > > > > With v2013.07-rc3, we are now at 376344B (??? 96% of 384KiB) for > > openrd_ultimate when built on Debian Wheezy using > > gcc-4.7-arm-linux-gnueabi from Emdebian. > > > > Is there an equivalent to CONFIG_SPL_MAX_SIZE for the "regular" U-Boot? > > Detecting the overlap at build time would prevent bricking the device > > using saveenv at run time. As an additional benefit, commits that push > > the size beyond the limit would also show up in buildman reports as > > build failures. > > Yes, you can try using CONFIG_BOARD_SIZE_LIMIT, which is missing from > the README, but does have a few examples (git grep around). A patch to > document it, and then a patch to enable for openrd would be much > appreciated. Thanks! Sascha, does this mean the env relocate patch here is dropped in favor of a CONFIG_BOARD_SIZE_LIMIT check at build time? Amicalement, -- Albert.