From: William Breathitt Gray <vilhelm.gray@gmail.com>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: "Maciej S. Szmigiero" <mail@maciej.szmigiero.name>,
Guenter Roeck <linux@roeck-us.net>,
"open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>,
linux-iio@vger.kernel.org,
LINUXWATCHDOG <linux-watchdog@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 5/5] pc104: Add EXPERT dependency for PC104 Kconfig option
Date: Thu, 22 Feb 2018 15:45:09 -0500 [thread overview]
Message-ID: <20180222204509.GB10834@sophia> (raw)
In-Reply-To: <CACRpkdbRkyzb9aML1-TEENWfpSDtjayVB_eEm9V1JKrAfaOMVA@mail.gmail.com>
On Thu, Feb 22, 2018 at 04:20:46PM +0100, Linus Walleij wrote:
>On Fri, Dec 29, 2017 at 9:14 PM, William Breathitt Gray
><vilhelm.gray@gmail.com> wrote:
>
>> PC/104 device driver Kconfig options previously had an implicit EXPERT
>> dependency by way of an explicit ISA_BUS_API dependency. Now that these
>> driver Kconfig options select ISA_BUS_API rather than depend on it, the
>> PC104 Kconfig option should have an explicit EXPERT dependency.
>>
>> The PC/104 form factor and bus architecture are common in embedded
>> and specialized systems, but uncommon in typical desktop setups. For
>> this reason, it is best to mask these devices and configurations via the
>> EXPERT Kconfig option because the majority of users will never need to
>> concern themselves with PC/104.
>>
>> Signed-off-by: William Breathitt Gray <vilhelm.gray@gmail.com>
>
>Patch applied to the GPIO tree for v4.17.
>
>I don't know what kind of definition of "expert" really applies here.
>
>Who in their right mind think that novices are using the kernel
>configuration interface. Isn't anyone doing kernel compiles
>entitled to be called an expert...
>
>Anyways I guess this has already been discussed.
>
>Yours,
>Linus Walleij
I'm inclined to agree with you on this point. I believe there is benefit
in masking uncommon hardware configurations such as PC/104 in order to
reduce clutter in the menus, but you are right that "expert" sounds
somewhat of a misnomer for this use case. I considered adding yet
another Kconfig option to mask these options, but since Kconfig code
isn't my forte, I ultimately decided to keep it simple and depend simply
on EXPERT.
The PC104 Kconfig option has been implicitly masked by EXPERT up to this
point, so I thought it best to keep it masked as it has been thus far.
However, since all this masking is essentially just an aesthetic matter,
I'm somewhat indifferent and open to removing the EXPERT dependency
as well; I'll let someone else make that call rather I.
William Breathitt Gray
next prev parent reply other threads:[~2018-02-22 20:45 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-29 20:13 [PATCH v2 0/5] Change ISA_BUS_API dependency to selection William Breathitt Gray
2017-12-29 20:13 ` [PATCH v2 1/5] iio: " William Breathitt Gray
2018-01-01 9:55 ` Jonathan Cameron
2018-02-22 15:14 ` Linus Walleij
2017-12-29 20:13 ` [PATCH v2 2/5] watchdog: " William Breathitt Gray
2018-02-22 15:15 ` Linus Walleij
2017-12-29 20:13 ` [PATCH v2 3/5] gpio: " William Breathitt Gray
2018-02-22 15:16 ` Linus Walleij
2018-02-22 20:30 ` William Breathitt Gray
2018-02-22 20:44 ` Maciej S. Szmigiero
2018-02-23 13:50 ` Maciej S. Szmigiero
2018-02-23 14:09 ` Linus Walleij
2017-12-29 20:14 ` [PATCH v2 4/5] isa: Remove ISA_BUS_API selection for ISA_BUS William Breathitt Gray
2018-02-22 15:18 ` Linus Walleij
2017-12-29 20:14 ` [PATCH v2 5/5] pc104: Add EXPERT dependency for PC104 Kconfig option William Breathitt Gray
2018-02-22 15:20 ` Linus Walleij
2018-02-22 20:45 ` William Breathitt Gray [this message]
2018-01-02 9:29 ` [PATCH v2 0/5] Change ISA_BUS_API dependency to selection Linus Walleij
2018-01-31 2:22 ` William Breathitt Gray
2018-02-19 14:47 ` Maciej S. Szmigiero
2018-02-22 15:13 ` Linus Walleij
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=20180222204509.GB10834@sophia \
--to=vilhelm.gray@gmail.com \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-watchdog@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=mail@maciej.szmigiero.name \
/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