On 08/27/2014 09:35 AM, Jassi Brar wrote: > On Wed, Aug 27, 2014 at 3:23 AM, Daniel Mack <zonque@xxxxxxxxx> wrote: >> + uac2->p_residue = 0; >> } else { >> dev_err(dev, "%s:%d Error!\n", __func__, __LINE__); >> return -EINVAL; >> @@ -1128,7 +1188,7 @@ afunc_set_alt(struct usb_function *fn, unsigned intf, unsigned alt) >> >> req->zero = 0; >> req->context = &prm->ureq[i]; >> - req->length = prm->max_psize; >> + req->length = req_len; > (b) req->length = req_len or 176 > >> req->complete = agdev_iso_complete; >> req->buf = prm->rbuf + i * req->length; >> > Here req[0]->buf is req->length, which is 176 bytes from (b). Ah. Sorry, now I see what you mean. Yes, that should be req->buf = prm->rbuf + i * p_maxsize Alright, let me spin v5 later. Thanks, Daniel -- 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