The option to support the old style PSK interface in the PS3
GELIC wireless drivers requires CONFIG_WEXT_PRIV to be set
Signed-off-by: Benjamin Herrenschmidt <[email protected]>
---
Please send to Linus asap (or I can put it in powerpc.git) as it's
breaking one of my test build configs :-)
diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
index e58a653..c0ecc77 100644
--- a/drivers/net/Kconfig
+++ b/drivers/net/Kconfig
@@ -2358,6 +2358,7 @@ config GELIC_WIRELESS
config GELIC_WIRELESS_OLD_PSK_INTERFACE
bool "PS3 Wireless private PSK interface (OBSOLETE)"
depends on GELIC_WIRELESS
+ select WEXT_PRIV
help
This option retains the obsolete private interface to pass
the PSK from user space programs to the driver. The PSK
On Fri, 18 Dec 2009 16:52:54 +1100 Benjamin Herrenschmidt wrote:
> The option to support the old style PSK interface in the PS3
> GELIC wireless drivers requires CONFIG_WEXT_PRIV to be set
>
> Signed-off-by: Benjamin Herrenschmidt <[email protected]>
> ---
>
> Please send to Linus asap (or I can put it in powerpc.git) as it's
> breaking one of my test build configs :-)
>
> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
> index e58a653..c0ecc77 100644
> --- a/drivers/net/Kconfig
> +++ b/drivers/net/Kconfig
> @@ -2358,6 +2358,7 @@ config GELIC_WIRELESS
> config GELIC_WIRELESS_OLD_PSK_INTERFACE
> bool "PS3 Wireless private PSK interface (OBSOLETE)"
> depends on GELIC_WIRELESS
> + select WEXT_PRIV
> help
> This option retains the obsolete private interface to pass
> the PSK from user space programs to the driver. The PSK
Probably also needs
depends on WLAN
to prevent build failures.
---
~Randy
From: Benjamin Herrenschmidt <[email protected]>
The option to support the old style PSK interface in the PS3
GELIC wireless drivers requires CONFIG_WEXT_PRIV to be set
Signed-off-by: Benjamin Herrenschmidt <[email protected]>
Signed-off-by: Geoff Levand <[email protected]>
---
v2: Added 'depends on WLAN'.
Please send to Linus asap (or I can put it in powerpc.git) as it's
breaking one of my test build configs :-)
Just as a note, I asked Hamish to work to remove this option.
No distros use it anymore, so we don't need it.
-Geoff
---
drivers/net/Kconfig | 2 ++
1 file changed, 2 insertions(+)
--- a/drivers/net/Kconfig
+++ b/drivers/net/Kconfig
@@ -2357,7 +2357,9 @@ config GELIC_WIRELESS
config GELIC_WIRELESS_OLD_PSK_INTERFACE
bool "PS3 Wireless private PSK interface (OBSOLETE)"
+ depends on WLAN
depends on GELIC_WIRELESS
+ select WEXT_PRIV
help
This option retains the obsolete private interface to pass
the PSK from user space programs to the driver. The PSK
On Fri, 2009-12-18 at 10:24 -0800, Geoff Levand wrote:
> From: Benjamin Herrenschmidt <[email protected]>
>
> The option to support the old style PSK interface in the PS3
> GELIC wireless drivers requires CONFIG_WEXT_PRIV to be set
>
> Signed-off-by: Benjamin Herrenschmidt <[email protected]>
> Signed-off-by: Geoff Levand <[email protected]>
> ---
> v2: Added 'depends on WLAN'.
>
> Please send to Linus asap (or I can put it in powerpc.git) as it's
> breaking one of my test build configs :-)
>
> Just as a note, I asked Hamish to work to remove this option.
> No distros use it anymore, so we don't need it.
Geoff, the depends on WLAN should be on config GELIC_WIRELESS
Cheers,
Ben.
> -Geoff
> ---
> drivers/net/Kconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> --- a/drivers/net/Kconfig
> +++ b/drivers/net/Kconfig
> @@ -2357,7 +2357,9 @@ config GELIC_WIRELESS
>
> config GELIC_WIRELESS_OLD_PSK_INTERFACE
> bool "PS3 Wireless private PSK interface (OBSOLETE)"
> + depends on WLAN
> depends on GELIC_WIRELESS
> + select WEXT_PRIV
> help
> This option retains the obsolete private interface to pass
> the PSK from user space programs to the driver. The PSK
On Fri, 2009-12-18 at 10:24 -0800, Geoff Levand wrote:
> From: Benjamin Herrenschmidt <[email protected]>
>
> The option to support the old style PSK interface in the PS3
> GELIC wireless drivers requires CONFIG_WEXT_PRIV to be set
>
> Signed-off-by: Benjamin Herrenschmidt <[email protected]>
> Signed-off-by: Geoff Levand <[email protected]>
> ---
> v2: Added 'depends on WLAN'.
Won't you want 'depends on WLAN' for GELIC_WIRELESS instead of here?
johannes
From: Benjamin Herrenschmidt <[email protected]>
The option to support the old style PSK interface in the PS3
GELIC wireless drivers requires CONFIG_WEXT_PRIV to be set
Signed-off-by: Benjamin Herrenschmidt <[email protected]>
Signed-off-by: Geoff Levand <[email protected]>
---
v2: Added 'depends on WLAN'.
v3: Moved 'depends on WLAN' to GELIC_WIRELESS.
Please send to Linus asap (or I can put it in powerpc.git) as it's
breaking one of my test build configs :-)
---
drivers/net/Kconfig | 2 ++
1 file changed, 2 insertions(+)
--- a/drivers/net/Kconfig
+++ b/drivers/net/Kconfig
@@ -2346,6 +2346,7 @@ config GELIC_NET
config GELIC_WIRELESS
bool "PS3 Wireless support"
+ depends on WLAN
depends on GELIC_NET
select WIRELESS_EXT
help
@@ -2358,6 +2359,7 @@ config GELIC_WIRELESS
config GELIC_WIRELESS_OLD_PSK_INTERFACE
bool "PS3 Wireless private PSK interface (OBSOLETE)"
depends on GELIC_WIRELESS
+ select WEXT_PRIV
help
This option retains the obsolete private interface to pass
the PSK from user space programs to the driver. The PSK
On 12/18/2009 10:30 AM, Johannes Berg wrote:
> On Fri, 2009-12-18 at 10:24 -0800, Geoff Levand wrote:
>> From: Benjamin Herrenschmidt <[email protected]>
>>
>> The option to support the old style PSK interface in the PS3
>> GELIC wireless drivers requires CONFIG_WEXT_PRIV to be set
>>
>> Signed-off-by: Benjamin Herrenschmidt <[email protected]>
>> Signed-off-by: Geoff Levand <[email protected]>
>> ---
>> v2: Added 'depends on WLAN'.
>
> Won't you want 'depends on WLAN' for GELIC_WIRELESS instead of here?
Yes, that seems correct. I'll post an update v3.
-Geoff
On Fri, 2009-12-18 at 08:55 -0800, Randy Dunlap wrote:
>
> Probably also needs
> depends on WLAN
> to prevent build failures.
Yeah, GELIC_WIRELESS should probably. I'll do a separate patch next week
but feel free to beat me to it.
Cheers,
Ben.