Hi Dan, > 1239 ret = iwl_mvm_remove_inactive_tids(mvm, mvmsta, i, > 1240 inactive_tid_bitmap, > 1241 &unshare_queues, > 1242 &changetid_queues); > 1243 if (ret >= 0 && free_queue < 0) > ^^^^^^^^ > > The iwl_mvm_remove_inactive_tids() returns a bool so it doesn't make > sense to test for >= 0. Probably we should test for ret == true? Huh, thanks for the report, we'll check. johannes