On Tue, 26 Apr 2016, Jiancheng Xue wrote:
> > If an error occurs here...
> >
> Sorry. It's really a problem. I'll modify it in v2 like below:
>
> err = reset_control_deassert(priv->rsts[rst]);
> if(err) {
> reset_control_put(priv->rsts[rst]);
> goto err_reset;
> }
Yes, that would be good. (Except for the missing ' ' between "if" and
"(".)
> Is it OK except this bug? If so, I'll send out v2 patch.
Everything else looked okay to me.
Alan Stern
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
