[U-Boot] Fwd: [PATCH] OMAP4: speed up booting on Pandaboard

Steve Sakoman
2010-11-04T18:10:07Z

Forgot to copy list on my reply!

---------- Forwarded message ----------
From: Steve Sakoman <sakoman@gmail.com>
Date: Thu, Nov 4, 2010 at 11:09 AM
Subject: Re: [U-Boot] [PATCH] OMAP4: speed up booting on Pandaboard
To: Sergiy Kibrik <sergiy.kibrik@globallogic.com>

On Thu, Nov 4, 2010 at 2:38 AM, Sergiy Kibrik
<sergiy.kibrik@globallogic.com> wrote:
> Improved default config for OMAP4 Pandaboard for faster boot:
> ? ? ? ?-reduced environment size to speed up memory initialization;

How much time does this save? ?I think shrinking to 2K might be a bit
extreme, how about 4K?

> ? ? ? ?-USB TTY driver turned off;

Not sure I agree with this. ?I think it might be useful to leave USB
gadget support in, since there are those who might prefer a USB
console over the serial port.

However if others disagree, I have this comment: ?If you are going to
turn off the gadget, why not remove the USB driver too?

> ? ? ? ?-tweaked blob load address to avoid image relocation (according to default uImage load address);

I don't think this does what you think it does! ?Unless I am horribly
mistaken, the loadaddr address environment variable has nothing to do
with relocation! ?It is the address typically used to store data when
you do a fatload, tftp, bootp, etc.

Steve

> Signed-off-by: Sergiy Kibrik <sergiy.kibrik@globallogic.com>
> ---
> ?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)
> ?#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
> ?#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" \
> ? ? ? ?"console=ttyS2,115200n8\0" \
> ? ? ? ?"usbtty=cdc_acm\0" \
> ? ? ? ?"vram=16M\0" \
> --
> 1.7.0.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>