On 01/13/2012 01:51 PM, Kalle Valo wrote: >> +static bool ath6kl_process_uapsdq(struct ath6kl_sta *conn, >> > + struct ath6kl_vif *vif, >> > + struct sk_buff *skb, >> > + u32 *flags) >> > +{ >> > + struct ath6kl *ar = vif->ar; >> > + bool is_apsdq_empty = false; >> > + struct ethhdr *datap = (struct ethhdr *) skb->data; >> > + u8 up = 0; > > This is initilised in an else branch I added below. This actually creates a compiler warning which I missed. I'll send a separate patch which reverts up initilisation back to the original. Kalle -- 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