From: Jean Delvare <jdelvare@suse.de>
To: Jiri Kosina <jkosina@suse.cz>, Michal Marek <mmarek@suse.cz>
Cc: linux-input@vger.kernel.org, linux-kbuild <linux-kbuild@vger.kernel.org>
Subject: Dependencies in the HID subsystem
Date: Wed, 29 May 2013 21:19:34 +0200 [thread overview]
Message-ID: <1369855174.4306.198.camel@chaos.site> (raw)
Hi all,
I am worried and confused by some Kconfig dependencies in the HID
subsystem.
There are 11 HID device drivers which are defined in drivers/hid/Kconfig
with:
tristate "..." if EXPERT
default !EXPERT
Unless EXPERT is enabled (and that's not the default), these driver
entries are hidden and automatically selected. If CONFIG_HID=m, they are
selected as modules. If CONFIG_HID=y, they are built into the kernel. So
it is impossible to have CONFIG_HID=y and build these device drivers as
modules - as device drivers typically are.
I would like to understand the reasoning behind this complexity. What is
so special about these 11 drivers, that we can't just let the (kernel
configuring) user chose if he/she wants them and in what form?
Wouldn't "default !EXPERT" and a good old "If unsure, say Y" in the help
text be enough?
I would be fine with "tristate ... if EXPERT" if I still had the choice
between built-in or modular when both are possible. Is there any chance
to change the meaning of this construct to that?
Also I find it unpleasant that this construct completely hides the
option from the user, as if it did not exist, except if other options
depend on it. This is inconsistent and makes it difficult for the user
to know whether a specific kernel version includes a given driver or not
(one has to check .config afterward to know the answer.)
Put in short, I don't like the way things are today and would welcome
changes in this area.
Thanks,
--
Jean Delvare
Suse L3
next reply other threads:[~2013-05-29 19:19 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-29 19:19 Jean Delvare [this message]
2013-05-29 20:52 ` Dependencies in the HID subsystem Jiri Kosina
2013-05-29 22:23 ` David Herrmann
2013-06-13 6:58 ` Jean Delvare
2013-06-13 7:11 ` Jean Delvare
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=1369855174.4306.198.camel@chaos.site \
--to=jdelvare@suse.de \
--cc=jkosina@suse.cz \
--cc=linux-input@vger.kernel.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=mmarek@suse.cz \
/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).