On Tue, Feb 12, 2013 at 9:56 AM, Marco Porsch <marco@xxxxxxxxxxx> wrote: > On 02/11/2013 10:07 PM, Thomas Pedersen wrote: >> + if (ifmsh->nonpeer_pm != nonpeer_pm || >> + ifmsh->ps_peers_light_sleep != light_sleep_cnt || >> + ifmsh->ps_peers_deep_sleep != deep_sleep_cnt) >> + changed = BSS_CHANGED_BEACON; > > > Here it only affects the beacon if the number of light/deep sleep peers > changed from/to zero or non-zero. > The following should avoid some unnecessary updates: > > !ifmsh->ps_peers_deep_sleep != !deep_sleep_cnt || > !ifmsh->ps_peers_deep_sleep != !deep_sleep_cnt) Ah nice. I'll add this to the v2, not checking deep_sleep_cnt twice of course :) -- Thomas -- 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
