All of lore.kernel.org
 help / color / mirror / Atom feed
From: Detlev Zundel <dzu@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC][PATCH] mkimage: Add compatibility option for legacy Multi-File images
Date: Wed, 01 Sep 2010 17:04:09 +0200	[thread overview]
Message-ID: <m21v9dec9i.fsf@ohwell.denx.de> (raw)
In-Reply-To: <1283237421.5338.8.camel@debian-thib> (Thibaut Girka's message of "Tue, 31 Aug 2010 08:50:21 +0200")

Hi Thibaut,

> Le lundi 30 ao?t 2010 ? 11:29 +0200, Detlev Zundel a ?crit :
>> Hi Thibaut,
> Hi,
>
>> generally I'm not a fan to include workarounds for bugs which we do not
>> have anymore in mainline U-Boot.
>
> Hm, yeah, I can understand that...
>
>> Isn't there any other alternative for this?
>
> Well, for my use case, we have to workaround this bug.
> We can do that (adding a byte at the end of some files) outside of
> mkimage, but it's really a u-boot thing, so, it could really go in
> there.

If it goes in, can we set a date to remove it again?  Or do we have to
carry it around forever?  If the latter is the case then I'm really not
a big fan of it.  If you can fix the problem up in the "project where it
is a problem", then the fix would better belong there.

Cheers
  Detlev

-- 
The  mathematician's patterns,  like the  painter's or  the poet's,  must be
beautiful;  the ideas, like the colours or the words, must fit together in a
harmonious way. Beauty is the first test: there is no permanent place in the
world for ugly mathematics.                       -- G. H. Hardy
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de

  reply	other threads:[~2010-09-01 15:04 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-18 20:48 [U-Boot] [RFC][PATCH] mkimage: Add compatibility option for legacy Multi-File images Thibaut Girka
2010-08-30  9:29 ` Detlev Zundel
2010-08-31  6:50   ` Thibaut Girka
2010-09-01 15:04     ` Detlev Zundel [this message]
2010-09-02 15:05 ` Wolfgang Denk
2010-08-29 13:05 Per Andersson

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=m21v9dec9i.fsf@ohwell.denx.de \
    --to=dzu@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.