> > > > - mReq->req.actual = node_first->ptr->token & TD_TOTAL_BYTES; > > > - mReq->req.actual >>= ffs_nr(TD_TOTAL_BYTES); > > > - mReq->req.actual = mReq->req.length - mReq->req.actual; > > > + mReq->req.actual = actual; > > > > If the mReq->req.actual will be changed at below line, any reasons to do it? > > Correct, will remove that. > > > > mReq->req.actual = mReq->req.status ? 0 : mReq->req.actual; > > I will fix that to: > > mReq->req.actual = mReq->req.status ? 0 : actual; Yes, correct > > -- > Pengutronix e.K. | | > Industrial Linux Solutions | http://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | > -- Best Regards, Peter Chen -- 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
