[PATCH] libertas: move lbs_update_channel out of assoc.c

Dan Williams dcbw at redhat.com
Wed Apr 2 23:33:28 EDT 2008


On Wed, 2008-04-02 at 16:52 +0200, Holger Schurig wrote:
> ... as it has nothing to do with pure association
> 
> Signed-off-by: Holger Schurig <hs4233 at mail.mn-solutions.de>

Acked-by: Dan Williams <dcbw at redhat.com>

> Index: wireless-testing/drivers/net/wireless/libertas/assoc.c
> ===================================================================
> --- wireless-testing.orig/drivers/net/wireless/libertas/assoc.c	2008-04-02 14:49:22.000000000 +0200
> +++ wireless-testing/drivers/net/wireless/libertas/assoc.c	2008-04-02 14:50:25.000000000 +0200
> @@ -583,23 +583,6 @@ done:
>  	return ret;
>  }
>  
> -
> -int lbs_update_channel(struct lbs_private *priv)
> -{
> -	int ret;
> -
> -	/* the channel in f/w could be out of sync; get the current channel */
> -	lbs_deb_enter(LBS_DEB_ASSOC);
> -
> -	ret = lbs_get_channel(priv);
> -	if (ret > 0) {
> -		priv->curbssparams.channel = ret;
> -		ret = 0;
> -	}
> -	lbs_deb_leave_args(LBS_DEB_ASSOC, "ret %d", ret);
> -	return ret;
> -}
> -
>  static int assoc_helper_channel(struct lbs_private *priv,
>                                  struct assoc_request * assoc_req)
>  {
> Index: wireless-testing/drivers/net/wireless/libertas/cmd.c
> ===================================================================
> --- wireless-testing.orig/drivers/net/wireless/libertas/cmd.c	2008-04-02 14:50:03.000000000 +0200
> +++ wireless-testing/drivers/net/wireless/libertas/cmd.c	2008-04-02 14:50:22.000000000 +0200
> @@ -826,6 +826,22 @@ out:
>  	return ret;
>  }
>  
> +int lbs_update_channel(struct lbs_private *priv)
> +{
> +	int ret;
> +
> +	/* the channel in f/w could be out of sync; get the current channel */
> +	lbs_deb_enter(LBS_DEB_ASSOC);
> +
> +	ret = lbs_get_channel(priv);
> +	if (ret > 0) {
> +		priv->curbssparams.channel = ret;
> +		ret = 0;
> +	}
> +	lbs_deb_leave_args(LBS_DEB_ASSOC, "ret %d", ret);
> +	return ret;
> +}
> +
>  /**
>   *  @brief Set the radio channel
>   *




More information about the libertas-dev mailing list