Re: [PATCH] usb: add reset endpoint operations

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

 



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
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux