linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Lukas Bulwahn <lukas.bulwahn@gmail.com>
To: Johannes Berg <johannes.berg@intel.com>,
	Gregory Greenman <gregory.greenman@intel.com>,
	 linux-wireless <linux-wireless@vger.kernel.org>
Cc: Kalle Valo <kvalo@kernel.org>,
	kernel-janitors <kernel-janitors@vger.kernel.org>,
	 Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Question about reference to config KERNEL_6_2
Date: Wed, 17 Jan 2024 09:37:31 +0100	[thread overview]
Message-ID: <CAKXUXMyfrM6amOR7Ysim3WNQ-Ckf9HJDqRhAoYmLXujo1UV+yA@mail.gmail.com> (raw)

Dear Johannes, dear Gregory,

I am a kernel janitor checking for issues in kernel build config
options in the repository. Amongst others, I check for references to
config options that have not been defined anywhere in the tree using
an existing script in the repository,
./scripts/checkkconfigsymbols.py.

At the beginning of this year, I checked which issues were introduced
in 2023 and were not yet resolved by others or my janitor work
throughout the year. In other words, looking at some issues that
simply slipped through last year.

Commit ffbd0c8c1e7f ("wifi: mac80211: add an element parsing unit
test") and commit 730eeb17bbdd ("wifi: cfg80211: add first kunit
tests, for element defrag") add new configs that depend on
!KERNEL_6_2, but the config option KERNEL_6_2 does not exist in the
tree.

Also, Kalle Valo [1] asked during patch review about that, but did not
get any response on the mailing list.

So, what is the KERNEL_6_2 option all about? Is this something that
was just used locally for early development?

Can we just delete this dependency in these two config options?

If so, I will gladly send a patch for that.

[1] https://lore.kernel.org/all/877cp6q7ws.fsf@kernel.org/


Best regards,

Lukas

             reply	other threads:[~2024-01-17  8:37 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-17  8:37 Lukas Bulwahn [this message]
2024-01-17  8:47 ` Question about reference to config KERNEL_6_2 Johannes Berg
2024-01-17  8:55   ` Kalle Valo
2024-01-18  8:28   ` Lukas Bulwahn

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=CAKXUXMyfrM6amOR7Ysim3WNQ-Ckf9HJDqRhAoYmLXujo1UV+yA@mail.gmail.com \
    --to=lukas.bulwahn@gmail.com \
    --cc=gregory.greenman@intel.com \
    --cc=johannes.berg@intel.com \
    --cc=kernel-janitors@vger.kernel.org \
    --cc=kvalo@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    /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).