All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/2] board: tbs2910: Remove FIT support in defconfig to reduce u-boot size
Date: Wed, 9 Jan 2019 17:01:37 +0100	[thread overview]
Message-ID: <a8158a1b-b0d0-6e5d-af0e-54a533d74a2b@denx.de> (raw)
In-Reply-To: <63e7db53-dcaa-35cf-0d0e-f7bf5483c5c8@web.de>

Hi Soeren,

On 08/01/19 12:03, Soeren Moch wrote:
> Hi Stefano,
> 
> On 08.01.19 11:24, Stefano Babic wrote:
>> Hi Soeren,
>>
>> On 08/01/19 11:14, Soeren Moch wrote:
>>> Stefano,
>>>
>>> can you apply this for v2019.01? This is really a important fix to avoid
>>>  environment and u-boot binary overwriting each other.
>>> It is also a small local fix which cannot hurt anybody else.
>> I will apply and I send a new PR. This is not the first fix in this
>> direction, u-boot becomes pretty large, it is becoming a common problem.
>>
> Thank you very much.
> 
> Yes, "in the good old days (tm)" there was much effort put into not
> increasing the binary size for existing boards when adding new features.

Right, fully agree.

> Unfortunately this is not true anymore.

I get in the same trouble with more as one project. A previous rule of
thumb was to reserve 512KB to the bootloader because it was pretty
unthinkable that bootloader could be larger. Mhmmhh....this remember me
someone else who said that 640Kb is enough for everything.

Anyway, as you noted, this is a big problem in field and it makes
difficult an upgrade without returning back the device to factory, what
nobody wants.

> And all the shiny new driver model / OF features only bring
> disadvantages for maintainers and users of existing boards.
> 
> I totally understand the desire to convert the driver code to "something
> better", which is easier to maintain long-term. But apparently new code
> is only tested on a single platform, while constantly breaking support
> for others without notice. But 'm complaining to the wrong person...

Yes, I am not the right person ;-)

I picked up your patches, I will run build tests and if everything is
fine I will sent PR for 2019.01.

Best regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

  reply	other threads:[~2019-01-09 16:01 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-05  8:31 [U-Boot] [PATCH 1/2] board: tbs2910: Add u-boot.imx size limit check Soeren Moch
2019-01-05  8:31 ` [U-Boot] [PATCH 2/2] board: tbs2910: Remove FIT support in defconfig to reduce u-boot size Soeren Moch
2019-01-08 10:14   ` Soeren Moch
2019-01-08 10:24     ` Stefano Babic
2019-01-08 11:03       ` Soeren Moch
2019-01-09 16:01         ` Stefano Babic [this message]
2019-01-09 22:39           ` Tom Rini
2019-01-10  1:28             ` Soeren Moch
2019-01-10  2:30               ` Tom Rini
2019-01-10 14:03                 ` Soeren Moch
2019-01-10 15:06                   ` Tom Rini
2019-01-11 13:11                     ` Soeren Moch
2019-01-11 14:32                       ` Tom Rini
2019-01-10  8:00             ` Stefano Babic
2019-01-10  8:11               ` Simon Goldschmidt
2019-01-10 15:56                 ` Tom Rini
2019-01-10 16:36                   ` Simon Goldschmidt
2019-01-10 16:54                     ` Tom Rini
2019-01-11  6:43                       ` Simon Goldschmidt
2019-01-11  7:22                       ` Simon Goldschmidt
2019-01-11 14:44                         ` Tom Rini
2019-01-10 14:44               ` Tom Rini
2019-01-10 14:51                 ` Stefano Babic
2019-01-10 15:12                   ` Tom Rini
2019-01-10 22:46                     ` Stefano Babic
2019-01-11  6:27                       ` Simon Goldschmidt
2019-01-10 15:53               ` Tom Rini
2019-01-11 13:11                 ` Sören Moch
2019-01-10  8:09             ` Joakim Tjernlund

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=a8158a1b-b0d0-6e5d-af0e-54a533d74a2b@denx.de \
    --to=sbabic@denx.de \
    --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.