From: Adrian Bunk <bunk@fs.tum.de>
To: Roman Zippel <zippel@linux-m68k.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: 2.6: Problem multiple bool/tristate prompts
Date: Fri, 8 Aug 2003 12:37:18 +0200 [thread overview]
Message-ID: <20030808103717.GP16091@fs.tum.de> (raw)
In-Reply-To: <Pine.LNX.4.44.0308081135520.714-100000@serv>
On Fri, Aug 08, 2003 at 12:03:45PM +0200, Roman Zippel wrote:
> Hi,
>
> On Fri, 8 Aug 2003, Adrian Bunk wrote:
>
> > config BLK_DEV_PS2
> > tristate "PS/2 ESDI hard disk support" if BROKEN_MODULAR
> > bool "PS/2 ESDI hard disk support" if !BROKEN_MODULAR
> >
> >
> > Every "make *config" gives the warning
> >
> > drivers/block/Kconfig:45: prompt redefined
> > drivers/block/Kconfig:45:warning: type of 'BLK_DEV_PS2' redefined from
> > 'tristate' to 'boolean'
> >
> > and the symbol is handled as tristate although BROKEN_MODULAR isn't
> > defined.
>
> A symbol can have only a single type and the warning is a bit misleading,
> the new type definition is simply ignored.
> I'm not sure what you're trying makes really sense, but you have to use a
I made a patch that lets all broken drivers depend on an (undefined)
BROKEN symbol and all drivers that don't compile on SMP on a
BROKEN_ON_SMP symbol that is only defined if !SMP.
This (undefined) BROKEN_MODULAR was an attempt to express that a driver
compiles only statically.
> separate symbol:
>
> config BLK_DEV_PS2_B
> bool "PS/2 ESDI hard disk support" if !BROKEN_MODULAR
>
> config BLK_DEV_PS2
> tristate "PS/2 ESDI hard disk support" if BROKEN_MODULAR
> default BLK_DEV_PS2_B
It's too complicated to be actually useful, but it seems I'd then need a
config BLK_DEV_PS2_TRISTATE
tristate "PS/2 ESDI hard disk support"
depends on BROKEN_MODULAR
default y if BLK_DEV_PS2=y
default m if BLK_DEV_PS2=m
config BLK_DEV_PS2_BOOL
bool "PS/2 ESDI hard disk support"
depends on !BROKEN_MODULAR
default y if BLK_DEV_PS2=y
config BLK_DEV_PS2
default y if BLK_DEV_PS2_TRISTATE=y || BLK_DEV_PS2_BOOL
default m if BLK_DEV_PS2_TRISTATE=m
Alternatively it might work with BLK_DEV_PS2_TRISTATE and
BLK_DEV_PS2_BOOL using select.
My problem isn't that important to satisfy such a complicated construct,
I can accept that there's no easy way to express this and live without
it.
> bye, Roman
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
next prev parent reply other threads:[~2003-08-08 10:37 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-08-07 23:59 2.6: Problem multiple bool/tristate prompts Adrian Bunk
2003-08-08 10:03 ` Roman Zippel
2003-08-08 10:37 ` Adrian Bunk [this message]
2003-08-08 13:26 ` Roman Zippel
-- strict thread matches above, loose matches on Subject: below --
2003-08-08 14:09 John Bradford
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=20030808103717.GP16091@fs.tum.de \
--to=bunk@fs.tum.de \
--cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox