On 29 January 2014 14:14, Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> wrote: > Marek Puzyniak <marek.puzyniak@xxxxxxxxx> writes: > >> ARP frames exchange does not work properly for UAPSD enabled AP. >> ARP requests which arrives with access category 0 are processed >> by network stack and send back with access category 0. FW changes >> access category to 6. This is causing problems when UAPSD associated >> STA is sleeping after has sent ARP request. Configure ARP access >> category in FW to best effort (0) solves this problem. ARP frames >> will be send with access category 0. >> >> Simplify arp ac override functionality by removing redundant entry in >> pdev param maping table. There should be only one entry in pdev param >> map but enum has different name for different FW. >> >> Signed-off-by: Marek Puzyniak <marek.puzyniak@xxxxxxxxx> > > I had a conflict when I applied this to ath-next-test. Please check that > I didn't break anything: > > https://github.com/kvalo/ath/commit/2e006642b3aad6a5fb91f9e560a2bbcdf74aa049 It looks ok. > >> + ret = ath10k_wmi_pdev_set_param(ar, >> + ar->wmi.pdev_param->arp_ac_override, 0); >> + if (ret) { >> + ath10k_warn("could not init WMI_PDEV_PARAM_ARP_AC_OVERRIDE (%d)\n", >> + ret); > > I also modified this warning message to follow the style used elsewhere > in the driver. > > -- > Kalle Valo -- 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