Skip to content

Commit

Permalink
libertas: kill useless #define LBS_MONITOR_OFF 0
Browse files Browse the repository at this point in the history
Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Acked-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
  • Loading branch information
Holger Schurig authored and John W. Linville committed Mar 27, 2008
1 parent 69dc5d9 commit c2b310a
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 12 deletions.
12 changes: 6 additions & 6 deletions drivers/net/wireless/libertas/main.c
Original file line number Diff line number Diff line change
Expand Up @@ -277,10 +277,10 @@ static ssize_t lbs_rtap_set(struct device *dev,
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)
Expand All @@ -293,9 +293,9 @@ static ssize_t lbs_rtap_set(struct device *dev,
}

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) {
Expand Down Expand Up @@ -392,7 +392,7 @@ static int lbs_dev_open(struct net_device *dev)

spin_lock_irq(&priv->driver_lock);

if (priv->monitormode != LBS_MONITOR_OFF) {
if (priv->monitormode) {
ret = -EBUSY;
goto out;
}
Expand Down
2 changes: 1 addition & 1 deletion drivers/net/wireless/libertas/rx.c
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ int lbs_process_rxed_packet(struct lbs_private *priv, struct sk_buff *skb)

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;
Expand Down
5 changes: 2 additions & 3 deletions drivers/net/wireless/libertas/tx.c
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ int lbs_hard_start_xmit(struct sk_buff *skb, struct net_device *dev)

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);
Expand Down Expand Up @@ -186,8 +186,7 @@ void lbs_send_tx_feedback(struct lbs_private *priv)
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;
Expand Down
2 changes: 0 additions & 2 deletions drivers/net/wireless/libertas/wext.h
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ struct lbs_ioctl_regrdwr {
u32 value;
};

#define LBS_MONITOR_OFF 0

extern struct iw_handler_def lbs_handler_def;
extern struct iw_handler_def mesh_handler_def;

Expand Down

0 comments on commit c2b310a

Please sign in to comment.