public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ben Warren <biggerbadderben@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Standardize bootp, tftpboot, rarpboot, dhcp, and nfs command descriptions
Date: Tue, 26 Aug 2008 13:30:32 -0700	[thread overview]
Message-ID: <48B467E8.7040909@gmail.com> (raw)
In-Reply-To: <1219770978.11226.34.camel@localhost.localdomain>

Peter Tyser wrote:
> Hi Ben,
>
> On Tue, 2008-08-26 at 09:31 -0700, Ben Warren wrote:
>   
>> Hi Peter,
>>
>> Peter Tyser wrote:
>>     
>>> cmd_net.c command descriptions were updated to describe the optional
>>> hostIPaddr argument.  The dhcp command help message was also updated
>>> to more closely reflect the other commands in cmd_net.c
>>>
>>> Signed-off-by: Peter Tyser <ptyser@xes-inc.com>
>>> ---
>>>   
>>>       
>> I'm too lazy to check if this is different from the patch that you sent 
>> previously (8/13 or so).  I imagine you want this one applied and the 
>> other discarded?
>>     
>
> The content of this patch is identical to the previous patch sent on
> 8/13.  However, the email on 8/13's title included "[RFC]" and in it I
> made the following comment:
>
> "Not sure if "boot" is the proper word to use in the command definitions
> since the downloaded images are only booted if "autostart=yes", and
> the majority of board configs don't have autostart set to yes.
> I'd vote for changing the descriptions to "load image via...".
> If a new, more granular bootm command is in the works,
> it seems like even less people will autoboot an image
> directly before doing some additional bootm commands to prep
> for booting an OS. Do others have an opinion?"
>
> No one seemed to bite as far as changing the wording of the commands, so
> I re-sent the same patch without the [RFC] and cleaned up the commit
> message to make it a formal patch.  So you're right, this patch should
> be applied and the previous one on 8/13 discarded.
>   
Good deal.  I'm hoping to get a few hours tonight to catch up.

regards,
Ben

  reply	other threads:[~2008-08-26 20:30 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-26 16:17 [U-Boot] [PATCH] Standardize bootp, tftpboot, rarpboot, dhcp, and nfs command descriptions Peter Tyser
2008-08-26 16:31 ` Ben Warren
2008-08-26 17:16   ` Peter Tyser
2008-08-26 20:30     ` Ben Warren [this message]
2008-08-27  5:46 ` Ben Warren

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=48B467E8.7040909@gmail.com \
    --to=biggerbadderben@gmail.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