Re: [PATCH] wl1251: add support for PG11 chips.

[Kalle Valo <kalle.valo@iki.fi>]

John Willis <John.Willis@Distant-earth.com> writes:

> From: David-John Willis <John.Willis@Distant-earth.com>
>
> This simple patch adds support for the PG11 variant of the WL1251 chip as
> used on the OpenPandora OMAP3 device.
>
> Signed-off-by: David-John Willis <John.Willis@Distant-earth.com>

Thanks for the patch. And excellent news that it's working on your
device.

Minor comment:

> --- a/drivers/net/wireless/wl12xx/wl1251_main.c
> +++ b/drivers/net/wireless/wl12xx/wl1251_main.c
> @@ -185,6 +185,9 @@ static int wl1251_chip_wakeup(struct wl1251 *wl)
>  		break;
>  	case CHIP_ID_1251_PG10:
>  	case CHIP_ID_1251_PG11:
> +		wl1251_debug(DEBUG_BOOT, "chip id 0x%x (1251 PG11)",
> +			     wl->chip_id);
> +		break;
>  	default:
>  		wl1251_error("unsupported chip id: 0x%x", wl->chip_id);
>  		ret = -ENODEV;

This patch also enables PG10 support, but as that hardware revision is
untested I think it's better not to enable it. I'll edit the patch a bit
regarding this and then send it to John.

-- 
Kalle Valo