From: viro@ZenIV.linux.org.uk
To: Roman Zippel <zippel@linux-m68k.org>
Cc: Linus Torvalds <torvalds@osdl.org>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Kconfig fix (BLK_DEV_FD dependencies)
Date: Tue, 6 Sep 2005 14:49:44 +0100 [thread overview]
Message-ID: <20050906134944.GV5155@ZenIV.linux.org.uk> (raw)
In-Reply-To: <Pine.LNX.4.61.0509061205510.3743@scrub.home>
On Tue, Sep 06, 2005 at 12:52:34PM +0200, Roman Zippel wrote:
> I'm not really a big fan of such dummy symbols, those whole point is to
> only enable dependencies, but are otherwise unsused.
> The basic problem is similiar to selects, that one has to grep the whole
> Kconfig to find out what modifies the dependencies of a symbol.
>
> If we really want to describe dependencies like this, I'd prefer to extend
> the Kconfig syntax for this:
>
> config FOO
> allow BAR
>
> config BAR
> option hidden
>
> The hidden option would explicitly hide the symbol, unless otherwise
> allowed.
I doubt that we really need that - there are, fortunately, very few drivers
that need either form.
Note that what happens here is very unusual:
* driver is for hardware shared by quite a few (sub)architectures
* it requires different (and irregular) glue for all of them
* it doesn't exist for more and more new architectures and never
existed for many old ones
* the same is true for subarchitectures of arm/mips/ppc - hell, even
for m68k, except that there we don't get new ones.
* it's not tied to any bus
Keeping a list of targets that must be excluded is obviously losing variant -
there will be more and more of those. Flipping to "it's allowed on <list>"
generates a big mess; yeah, you get a long expression that tells you where
is that sucker defined. And it turns into a convoluted way to say "many
platforms have that, many do not, it's really up to platform, no general
rules here". Which is exactly what
depends on ARCH_MAY_HAVE_PC_FDC
is saying in far more readable way.
We could go for your "allow" form, but what else would need it? USB gadget
stuff with its "must have at most one low-level driver, high-level drivers
should be allowed only if a low-level one is present"? RTC mess is better
solved in other ways, PARPORT_PC is mostly solved by now, what's left?
VGA_CONSOLE? I really don't see enough uses for such construct...
next prev parent reply other threads:[~2005-09-06 13:49 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-09-06 0:48 [PATCH] Kconfig fix (BLK_DEV_FD dependencies) viro
2005-09-06 10:52 ` Roman Zippel
2005-09-06 13:49 ` viro [this message]
2005-09-06 14:50 ` Geert Uytterhoeven
2005-09-06 15:05 ` Roman Zippel
2005-09-06 15:23 ` viro
2005-09-06 16:13 ` Roman Zippel
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=20050906134944.GV5155@ZenIV.linux.org.uk \
--to=viro@zeniv.linux.org.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@osdl.org \
--cc=zippel@linux-m68k.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.