Re: [PATCH] usb: add reset endpoint operations

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

 



On Thu, Apr 9, 2009 at 3:40 AM, Greg KH <gregkh@xxxxxxx> wrote:
> On Thu, Apr 09, 2009 at 03:36:51AM +0400, Alexey Klimov wrote:
>> (added linux-media maillist)
>>
>> Hello, David
>>
>> On Wed, Apr 8, 2009 at 9:36 PM, David Vrabel <david.vrabel@xxxxxxx> wrote:
>> > Wireless USB endpoint state has a sequence number and a current
>> > window and not just a single toggle bit.  So allow HCDs to provide a
>> > endpoint_reset method and call this or clear the software toggles as
>> > required (after a clear halt, set configuration etc.).
>> >
>> > usb_settoggle() and friends are then HCD internal and are moved into
>> > core/hcd.h and all device drivers call usb_reset_endpoint() instead.
>> >
>> > If the device endpoint state has been reset (with a clear halt) but
>> > the host endpoint state has not then subsequent data transfers will
>> > not complete. The device will only work again after it is reset or
>> > disconnected.
>> >
>> > Signed-off-by: David Vrabel <david.vrabel@xxxxxxx>
>> > ---
>> >  drivers/block/ub.c                        |   20 ++++-----
>> >  drivers/isdn/hisax/st5481_usb.c           |    9 +----
>> >  drivers/media/video/pvrusb2/pvrusb2-hdw.c |    1 -
>>
>> Looks like you change file under /drivers/video. It's better at least
>> to add linux-media maillist  or driver maintainer (not only linux-usb
>> list) to let developers know that you change drivers.
>
> He's already gotten an Ack from this driver author on the last time this
> patch was sent out.  Don't know why he forgot to add it to this version
> of the patch, it should still be valid :)
>
> thanks,
>
> greg k-h

Oh, good to know :)
Sorry for the mess.
-- 
Best regards, Klimov Alexey
--
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