On Sunday 16 August 2009 20:22:41 Gábor Stefanik wrote:
> Larry has reported success getting scan data with an LP-PHY device,
> so it's probably time to release LP-PHY support for testing.
> 
> Also disable 802.11a support for now, as 802.11a currently causes
> the driver to panic on startup (NULL pointer dereference).
> 
> Signed-off-by: Gábor Stefanik <[email protected]>

ack

> ---
> V2:
> -Remove "VERY" from the warning in the symbol name.
> -Remove list of known devices.
> -Don't create a separate symbol for 5GHz, just disable it in the code.
> 
>  drivers/net/wireless/b43/Kconfig |   14 +++++++-------
>  drivers/net/wireless/b43/main.c  |    2 ++
>  2 files changed, 9 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/net/wireless/b43/Kconfig 
> b/drivers/net/wireless/b43/Kconfig
> index 67f564e..237b1aa 100644
> --- a/drivers/net/wireless/b43/Kconfig
> +++ b/drivers/net/wireless/b43/Kconfig
> @@ -80,16 +80,16 @@ config B43_NPHY
>         SAY N.
>  
>  config B43_PHY_LP
> -     bool "IEEE 802.11g LP-PHY support (BROKEN)"
> -     depends on B43 && EXPERIMENTAL && BROKEN
> +     bool "Support for low-power (LP-PHY) devices (EXPERIMENTAL)"
> +     depends on B43 && EXPERIMENTAL
>       ---help---
>         Support for the LP-PHY.
> -       The LP-PHY is an IEEE 802.11g based PHY built into some notebooks
> -       and embedded devices.
> -
> -       THIS IS BROKEN AND DOES NOT WORK YET.
> +       The LP-PHY is a low-power PHY built into some notebooks
> +       and embedded devices. It supports 802.11a/g
> +       (802.11a support is optional, and currently disabled).
>  
> -       SAY N.
> +       This is heavily experimental, and probably will not work for you.
> +       Say N unless you want to help debug the driver.
>  
>  # This config option automatically enables b43 LEDS support,
>  # if it's possible.
> diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
> index 99b41ce..c5bece0 100644
> --- a/drivers/net/wireless/b43/main.c
> +++ b/drivers/net/wireless/b43/main.c
> @@ -4514,7 +4514,9 @@ static int b43_wireless_core_attach(struct b43_wldev 
> *dev)
>                       have_5ghz_phy = 1;
>                       break;
>               case B43_PHYTYPE_LP: //FIXME not always!
> +#if 0 //FIXME enabling 5GHz causes a NULL pointer dereference
>                       have_5ghz_phy = 1;
> +#endif
>               case B43_PHYTYPE_G:
>               case B43_PHYTYPE_N:
>                       have_2ghz_phy = 1;



-- 
Greetings, Michael.
_______________________________________________
Bcm43xx-dev mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev

Reply via email to