Re: [PATCH v7 0/6] usb: chipidea: udc: bugfixes

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

 



Michael Grzeschik <mgr@xxxxxxxxxxxxxx> writes:

> On Wed, Mar 27, 2013 at 06:35:30PM +0200, Alexander Shishkin wrote:
>> Michael Grzeschik <m.grzeschik@xxxxxxxxxxxxxx> writes:
>> 
>> > Hi,
>> >
>> > this series solve some issues with the chipidea udc.
>> >
>> > The series is based on v3.9-rc4.
>> >
>> > Michael Grzeschik (6):
>> >   usb: chipidea: udc: add attribute aligned(4) to shared memory structs
>> >   usb: chipidea: udc: only clear active and halted bits in qhead
>> >   usb: chipidea: udc: move ZLT flag change to ep_enable
>> >   usb: chipidea: udc: read status of td only once in hardware_dequeue
>> >   usb: chipidea: udc: don't truncate requests to single tds
>> >   usb: chipidea: udc: move _ep_queue into an unlocked function
>> 
>> Actually, do any of these patches fix tangible bugs, that can be
>> reproduced? If so, you should certainly mention that. But somehow I doubt
>> that, because otherwise the driver will be totally unusable.
>> 
>> What I see is patches that bring some parts of udc code in accordance
>> with the spec, which is a good thing to do, but doesn't fix any "real
>> bugs that bother people". Or does it?
>
> Obviously we were hunting a real issue while writing that patches. The
> most essential one is Patch 1/6, which definitely is a stable patch. I
> will clearify that in the patch description.

Does it then fall in "never worked before" category, since it looks like
it has never worked before on armv5 or did it work with older compilers?
At least the xscale version seems to be fine.

> The others though should go into v3.10, agreed?

Agreed. What about the multi-td patch and the rest? Are you going to
resend them still?

Regards,
--
Alex
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]