On Thu, 2015-05-07 at 15:42 +0300, Dan Carpenter wrote:

> > >           _iwl_trans_stop_device(mvm->trans, false);
> > >           _iwl_trans_start_hw(mvm->trans, false);
> > > -         if (ret)
> > > -                 return ret;
> > 
> > It looks like "ret = _iwl_trans_start_hw(...)" was intended though?
> > 
> 
> Hm...  You may be right.  Should it also be "goto error;" instead of a
> direct return?  I'm not positive.

Yes, looks like.

> Eran can you fix this and give me a Reported-by tag?

I think Eran is on sabbatical, I'll get this on the way internally with
your reported-by.

johannes

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to