CMD_SYNC is zero so the if (cmd->flags & CMD_SYNC) is never true and we
never check the assertion.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
         * the mutex, this ensures we don't try to send two
         * (or more) synchronous commands at a time.
         */
-       if (cmd->flags & CMD_SYNC)
+       if (!(cmd->flags & CMD_ASYNC))
                lockdep_assert_held(&priv->mutex);
 
        if (priv->ucode_owner == IWL_OWNERSHIP_TM &&