From: Thomas Hebb <tommyhebb@gmail.com>
To: linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org
Cc: Masahiro Yamada <masahiroy@kernel.org>,
Thomas Hebb <tommyhebb@gmail.com>,
Palmer Dabbelt <palmer@dabbelt.com>,
Paul Walmsley <paul.walmsley@sifive.com>,
linux-riscv@lists.infradead.org
Subject: [PATCH v2 0/3] kconfig: rework symbol help text
Date: Tue, 17 Dec 2019 08:15:42 -0800 [thread overview]
Message-ID: <cover.1576599202.git.tommyhebb@gmail.com> (raw)
This series fixes several issues with help text generated by Kconfig,
mainly affecting symbols that are defined in multiple places. Although
results of these patches are somewhat visible for the symbols in Linux,
what prompted me to write the series was working on U-Boot, which also
uses Kconfig and makes very heavy use of multiple definitions (e.g. for
overriding defaults). I have provided Linux examples where I could find
them, but the example for the biggest patch (the first one) is taken
from U-Boot because it was more illustrative than anything I could find
in Linux.
Changes in v2:
- Added explicit U-Boot version in commit message + other rewordings
- Made the new "Depends on:" line print actual dependencies instead of
visibility to avoid an intra-series regression, and noted that in the
commit message.
- Get rid of redundant "with prompt" and "without prompt" notes in
definition text, but continue to ensure that definitions with prompts
are printed before ones without.
- Fixed checkpatch issues
- Omit already-merged patch "kconfig: don't crash on NULL expressions in
expr_eq()"
Thomas Hebb (3):
kconfig: list all definitions of a symbol in help text
kconfig: distinguish between dependencies and visibility in help text
kconfig: fix nesting of symbol help text
scripts/kconfig/expr.c | 3 +-
scripts/kconfig/expr.h | 1 +
scripts/kconfig/menu.c | 82 +++++++++++++++++++++++++-----------------
3 files changed, 52 insertions(+), 34 deletions(-)
--
2.24.1
next reply other threads:[~2019-12-17 16:16 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-17 16:15 Thomas Hebb [this message]
2019-12-17 16:15 ` [PATCH v2 1/3] kconfig: list all definitions of a symbol in help text Thomas Hebb
2019-12-17 16:15 ` [PATCH v2 2/3] kconfig: distinguish between dependencies and visibility " Thomas Hebb
2019-12-17 16:15 ` [PATCH v2 3/3] kconfig: fix nesting of symbol " Thomas Hebb
2019-12-18 14:09 ` [PATCH v2 0/3] kconfig: rework " Masahiro Yamada
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=cover.1576599202.git.tommyhebb@gmail.com \
--to=tommyhebb@gmail.com \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-riscv@lists.infradead.org \
--cc=masahiroy@kernel.org \
--cc=palmer@dabbelt.com \
--cc=paul.walmsley@sifive.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox