Skip to content

Commit

Permalink
mac80211: remove misleading 'res' variable
Browse files Browse the repository at this point in the history
When this function returns != CONTINUE, it needs to put the
station struct it has acquired. Hence, having this unused
variable is not just superfluous but also misleading.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Johannes Berg authored and David S. Miller committed Jan 28, 2008
1 parent 1036d86 commit 6d65f5d
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions net/mac80211/tx.c
Original file line number Diff line number Diff line change
Expand Up @@ -932,7 +932,6 @@ __ieee80211_tx_prepare(struct ieee80211_txrx_data *tx,
struct ieee80211_local *local = wdev_priv(dev->ieee80211_ptr);
struct ieee80211_hdr *hdr;
struct ieee80211_sub_if_data *sdata;
ieee80211_txrx_result res = TXRX_CONTINUE;

int hdrlen;

Expand Down Expand Up @@ -997,7 +996,7 @@ __ieee80211_tx_prepare(struct ieee80211_txrx_data *tx,
}
control->flags |= IEEE80211_TXCTL_FIRST_FRAGMENT;

return res;
return TXRX_CONTINUE;
}

/* Device in tx->dev has a reference added; use dev_put(tx->dev) when
Expand Down

0 comments on commit 6d65f5d

Please sign in to comment.