On 01/09/2012 09:45 AM, Thirumalai wrote: > + ath6kl_wmi_set_pvb_cmd(ar->wmi, vif->fw_vif_idx, conn->aid, 0); > + if (is_apsdq_empty_at_start) { > + ath6kl_wmi_set_apsd_bfrd_traf(ar->wmi, > + vif->fw_vif_idx, > + conn->aid, 0, > + WMI_AP_APSD_NO_DELIVERY_FRAMES); > + } else { > + ath6kl_wmi_set_apsd_bfrd_traf(ar->wmi, > + vif->fw_vif_idx, > + conn->aid, 0, 0); > + } To have only one call to ath6kl_wmi_set_apsd_bfrd_traf() you could add a variable like this: u32 flags; .... if (is_apsdq_empty_at_start) flags = WMI_AP_APSD_NO_DELIVERY_FRAMES; else flags = 0; ath6kl_wmi_set_pvb_cmd(ar->wmi, vif->fw_vif_idx, conn->aid, 0); ath6kl_wmi_set_apsd_bfrd_traf(ar->wmi, vif->fw_vif_idx, conn->aid, 0, flags); 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