Dan,
I tried your patch, it works on the XO-1.5.
Thank you.
- Steve
On Wed, Apr 28, 2010 at 12:37 PM, Dan Williams <dcbw@xxxxxxxxxx> wrote:
> The 'ready' condition was incorrectly evaluated which sometimes lead to
> failures loading the second-stage firmware on 8686 devices.
>
> Signed-off-by: Dan Williams <dcbw@xxxxxxxxxx>
>
> ---
> diff --git a/drivers/net/wireless/libertas/if_sdio.c b/drivers/net/wireless/libertas/if_sdio.c
> index 13dfeda..64dd345 100644
> --- a/drivers/net/wireless/libertas/if_sdio.c
> +++ b/drivers/net/wireless/libertas/if_sdio.c
> @@ -324,7 +324,9 @@ static int if_sdio_wait_status(struct if_sdio_card *card, const u8 condition)
> timeout = jiffies + HZ;
> while (1) {
> status = sdio_readb(card->func, IF_SDIO_STATUS, &ret);
> - if (ret || (status & condition))
> + if (ret)
> + return ret;
> + if ((status & condition) == condition)
> break;
> if (time_after(jiffies, timeout))
> return -ETIMEDOUT;
>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
