All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Arnaud Rébillout" <rebillout@syscom.ch>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] package/uboot-tools: Add mkenvimage for host.
Date: Wed, 19 Dec 2012 14:03:25 +0100	[thread overview]
Message-ID: <50D1BB1D.3090907@syscom.ch> (raw)
In-Reply-To: <20121219135800.6182a9ba@skate>

Hi Mr. Petazzoni,

>   *) Could you send a proper git patch, with a description,
>   Signed-off-by line, and sent using git send-email so that it doesn't
>   get line-wrapped by your mail client.
Ok I will look at that and do my best.

>   *) I would like to see an option being added in the U-Boot
>   configuration options to pass a txt file containing an environment
>   definition, which would be passed through mkenvimage to generate an
>   image environment, and then installed in $(BINARIES_DIR). Could you
>   cook such a patch, or do you prefer if someone else does it?
Sounds good to me, I will send you a new patch. Hopefully before the end 
of the week.

Thanks,



On 19/12/12 13:58, Thomas Petazzoni wrote:
> Dear Arnaud R?billout,
>
> On Wed, 19 Dec 2012 13:47:11 +0100, Arnaud R?billout wrote:
>
>> I need to use mkenvimage. And it seems that the easiest way is to
>> automatically install it in the HOST_UBOOT_TOOLS_INSTALL_CMDS target.
>> The tool mkenvimage appeared v2011.12 of uboot, so it's present in all
>> the versions that buildroot supports.
>> The patch is a simple line, if anyone is interested.
> Indeed, mkenvimage is useful.
>
> However:
>
>   *) Could you send a proper git patch, with a description,
>   Signed-off-by line, and sent using git send-email so that it doesn't
>   get line-wrapped by your mail client.
>
>   *) I would like to see an option being added in the U-Boot
>   configuration options to pass a txt file containing an environment
>   definition, which would be passed through mkenvimage to generate an
>   image environment, and then installed in $(BINARIES_DIR). Could you
>   cook such a patch, or do you prefer if someone else does it?
>
> Thanks,
>
> Thomas


-- 
Arnaud R?billout
SYSCOM Instruments SA
rue de l'industrie 21
CH-1450 Sainte-Croix
Switzerland
Tel: +41 (0)24 455 44 11

  reply	other threads:[~2012-12-19 13:03 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-19 12:47 [Buildroot] [PATCH] package/uboot-tools: Add mkenvimage for host Arnaud Rébillout
2012-12-19 12:58 ` Thomas Petazzoni
2012-12-19 13:03   ` Arnaud Rébillout [this message]
2012-12-19 13:13     ` Thomas Petazzoni

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=50D1BB1D.3090907@syscom.ch \
    --to=rebillout@syscom.ch \
    --cc=buildroot@busybox.net \
    /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.