All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nishanth Menon <menon.nishanth@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] OMAP4: speed up booting on Pandaboard
Date: Thu, 04 Nov 2010 11:46:53 -0400	[thread overview]
Message-ID: <4CD2D56D.70302@gmail.com> (raw)
In-Reply-To: <4CD27F08.8060309@globallogic.com>

Sergiy Kibrik wrote, on 11/04/2010 05:38 AM:
> Improved default config for OMAP4 Pandaboard for faster boot:
> 	-reduced environment size to speed up memory initialization;
> 	-USB TTY driver turned off;
Do we really want to do this?
> 	-tweaked blob load address to avoid image relocation (according to default uImage load address);
>
> Signed-off-by: Sergiy Kibrik<sergiy.kibrik@globallogic.com>
What kind of savings did we get? I am guessing we have some time x savings..

> ---
>   include/configs/omap4_panda.h |    8 ++++----
>   1 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/include/configs/omap4_panda.h b/include/configs/omap4_panda.h
> index 74defab..5aba424 100644
> --- a/include/configs/omap4_panda.h
> +++ b/include/configs/omap4_panda.h
> @@ -62,10 +62,10 @@
>
>   /*
>    * Size of malloc() pool
> - * Total Size Environment - 256k
> + * Total Size Environment - 2k
>    * Malloc - add 256k
>    */
> -#define CONFIG_ENV_SIZE			(256<<  10)
> +#define CONFIG_ENV_SIZE			(256<<  4)
/me likes the change, but not sure how it saves boot time.

>   #define CONFIG_SYS_MALLOC_LEN		(CONFIG_ENV_SIZE + (256<<  10))
>   						/* initial data */
>   /* Vector Base */
> @@ -117,7 +117,7 @@
>
>   /* USB device configuration */
>   #define CONFIG_USB_DEVICE		1
> -#define CONFIG_USB_TTY			1
> +#undef CONFIG_USB_TTY
do we need to have the undef? it might be better to just drop the define 
perhaps? what impact do we have on boot time with this change?

>   #define CONFIG_SYS_CONSOLE_IS_IN_ENV	1
>
>   /* Flash */
> @@ -146,7 +146,7 @@
>   #define CONFIG_ENV_OVERWRITE
>
>   #define CONFIG_EXTRA_ENV_SETTINGS \
> -	"loadaddr=0x82000000\0" \
> +	"loadaddr=0x80007FC0\0" \
btw, Dumb question: how did we decide on this address? building from 
kernel.org, I see System.map @ c0004000 ==> 80004000

when I do make uImage,
/bin/bash /home/nmenon/Src/opensource/linux-2.6/scripts/mkuboot.sh -A 
arm -O linux -T kernel -C none -a 0x80008000 -e 0x80008000 -n 
'Linux-2.6.37-rc1-00005-gd88c092'


Regards,
Nishanth Menon

  reply	other threads:[~2010-11-04 15:46 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-04  9:38 [U-Boot] [PATCH] OMAP4: speed up booting on Pandaboard Sergiy Kibrik
2010-11-04 15:46 ` Nishanth Menon [this message]
2010-11-05 12:35   ` Sergiy Kibrik
2010-11-05 13:12     ` Nishanth Menon
2010-11-05 14:44       ` Sergiy Kibrik
2010-11-05 15:49         ` Nishanth Menon
2010-11-05 16:55           ` Sergiy Kibrik
     [not found] ` <AANLkTimzqVd6NqJ-s6-2Vw1gNbiOgJKHjNs-teTz5wFY@mail.gmail.com>
2010-11-04 18:10   ` [U-Boot] Fwd: " Steve Sakoman
2010-11-14 21:18 ` [U-Boot] " Wolfgang Denk
2010-11-15 10:26   ` Sergiy Kibrik

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4CD2D56D.70302@gmail.com \
    --to=menon.nishanth@gmail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.