Re: [PATCH] USB: musb_gadget: fix ZLP sending in musb_g_tx(v1)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2010/9/23 Felipe Balbi <balbi@xxxxxx>:
> Hi,
>
> On Wed, Sep 22, 2010 at 10:15:09AM -0500, tom.leiming@xxxxxxxxx wrote:
>>
>> +               if ((request->zero && request->length
>
> I think you can actually remove the request->length check, I doubt any
> gadget driver will queue a request without length sent to something more
> than 0. could you test that please ?

I think so too. But I am not sure, since I don't know why the
request->length check is introduced, so I don't remove the check in
the patch and just fix the ZLP issue only.

Also seems req->zero is used in some corner case only, I don't think
there are good test cases to verify the change of removing the check,
right?

So suggest to keep the check until we are sure it is safe to remove it.

-- 
Lei Ming
--
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


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux