Re: [PATCH 0/7] USB: fix ci13xxx_udc and gadgetfs related problems

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

 



On 10/13/2011 07:31 PM, Felipe Balbi wrote:
> Hi,
> 
> On Thu, Oct 13, 2011 at 05:01:52PM +0200, Michael Grzeschik wrote:
>> On Thu, Oct 13, 2011 at 01:51:52PM +0200, Marc Kleine-Budde wrote:
>>> On 10/11/2011 06:54 AM, Pavan Kondeti wrote:
>>>>> while playing around with gadgetfs and the ci13xxx_udc driver on mx28
>>>>> I found some problems regarding locking and DMA handling. Another problem
>>>>> arises when using the ci13xxx_udc driver on mx28, because it has only 8
>>>>> compared to 16 endpoints as the driver expects.
>>>
>>>>> With these patches g_ether and gadgetfs work on the mx28, however I was
>>>>> not able to test these patches on any other hardware. Voluntaries welcome.
>>>
>>>> Thanks for the improvements and bug fixes in ci13xxx_udc driver. All the
>>>> patches looks good to me. I am slightly busy with other stuff. Will test
>>>> these patches on MSM platform next week.
>>>
>>> We've discovered a problem with the driver (on a mx23).
>>> g_zero test 2 fails.
>>
>> We now are able to get all g_zero tests [1] working, but for that we have to
>> load the g_zero module with buflen=512.
>>
>> Has this to do with the limitation in the _hardware_enqueue routine that
>> it currently can not handle requests which would spawn into several TDs?
> 
> I just need a decision if I should take these on this merge window or
> wait a bit longer :-)

If you can check if these patches don't breaking any existing platforms,
then this can go into this merge window. I don't have any MSM platform
available...

regards, Marc

-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |

Attachment: signature.asc
Description: OpenPGP digital signature


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

  Powered by Linux