All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marian Balakowicz <m8@semihalf.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH 5/8] [new uImage] Add libfdt support to mkimage
Date: Mon, 03 Mar 2008 11:01:58 +0100	[thread overview]
Message-ID: <47CBCC96.4080802@semihalf.com> (raw)
In-Reply-To: <47C9BD4B.3090902@gmail.com>

Jerry Van Baren wrote:
> Marian Balakowicz wrote:
>> From: Bartlomiej Sieka <tur@semihalf.com>
>>
>> Signed-off-by: Bartlomiej Sieka <tur@semihalf.com>
>> ---
>>
>>  Makefile              |    3 +++
>>  include/libfdt_env.h  |    4 ++++
>>  libfdt/fdt.c          |    4 ++++
>>  libfdt/fdt_ro.c       |    4 ++++
>>  libfdt/fdt_rw.c       |    4 ++++
>>  libfdt/fdt_strerror.c |    4 ++++
>>  libfdt/fdt_wip.c      |    4 ++++
>>  tools/.gitignore      |    7 +++++++
>>  tools/Makefile        |   51
>> ++++++++++++++++++++++++++++++++++++++++++++++---
>>  tools/fdt_host.h      |   28 +++++++++++++++++++++++++++
>>  tools/mkimage.h       |    1 +
>>  11 files changed, 111 insertions(+), 3 deletions(-)
>>  create mode 100644 tools/fdt_host.h
> 
> Arrrr, I'm not wild about this, but I cannot suggest a better way. :-(
> 
> All these (trivial) touches to libfdt/* are annoying and could cause
> patch tracking of the dtc/libfdt source to break.  I guess that is the
> risk we live with unless or until someone comes up with a better way.  :-(

Right, this is ugly, I don't like that solution either. I'd be glad to
have something cleaner but I spent a few moments on it and can't
figure out anything better and simple enough.

Cheers,
m.

  reply	other threads:[~2008-03-03 10:01 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-29 23:19 [U-Boot-Users] [PATCH 0/8] [new uImage] patchset6 - new uImage format low level API Marian Balakowicz
2008-02-29 23:19 ` [U-Boot-Users] [PATCH 1/8] [new uImage] Update naming convention for bootm/uImage related code Marian Balakowicz
2008-02-29 23:19 ` [U-Boot-Users] [PATCH 2/8] [new uImage] Share common uImage code between mkimage and U-boot Marian Balakowicz
2008-02-29 23:19 ` [U-Boot-Users] [PATCH 3/8] [new uImage] Fix component handling for legacy multi component images Marian Balakowicz
2008-02-29 23:19 ` [U-Boot-Users] [PATCH 4/8] [new uImage] Add sha1.o object to mkimage binary build Marian Balakowicz
2008-02-29 23:19 ` [U-Boot-Users] [PATCH 5/8] [new uImage] Add libfdt support to mkimage Marian Balakowicz
2008-03-01 20:32   ` Jerry Van Baren
2008-03-03 10:01     ` Marian Balakowicz [this message]
2008-03-03 14:29     ` Jon Loeliger
2008-02-29 23:19 ` [U-Boot-Users] [PATCH 6/8] libfdt: Add and use a node iteration helper function Marian Balakowicz
2008-03-01 19:02   ` Jerry Van Baren
2008-03-03  9:21     ` Marian Balakowicz
2008-03-03 13:14       ` Jerry Van Baren
2008-02-29 23:20 ` [U-Boot-Users] [PATCH 7/8] [new uImage] Fix FDT blob totalsize calculation in boot_relocate_fdt() Marian Balakowicz
2008-02-29 23:20 ` [U-Boot-Users] [PATCH 8/8] [new uImage] New uImage low-level API Marian Balakowicz

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=47CBCC96.4080802@semihalf.com \
    --to=m8@semihalf.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 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.