The Linux Kernel Mailing List
 help / color / mirror / Atom feed
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


  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