U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew F. Davis <afd@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] am335x_hs_evm: Trim options in SPL to reduce binary size
Date: Tue, 19 Dec 2017 08:54:25 -0600	[thread overview]
Message-ID: <b7b751b5-6984-e490-4f05-e72a694a99c2@ti.com> (raw)
In-Reply-To: <1513483493-5157-1-git-send-email-trini@konsulko.com>

On 12/16/2017 10:04 PM, Tom Rini wrote:
> The am335x_hs_evm runs into size constraint problems at times with
> various toolchains as changes come in due to the config have a large
> number of options in SPL (to showcase what is possible) while also
> having rather constrained binary limits.  Gain some of this room back by
> lowering the loglevel, disabling HW partition support and switching over
> to the tiny FIT image support.
> 
> Cc: Andrew F. Davis <afd@ti.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
> I'd really appreciate a run-time test of this patch if at all possible
> as I'm a little worried about TINY_FIT being incompatible with all of
> the security options.  Thanks!
> ---
>  configs/am335x_hs_evm_defconfig | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/configs/am335x_hs_evm_defconfig b/configs/am335x_hs_evm_defconfig
> index 48b0e8583997..8eb304686dc7 100644
> --- a/configs/am335x_hs_evm_defconfig
> +++ b/configs/am335x_hs_evm_defconfig
> @@ -13,10 +13,12 @@ CONFIG_ANDROID_BOOT_IMAGE=y
>  CONFIG_FIT_IMAGE_POST_PROCESS=y
>  CONFIG_SPL_LOAD_FIT=y
>  CONFIG_SPL_FIT_IMAGE_POST_PROCESS=y
> +CONFIG_LOGLEVEL=3
>  CONFIG_SYS_CONSOLE_INFO_QUIET=y
>  CONFIG_VERSION_VARIABLE=y
>  CONFIG_ARCH_MISC_INIT=y
>  CONFIG_SPL=y
> +CONFIG_SPL_FIT_IMAGE_TINY=y
>  # CONFIG_SPL_ENV_SUPPORT is not set
>  # CONFIG_SPL_EXT_SUPPORT is not set
>  CONFIG_SPL_MTD_SUPPORT=y
> @@ -37,6 +39,7 @@ CONFIG_DFU_RAM=y
>  CONFIG_DM_I2C=y
>  CONFIG_MISC=y
>  CONFIG_DM_MMC=y
> +# CONFIG_MMC_HW_PARTITIONING is not set

I haven't gotten around to testing the FIT_IMAGE_TINY stuff yet, but
conceptually I have a much bigger problem with this part.

Sacrificing functionality to allow continued SPL bloat is just wrong.

Whatever caused SPL to grow should be re-worked or the author should
have also made some optimization elsewhere to offset this. Now I'll have
to go hunt for more optimizations somewhere so I can get all my features
back here :(

>  CONFIG_MMC_OMAP_HS=y
>  CONFIG_NAND=y
>  CONFIG_NAND_OMAP_GPMC_PREFETCH=y
> @@ -61,5 +64,6 @@ CONFIG_USB_GADGET_VENDOR_NUM=0x0451
>  CONFIG_USB_GADGET_PRODUCT_NUM=0xd022
>  CONFIG_USB_GADGET_DOWNLOAD=y
>  CONFIG_USB_ETHER=y
> +CONFIG_SPL_TINY_MEMSET=y
>  CONFIG_RSA=y
>  CONFIG_LZO=y
> 

  parent reply	other threads:[~2017-12-19 14:54 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-17  4:04 [U-Boot] [PATCH] am335x_hs_evm: Trim options in SPL to reduce binary size Tom Rini
2017-12-18 23:37 ` Tom Rini
2017-12-19 14:54 ` Andrew F. Davis [this message]
2017-12-19 15:00   ` Tom Rini
2017-12-20 13:16     ` Jean-Jacques Hiblot
2017-12-21 12:22       ` Jean-Jacques Hiblot

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=b7b751b5-6984-e490-4f05-e72a694a99c2@ti.com \
    --to=afd@ti.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox