From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergiy Kibrik Date: Mon, 15 Nov 2010 12:26:30 +0200 Subject: [U-Boot] [PATCH] OMAP4: speed up booting on Pandaboard In-Reply-To: <20101114211818.2FD5D134F54@gemini.denx.de> References: <4CD27F08.8060309@globallogic.com> <20101114211818.2FD5D134F54@gemini.denx.de> Message-ID: <4CE10AD6.3010305@globallogic.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 11/14/2010 11:18 PM, Wolfgang Denk wrote: > Dear Sergiy Kibrik, > > In message <4CD27F08.8060309@globallogic.com> you wrote: >> Improved default config for OMAP4 Pandaboard for faster boot: >> -reduced environment size to speed up memory initialization; >> -USB TTY driver turned off; >> -tweaked blob load address to avoid image relocation (according to default uImage load address); >> >> Signed-off-by: Sergiy Kibrik >> --- >> include/configs/omap4_panda.h | 8 ++++---- >> 1 files changed, 4 insertions(+), 4 deletions(-) > ... >> - * Total Size Environment - 256k >> + * Total Size Environment - 2k >> * Malloc - add 256k >> */ >> -#define CONFIG_ENV_SIZE (256 << 10) >> +#define CONFIG_ENV_SIZE (256 << 4) > > I think we had this discussion before. "256 << 4" makes no sense when > you mean 2 KiB - it requires the reader to actually perform the > calculation, which means it makes it hard for him to understand the > code. Without need. Please either write 2048, which everybody can > parse easily, or write "2 << 10" which can be parsed easily well. > I've considered advices you gave me in this thread: http://marc.info/?t=128897628400001&r=1&w=2 and resubmitted the patch http://marc.info/?l=u-boot&m=128922349828119&w=2 Haven't you received that? -regards, Sergey