linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Lukas Bulwahn <lukas.bulwahn@gmail.com>
To: Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
	linux-mips@vger.kernel.org
Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org,
	"Maciej W . Rozycki" <macro@orcam.me.uk>,
	Sander Vanheule <sander@svanheule.net>,
	Bert Vermeulen <bert@biot.com>,
	Alexey Malahov <Alexey.Malahov@baikalelectronics.ru>,
	Serge Semin <Sergey.Semin@baikalelectronics.ru>,
	Paul Burton <paulburton@kernel.org>,
	Daniel Silsby <dansilsby@gmail.com>,
	Paul Cercueil <paul@crapouillou.net>,
	Lukas Bulwahn <lukas.bulwahn@gmail.com>
Subject: [PATCH 0/9] Kconfig symbol clean-up on ./arch/mips/
Date: Mon, 13 Dec 2021 12:16:33 +0100	[thread overview]
Message-ID: <20211213111642.11317-1-lukas.bulwahn@gmail.com> (raw)

Dear Thomas,

The script ./scripts/checkkconfigsymbols.py warns on invalid references to
Kconfig symbols (often, minor typos, name confusions or outdated references).
This is a patch series addressing the issues reported by
./scripts/checkkconfigsymbols.py in the ./arch/mips/ directory, quickly
filtered down with:

  ./scripts/checkkconfigsymbols.py | grep "arch/mips" -B 1

without considering kernel configs, i.e., after removing ./arch/mips/configs
in the working tree. It addresses all issues I considered to be "true
positives"; so, issues that should be addressed and cleaned up. 

The patches on updating the comments might arguably be considered of lower
value by itself, but as the other patches show, checkkconfigsymbols does detect
some relevant issues in the kernel tree otherwise being unnoticed.
So, it might be worth to clean up the kernel tree to have checkkconfigsymbols
produce a rather short list of issues and then continuously follow and check
what checkkconfigsymbols reports.

These are the remaining reports from checkkconfigsymbols in ./arch/mips, which
are not addressed in this series:

  SGI_IP35
  Referencing files: arch/mips/include/asm/sn/addrs.h, arch/mips/include/asm/sn/agent.h, arch/mips/include/asm/sn/klconfig.h

  Rationale: Ralf Baechle rejected a patch to remove the "upstream-dead"
  SGI_IP35 code in 2014, because there was still some out-of-tree users.
  We might want to recheck if these out-of-tree users still exist or if this
  code can be dropped now.
  See email discussion: https://lore.kernel.org/lkml/1400584909.4912.35.camel@x220/T/#mf1d73d4f531bab670beb30aa0adfce34cbcad528

  SHELL
  Referencing files: arch/mips/Makefile, arch/mips/kernel/syscalls/Makefile

  Rationale: Ignore CONFIG_SHELL. Default variable in Kbuild build system.


Each patch in this series can be discussed and applied individually. They are
sent in one patch series, as they all orginate from the investigation guided
by the same tool and hence share similar topics and resolutions.

Please pick this series of minor clean-up patches on ./arch/mips/. It applies
cleanly on next-20211210. The patches with Fixes: tags might be considerable
for stable.


Best regards,

Lukas


Lukas Bulwahn (9):
  mips: drop selecting the non-existing config SYS_HAS_EARLY_PRINTK_8250
  mips: add SYS_HAS_CPU_MIPS64_R5 config for MIPS Release 5 support
  mips: drop selecting non-existing config NR_CPUS_DEFAULT_2
  mips: dec: provide the correctly capitalized config CPU_R4X00 in init
    error message
  mips: kgdb: adjust the comment to the actual ifdef condition
  mips: remove obsolete selection of CPU_HAS_LOAD_STORE_LR
  mips: alchemy: remove historic comment on gpio build constraints
  mips: txx9: remove left-over for removed TXX9_ACLC configs
  mips: fix Kconfig reference to PHYS_ADDR_T_64BIT

 arch/mips/Kconfig                  |  9 +++++----
 arch/mips/alchemy/common/gpiolib.c |  2 --
 arch/mips/dec/prom/init.c          |  2 +-
 arch/mips/include/asm/kgdb.h       |  2 +-
 arch/mips/txx9/Kconfig             |  3 ---
 arch/mips/txx9/generic/setup.c     | 28 ----------------------------
 6 files changed, 7 insertions(+), 39 deletions(-)

-- 
2.26.2


             reply	other threads:[~2021-12-13 11:16 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-13 11:16 Lukas Bulwahn [this message]
2021-12-13 11:16 ` [PATCH 1/9] mips: drop selecting the non-existing config SYS_HAS_EARLY_PRINTK_8250 Lukas Bulwahn
2021-12-13 18:52   ` Sander Vanheule
2021-12-14 12:25     ` Lukas Bulwahn
2021-12-15 20:17       ` Sander Vanheule
2021-12-13 11:16 ` [PATCH 2/9] mips: add SYS_HAS_CPU_MIPS64_R5 config for MIPS Release 5 support Lukas Bulwahn
2021-12-13 11:16 ` [PATCH 3/9] mips: drop selecting non-existing config NR_CPUS_DEFAULT_2 Lukas Bulwahn
2021-12-13 11:16 ` [PATCH 4/9] mips: dec: provide the correctly capitalized config CPU_R4X00 in init error message Lukas Bulwahn
2021-12-13 11:16 ` [PATCH 5/9] mips: kgdb: adjust the comment to the actual ifdef condition Lukas Bulwahn
2021-12-13 11:16 ` [PATCH 6/9] mips: remove obsolete selection of CPU_HAS_LOAD_STORE_LR Lukas Bulwahn
2021-12-13 11:16 ` [PATCH 7/9] mips: alchemy: remove historic comment on gpio build constraints Lukas Bulwahn
2021-12-13 11:16 ` [PATCH 8/9] mips: txx9: remove left-over for removed TXX9_ACLC configs Lukas Bulwahn
2021-12-13 11:16 ` [PATCH 9/9] mips: fix Kconfig reference to PHYS_ADDR_T_64BIT Lukas Bulwahn
2021-12-16 15:03 ` [PATCH 0/9] Kconfig symbol clean-up on ./arch/mips/ Thomas Bogendoerfer

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=20211213111642.11317-1-lukas.bulwahn@gmail.com \
    --to=lukas.bulwahn@gmail.com \
    --cc=Alexey.Malahov@baikalelectronics.ru \
    --cc=Sergey.Semin@baikalelectronics.ru \
    --cc=bert@biot.com \
    --cc=dansilsby@gmail.com \
    --cc=kernel-janitors@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@vger.kernel.org \
    --cc=macro@orcam.me.uk \
    --cc=paul@crapouillou.net \
    --cc=paulburton@kernel.org \
    --cc=sander@svanheule.net \
    --cc=tsbogend@alpha.franken.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;
as well as URLs for NNTP newsgroup(s).