On Thu, Jun 24, 2010 at 03:12:45PM +0530, Vasanthakumar Thiagarajan wrote: > Signed-off-by: Vasanthakumar Thiagarajan <vasanth@xxxxxxxxxxx> > --- > drivers/net/wireless/ath/ath9k/main.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c > index 6d57877..484d113 100644 > --- a/drivers/net/wireless/ath/ath9k/main.c > +++ b/drivers/net/wireless/ath/ath9k/main.c > @@ -271,7 +271,7 @@ void ath_paprd_calibrate(struct work_struct *work) > ath9k_ps_wakeup(sc); > skb = alloc_skb(len, GFP_KERNEL); > if (!skb) > - return; > + goto fail_paprd; > > tx_info = IEEE80211_SKB_CB(skb); Please drop this particular patch, ath9k_ps_wakeup() does not need to be there before alloc_skb() in the first place. I'll send a cleaner one. Vasanth -- 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