On Tue, 2011-08-09 at 12:13 +0300, Eliad Peller wrote: > @@ -300,23 +300,18 @@ static void wl1271_tx_fill_hdr(struct wl1271 *wl, struct sk_buff *skb, > basic rates */ > if (control->control.sta) > rate_idx = ACX_TX_AP_FULL_RATE; > else > rate_idx = ACX_TX_BASIC_RATE; > } else { > - switch (hlid) { > - case WL1271_AP_GLOBAL_HLID: > + if (hlid == wl->ap_global_hlid) > rate_idx = ACX_TX_AP_MODE_MGMT_RATE; > - break; > - case WL1271_AP_BROADCAST_HLID: > + else if (hlid == wl->ap_bcast_hlid) Extra whitespace after ==. -- Cheers, Luca. -- 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