[PATCH -mm] ssb: Make pcmciahost depend on PCMCIA=y

Previous thread: Re: [RFC] Union Mount: Readdir approaches by hooanon05 on Tuesday, September 11, 2007 - 10:05 pm. (1 message)

Next thread: [PATCH v2] doc: about email clients for Linux patches by Randy Dunlap on Tuesday, September 11, 2007 - 11:42 pm. (1 message)
To: Andrew Morton <akpm@...>
Cc: Michael Buesch <mb@...>, <linux-kernel@...>
Date: Tuesday, September 11, 2007 - 10:11 pm

SSB uses a bool (SSB_PCMCIAHOST_POSSIBLE) to determine whether to
build in PCMCIA support or not, as the PCMCIA host code itself is
also only a bool, make SSB_PCMCIAHOST_POSSIBLE depend on PCMCIA=y.

Without this, SSB_PCMCIAHOST_POSSIBLE evaluates to y when PCMCIA
is built as a module, which results in link errors due to the
pcmcia_access_configuration_register() accesses, where the symbol
is only defined in a module.

Signed-off-by: Paul Mundt <lethal@linux-sh.org>

--

drivers/ssb/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

--- linux-2.6.23-rc4-mm1.orig/drivers/ssb/Kconfig 2007-09-11 15:15:52.000000000 +0900
+++ linux-2.6.23-rc4-mm1/drivers/ssb/Kconfig 2007-09-12 10:51:53.000000000 +0900
@@ -37,7 +37,7 @@

config SSB_PCMCIAHOST_POSSIBLE
bool
- depends on SSB && PCMCIA && EXPERIMENTAL
+ depends on SSB && PCMCIA=y && EXPERIMENTAL
default y

config SSB_PCMCIAHOST
-

To: Paul Mundt <lethal@...>
Cc: Andrew Morton <akpm@...>, <linux-kernel@...>
Date: Wednesday, September 12, 2007 - 6:09 am

There we go. The usual SELECT dependency hell again...
Would changing SSB_PCMCIAHOST_POSSIBLE to tristate also fix it?
What would be the sideeffects?

I think this PCMCIA=y is not good, because it's perfectly
legal to build PCMCIA as module, while SSB and everything
that depends on SSB is also a module.
Would tristate fix that?

--
Greetings Michael.
-

To: Michael Buesch <mb@...>
Cc: Andrew Morton <akpm@...>, <linux-kernel@...>
Date: Wednesday, September 12, 2007 - 6:17 am

I tried that first, if you do that you have to change the default to
SSB && PCMCIA, and then anything that depends on it also has to be a
tristate. That worked ok for SSB_PCMCIAHOST, but it didn't work ok for
the b43 wireless + PCMCIA, which is why I opted for the PCMCIA=y thing
instead, which makes sure that SSB_PCMCIAHOST can't be enabled if PCMCIA
is modular.
-

To: Paul Mundt <lethal@...>
Cc: Andrew Morton <akpm@...>, <linux-kernel@...>
Date: Wednesday, September 12, 2007 - 6:59 am

Ok, so much for "SELECT is easy and it works if used correctly..." :)
Well, let's apply that patch then. It needlessly restricts the
choice to not allow modular pcmcia in that case, though.

--
Greetings Michael.
-

To: Michael Buesch <mb@...>
Cc: Andrew Morton <akpm@...>, <linux-kernel@...>
Date: Wednesday, September 12, 2007 - 9:43 pm

That is the compromise, yes. Feel free to propose a better solution ;-)
-

To: Paul Mundt <lethal@...>, Michael Buesch <mb@...>, Andrew Morton <akpm@...>, <linux-kernel@...>
Date: Thursday, September 20, 2007 - 3:42 pm

I just ran into this link error (CONFIG_PCMCIA=m;
CONFIG_SSB_PCMCIAHOST=y). No big deal; I don't have the hardware.

--
Joseph Fannin
jfannin@gmail.com

-

To: Paul Mundt <lethal@...>, Michael Buesch <mb@...>, Andrew Morton <akpm@...>, <linux-kernel@...>
Date: Thursday, September 20, 2007 - 4:33 pm

config SSB_PCMCIAHOST_POSSIBLE
bool
depends on SSB && PCMCIA && EXPERIMENTAL
default y

config SSB_PCMCIAHOST
bool "Support for SSB on PCMCIA-bus host (EXPERIMENTAL)"
depends on SSB_PCMCIAHOST_POSSIBLE
help

What tree are you using?

--
John W. Linville
linville@tuxdriver.com
-

Previous thread: Re: [RFC] Union Mount: Readdir approaches by hooanon05 on Tuesday, September 11, 2007 - 10:05 pm. (1 message)

Next thread: [PATCH v2] doc: about email clients for Linux patches by Randy Dunlap on Tuesday, September 11, 2007 - 11:42 pm. (1 message)