public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Karl Apsite <Karl.Apsite@dornerworks.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v1 0/3] New tag for Flattened Image Trees (FIT) - Booting Xen from a FIT.
Date: Fri, 15 May 2015 10:33:02 -0400	[thread overview]
Message-ID: <5556039E.20002@dornerworks.com> (raw)
In-Reply-To: <1431521640-31083-1-git-send-email-Karl.Apsite@dornerworks.com>

Hi Simon,

I read your initial comments, and I immediately realized I made a mistake.  I
had initially submitted my series on a local branch, but my mailing client
didn't send out anything past the cover letter.  (I think Patman had CC'd too
many people, and my mailer was preventing spam)  When you first responded, I
then rushed out the patch again, trimming down what Patman was trying to do.  I
was on a different branch and had made a couple small efforts towards
refactoring. On that branch, I had intended to submit commits A, B, and C, but
instead, sent you C, D, and E.

Commit A will have the tests I added to the test runner. Commit B will have some
of the missing documentation you had mentioned.  And you responded favorably to
Commit D.

I will submit a new patch series with the correct commits (A, B, C, and D), with
modifications based on your review comments.  I will not be pursuing a larger
refactor at this time.

Thank you, and I apologize for any confusion I may have caused.

On 05/13/2015 08:53 AM, Karl Apsite wrote:
> 
> The FIT config now supports a tag named "loadables:" which is a
> comma separated list.  Users can add any number of images to the list,
> and u-boot will move the selected binaries to their listed
> load_addresses. This allows u-boot to boot xen from using an FIT
> configuration.  Xen expects a kernel to be placed at a predetermined
> location, however the "kernel" field was already filled by xen itself.
> This change allows the user to move the required binary before xen
> boots, all within the FIT's configuration.
> 
> 
> Karl Apsite (3):
>   add boot_get_loadables() to load listed images
>   Combine bootm_find_<thing> functions together
>   Remove the bootm_find_other() wrapper
> 
>  common/bootm.c      | 50 ++++++++++------------------
>  common/cmd_bootm.c  |  4 +--
>  common/image-fit.c  |  8 ++++-
>  common/image.c      | 93 +++++++++++++++++++++++++++++++++++++++++++++++++++++
>  include/bootm.h     |  2 +-
>  include/bootstage.h |  1 +
>  include/image.h     |  5 ++-
>  7 files changed, 125 insertions(+), 38 deletions(-)
> 

  parent reply	other threads:[~2015-05-15 14:33 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-13 12:53 [U-Boot] [PATCH v1 0/3] New tag for Flattened Image Trees (FIT) - Booting Xen from a FIT Karl Apsite
2015-05-13 12:53 ` [U-Boot] [PATCH v1 1/3] add boot_get_loadables() to load listed images Karl Apsite
2015-05-15 13:57   ` Simon Glass
2015-05-15 20:17     ` Karl Apsite
2015-05-15 20:38       ` Simon Glass
2015-05-13 12:53 ` [U-Boot] [PATCH v1 2/3] Combine bootm_find_<thing> functions together Karl Apsite
2015-05-15 13:57   ` Simon Glass
2015-05-15 20:17     ` Karl Apsite
2015-05-13 12:54 ` [U-Boot] [PATCH v1 3/3] Remove the bootm_find_other() wrapper Karl Apsite
2015-05-15 13:58   ` Simon Glass
2015-05-15 13:58     ` Simon Glass
2015-05-15 14:33 ` Karl Apsite [this message]
2015-05-15 14:52   ` [U-Boot] [PATCH v1 0/3] New tag for Flattened Image Trees (FIT) - Booting Xen from a FIT Simon Glass
  -- strict thread matches above, loose matches on Subject: below --
2015-05-11 18:03 Karl Apsite
2015-05-13  1:52 ` Simon Glass

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=5556039E.20002@dornerworks.com \
    --to=karl.apsite@dornerworks.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