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
>
next prev 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