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

Reply via email to