From: "Robert P. J. Day" <rpjday@crashcourse.ca>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: "U-Boot Version 2 (barebox)" <barebox@lists.infradead.org>
Subject: Re: [PATCH] Clean up Kconfig and source for a couple device tree commands.
Date: Mon, 7 Jul 2014 06:07:47 -0400 (EDT) [thread overview]
Message-ID: <alpine.LFD.2.11.1407070606100.3134@localhost> (raw)
In-Reply-To: <4DA026CE-D019-4CBB-B46B-834064F91333@jcrosoft.com>
[-- Attachment #1: Type: TEXT/PLAIN, Size: 1716 bytes --]
On Mon, 7 Jul 2014, Jean-Christophe PLAGNIOL-VILLARD wrote:
>
> On Jul 5, 2014, at 12:18 AM, Robert P. J. Day <rpjday@crashcourse.ca> wrote:
>
> >
> >
> > Correct options listings for a couple device tree-related commands.
> >
> > Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> >
> > ---
> >
> > i *think* the following is valid.
> >
> > diff --git a/commands/Kconfig b/commands/Kconfig
> > index eed6fbd..3248de9 100644
> > --- a/commands/Kconfig
> > +++ b/commands/Kconfig
> > @@ -1950,10 +1950,11 @@ config CMD_OFTREE
> > help
> > oftree - handle device trees
> >
> > - Usage: oftree [-lpfdn] [DTB]
> > + Usage: oftree [-lspf] [DTB]
> >
> > Options:
> > -l Load DTB to internal device tree
> > + -s save internal device tree to DTB
> > -p probe devices from stored device tree
> > -f free stored device tree
> >
>
> do we really need the help here
>
> it’s a pain in the ass to keep it sync
as a followup to my reply, if it were me, i would just toss all the
usage/synopsis info from the Kconfig help -- as the OP quite
correctly points out, it's a pain in the ass to keep it sync'ed with
the source, and i've already run across several commands whose Kconfig
help doesn't match the source.
rday
--
========================================================================
Robert P. J. Day Ottawa, Ontario, CANADA
http://crashcourse.ca
Twitter: http://twitter.com/rpjday
LinkedIn: http://ca.linkedin.com/in/rpjday
========================================================================
[-- Attachment #2: Type: text/plain, Size: 149 bytes --]
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2014-07-07 10:08 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-04 16:18 [PATCH] Clean up Kconfig and source for a couple device tree commands Robert P. J. Day
2014-07-07 6:35 ` Sascha Hauer
2014-07-07 6:39 ` Jean-Christophe PLAGNIOL-VILLARD
2014-07-07 9:11 ` Robert P. J. Day
2014-07-07 18:09 ` Jean-Christophe PLAGNIOL-VILLARD
2014-07-07 21:10 ` Holger Schurig
2014-07-07 10:07 ` Robert P. J. Day [this message]
2014-07-07 12:07 ` Sascha Hauer
2014-07-07 21:11 ` Holger Schurig
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=alpine.LFD.2.11.1407070606100.3134@localhost \
--to=rpjday@crashcourse.ca \
--cc=barebox@lists.infradead.org \
--cc=plagnioj@jcrosoft.com \
/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.