public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/3] sunxi: Fix boot of Cubietruk and al.
Date: Thu, 19 Oct 2017 09:20:47 -0400	[thread overview]
Message-ID: <20171019132047.GU12015@bill-the-cat> (raw)
In-Reply-To: <CACY+gR0XN4jjKTB66Y+XuFBoFOcd=ieermJLviJ4ibh4WthwuA@mail.gmail.com>

On Thu, Oct 19, 2017 at 12:10:44PM +0300, Siarhei Siamashka wrote:
> On Thu, Oct 19, 2017 at 11:26 AM, Maxime Ripard
> <maxime.ripard@free-electrons.com> wrote:
> > Hi,
> >
> > Most featureful boards, such as the Cubietruck, have been broken since
> > the release 2017.09.
> >
> > This is due to a size increase of the binary that will trip us across
> > the size we've been using in the u-boot-sunxi-with-spl.bin file.
> >
> > We would have two ways to work around it. The first one would be to
> > just increase the offset of the environment. However, since it would
> > break all the environments of our users and possibly the custom
> > partition scheme that they would have created, it doesn't really seem
> > like a smart move.
> >
> > Another one would be to start trimming down a bit our enabled options
> > in order to reduce the size and to gain some extra space for users
> > customisations. I've taken care some of the low hanging fruits, and we
> > should probably take another go at it in the future (and add a size
> > check in the image build somehow?)
> >
> > Maxime
> >
> > Maxime Ripard (3):
> >   ARM: sunxi: Disable USB host options by default
> >   ARM: sunxi: Disable FAT write by default
> >   efi_loader: Do not enable it by default for sunxi
> >
> >  arch/arm/Kconfig       | 4 ----
> >  lib/efi_loader/Kconfig | 2 +-
> >  2 files changed, 1 insertion(+), 5 deletions(-)
> 
> We can first enable Thumb2 build by default for 32-bit sunxi boards
> and this should fix Cubietruck problems.
> 
> $ cat .config | grep THUMB
> CONFIG_HAS_THUMB2=y
> # CONFIG_SYS_THUMB_BUILD is not set
> CONFIG_SPL_SYS_THUMB_BUILD=y
> 
> As a test, enabling CONFIG_SYS_THUMB_BUILD=y in Cubietruck_defconfig
> reduces the size of the resulting U-Boot binary.
> 
> == Before: ==
> 
> $ arm-linux-gnueabihf-size u-boot
>    text       data        bss        dec        hex    filename
>  489398      26492     249240     765130      bacca    u-boot.orig
> 
> == After: ==
> 
> $ arm-linux-gnueabihf-size u-boot
>    text       data        bss        dec        hex    filename
>  366314      26492     249232     642038      9cbf6    u-boot

Yes, I would strongly encourage enabling Thumb2 support instead of
removing various features.

I would also encourage looking at moving environment either "up", or
switching to env in filesystem in the future as a "Yes, we broke your
existing setup, but we gave you a real useful feature".

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171019/5dec1bb8/attachment.sig>

  reply	other threads:[~2017-10-19 13:20 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-19  8:26 [U-Boot] [PATCH 0/3] sunxi: Fix boot of Cubietruk and al Maxime Ripard
2017-10-19  8:26 ` [U-Boot] [PATCH 1/3] ARM: sunxi: Disable USB host options by default Maxime Ripard
2017-10-19  8:48   ` Alexander Graf
2017-10-19  9:11     ` Andre Przywara
2017-10-19 11:55       ` Maxime Ripard
2017-10-19  8:54   ` Peter Robinson
2017-10-19 11:44   ` Mark Kettenis
2017-10-19  8:26 ` [U-Boot] [PATCH 2/3] ARM: sunxi: Disable FAT write " Maxime Ripard
2017-10-19  8:26 ` [U-Boot] [PATCH 3/3] efi_loader: Do not enable it by default for sunxi Maxime Ripard
2017-10-19  8:43   ` Peter Robinson
2017-10-19  9:01     ` Maxime Ripard
2017-10-19  9:06       ` Peter Robinson
2017-10-19  9:12         ` Peter Robinson
2017-10-19 11:43           ` Maxime Ripard
2017-10-19 21:40             ` Rob Clark
2017-10-20  7:20               ` Maxime Ripard
2017-10-20 12:27                 ` Peter Robinson
2017-10-20 12:36                   ` Maxime Ripard
2017-10-20 12:54                     ` Tom Rini
2017-10-20 16:39                       ` Maxime Ripard
2017-10-20 17:57                         ` Tom Rini
2017-10-20 18:52                         ` Peter Robinson
2017-10-20 12:56                     ` Peter Robinson
2017-10-19  9:12         ` Maxime Ripard
2017-10-23 13:35     ` Heinrich Schuchardt
2017-10-19  8:51   ` Alexander Graf
2017-10-19 10:54     ` Jonathan Gray
2017-10-19 11:52     ` Maxime Ripard
2017-10-19 11:39   ` Mark Kettenis
2017-10-19 11:48     ` Maxime Ripard
2017-10-19 13:24   ` Tom Rini
2017-10-19  8:44 ` [U-Boot] [PATCH 0/3] sunxi: Fix boot of Cubietruk and al Alexander Graf
2017-10-19  9:11   ` Maxime Ripard
2017-10-19 12:10     ` Alexander Graf
2017-10-19  9:10 ` Siarhei Siamashka
2017-10-19 13:20   ` Tom Rini [this message]
2017-10-19 13:26   ` Maxime Ripard
2017-10-19 13:03 ` Andre Przywara
2017-10-19 13:24   ` Maxime Ripard
2017-10-19 13:31     ` Tom Rini
2017-10-19 14:42     ` Andre Przywara
2017-10-19 14:58       ` Maxime Ripard
2017-10-20 21:33         ` Dennis Gilmore
2017-10-23  7:35           ` Maxime Ripard
2017-10-24 17:05             ` Dennis Gilmore
2017-10-24 17:21               ` Andre Przywara
2017-10-25  9:42                 ` Maxime Ripard
2017-10-25  9:55                   ` Jagan Teki
2017-10-25 12:30                     ` Maxime Ripard
2017-10-25 10:01                   ` Andre Przywara
2017-10-25 11:58                 ` Siarhei Siamashka
2017-10-25 13:42                   ` Andre Przywara
     [not found]                     ` <20171025184101.aihe47qongf52e7c@excalibur.cnev.de>
2017-10-26  1:46                       ` Dennis Gilmore
2017-10-25 13:45                   ` Tom Rini
2017-10-25 13:50                   ` Maxime Ripard
2017-10-20 21:31     ` Dennis Gilmore
2017-10-19 13:28 ` Tom Rini
2017-10-19 13:50   ` Maxime Ripard

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=20171019132047.GU12015@bill-the-cat \
    --to=trini@konsulko.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