Re: [PATCH] usb: gadget: f_midi: Add checking if it need align buffer's size to an ep's maxpacketsize

[Michal Nazarewicz <mina86@mina86.com>]

> Baolin Wang <baolin.wang@linaro.org> writes:
>> @@ -359,10 +361,12 @@ static int f_midi_set_alt(struct usb_function *f, unsigned intf, unsigned alt)
>>  
>>  	/* allocate a bunch of read buffers and queue them all at once. */
>>  	for (i = 0; i < midi->qlen && err == 0; i++) {
>> -		struct usb_request *req =
>> -			midi_alloc_ep_req(midi->out_ep,
>> -				max_t(unsigned, midi->buflen,
>> -					bulk_out_desc.wMaxPacketSize));
>> +		struct usb_request *req;
>> +
>> +		length = usb_ep_align_maybe(midi->gadget, midi->out_ep,
>> +					    midi->buflen);

On Fri, Jul 08 2016, Felipe Balbi wrote:
> after calling usb_ep_align_maybe()...
>
>> +		req = midi_alloc_ep_req(midi->out_ep,
>> +			max_t(unsigned, length, bulk_out_desc.wMaxPacketSize));
>
> ... max_t() is pointless. length will *always* >= wMaxPacketSize.

That is only true for gadgets with the quirk.  usb_ep_align_maybe is
a noöp for gadgets without the quirk.

-- 
Best regards
ミハウ “𝓶𝓲𝓷𝓪86” ナザレヴイツ
«If at first you don’t succeed, give up skydiving»