Search Linux Wireless

Re: [PATCH] libertas: kill useless #define LBS_MONITOR_OFF 0

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2008-03-26 at 09:57 +0100, Holger Schurig wrote:
> Signed-off-by: Holger Schurig <hs4233@xxxxxxxxxxxxxxxxxxxx>

Acked-by: Dan Williams <dcbw@xxxxxxxxxx>

> --- wireless-testing.orig/drivers/net/wireless/libertas/wext.h
> +++ wireless-testing/drivers/net/wireless/libertas/wext.h
> @@ -15,8 +15,6 @@
>  	u32 value;
>  };
>  
> -#define LBS_MONITOR_OFF			0
> -
>  extern struct iw_handler_def lbs_handler_def;
>  extern struct iw_handler_def mesh_handler_def;
>  
> --- wireless-testing.orig/drivers/net/wireless/libertas/main.c
> +++ wireless-testing/drivers/net/wireless/libertas/main.c
> @@ -277,10 +277,10 @@
>  	struct lbs_private *priv = to_net_dev(dev)->priv;
>  
>  	sscanf(buf, "%x", &monitor_mode);
> -	if (monitor_mode != LBS_MONITOR_OFF) {
> -		if(priv->monitormode == monitor_mode)
> +	if (monitor_mode) {
> +		if (priv->monitormode == monitor_mode)
>  			return strlen(buf);
> -		if (priv->monitormode == LBS_MONITOR_OFF) {
> +		if (!priv->monitormode) {
>  			if (priv->infra_open || priv->mesh_open)
>  				return -EBUSY;
>  			if (priv->mode == IW_MODE_INFRA)
> @@ -293,9 +293,9 @@
>  	}
>  
>  	else {
> -		if (priv->monitormode == LBS_MONITOR_OFF)
> +		if (!priv->monitormode)
>  			return strlen(buf);
> -		priv->monitormode = LBS_MONITOR_OFF;
> +		priv->monitormode = 0;
>  		lbs_remove_rtap(priv);
>  
>  		if (priv->currenttxskb) {
> @@ -392,7 +392,7 @@
>  
>  	spin_lock_irq(&priv->driver_lock);
>  
> -	if (priv->monitormode != LBS_MONITOR_OFF) {
> +	if (priv->monitormode) {
>  		ret = -EBUSY;
>  		goto out;
>  	}
> --- wireless-testing.orig/drivers/net/wireless/libertas/rx.c
> +++ wireless-testing/drivers/net/wireless/libertas/rx.c
> @@ -155,7 +155,7 @@
>  
>  	skb->ip_summed = CHECKSUM_NONE;
>  
> -	if (priv->monitormode != LBS_MONITOR_OFF)
> +	if (priv->monitormode)
>  		return process_rxed_802_11_packet(priv, skb);
>  
>  	p_rx_pkt = (struct rxpackethdr *) skb->data;
> --- wireless-testing.orig/drivers/net/wireless/libertas/tx.c
> +++ wireless-testing/drivers/net/wireless/libertas/tx.c
> @@ -151,7 +151,7 @@
>  
>  	dev->trans_start = jiffies;
>  
> -	if (priv->monitormode != LBS_MONITOR_OFF) {
> +	if (priv->monitormode) {
>  		/* Keep the skb to echo it back once Tx feedback is
>  		   received from FW */
>  		skb_orphan(skb);
> @@ -186,8 +186,7 @@
>  	int txfail;
>  	int try_count;
>  
> -	if (priv->monitormode == LBS_MONITOR_OFF ||
> -	    priv->currenttxskb == NULL)
> +	if (!priv->monitormode || priv->currenttxskb == NULL)
>  		return;
>  
>  	radiotap_hdr = (struct tx_radiotap_hdr *)priv->currenttxskb->data;

--
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

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux