Re: Fwd: Using select in boolean dependents of a tristate symbol

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Thu, 13 Jul 2006, Dmitry Torokhov wrote:

> On Thursday 13 July 2006 18:58, Roman Zippel wrote:
> > Hi,
> > 
> > On Thu, 13 Jul 2006, Dmitry Torokhov wrote:
> > 
> > > config THRUSTMASTER_FF
> > >       bool "ThrustMaster FireStorm Dual Power 2 support (EXPERIMENTAL)"
> > >       depends on HID_FF && EXPERIMENTAL
> > > +       select INPUT_FF_MEMLESS
> > >       help
> > >         Say Y here if you have a THRUSTMASTER FireStore Dual Power 2,
> > >         and want to enable force feedback support for it.
> > > 
> > > Unfortunately this forces INPUT_FF_MEMLESS to always be built-in,
> > > although if HID is a module it could be a module as well. Do you have
> > > any suggestions as to how allow INPUT_FF_MEMLESS to be compiled as a
> > > module?
> > 
> > You need to directly include HID into the dependencies, only the direct 
> > dependencies for config entry are used for the select.
> >
> 
> Oh, this indeed works, thanks a lot! And I was thinking I would need to
> implement something like "select <expr> as <expr>" in kconfig ;)  

What you could do is to use "select INPUT_FF_MEMLESS if HID" to make it 
visible that this dependency is actually for select.
This point is a little subtle and I'm not completely happy with it, maybe 
I'm going to split this into two select variations - one which includes 
all the dependencies and one which only uses the config symbol to select.

bye, Roman
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux