Re: [linux-usb-devel] ehci problem triggerable by storage

2007-08-02 Thread Greg KH
On Mon, Jul 30, 2007 at 11:11:54AM -0700, David Brownell wrote: > On Monday 30 July 2007, Oliver Neukum wrote: > > > > > But -- never actually having had both a highspeed USB sniffer *AND* > > > hardware exhibiting this problem in the same place -- I've not been > > > able to test that theory. > >

Re: [linux-usb-devel] autosuspend for asix driver

2007-08-02 Thread David Brownell
On Thursday 02 August 2007, Oliver Neukum wrote: > Am Freitag 03 August 2007 schrieb David Brownell: > > On Wednesday 01 August 2007, Oliver Neukum wrote: > > > Am Dienstag 31 Juli 2007 schrieb Alan Stern: > > > > You assign dev->intf in both the usbnet framework driver and the > > > > subdriver.  

Re: [linux-usb-devel] [PATCH] USB BIOS early handoff only when the we the driver is configured

2007-08-02 Thread David Engraf
Greg KH schrieb: > On Thu, Aug 02, 2007 at 10:32:21AM -0400, Alan Stern wrote: > >> On Thu, 2 Aug 2007, David Engraf wrote: >> >> >>> This would be solution too, but what if someone uses the uhci controller >>> and don't want the >>> ehci. So a single Kconfig flag wouldn't be enough, we hav

Re: [linux-usb-devel] autosuspend for asix driver

2007-08-02 Thread Oliver Neukum
Am Freitag 03 August 2007 schrieb David Brownell: > On Wednesday 01 August 2007, Oliver Neukum wrote: > > Am Dienstag 31 Juli 2007 schrieb Alan Stern: > > > You assign dev->intf in both the usbnet framework driver and the > > > subdriver.  Could the subdriver's assignment be removed? > > > > Here

Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes

2007-08-02 Thread David Brownell
On Thursday 02 August 2007, Matthew Garrett wrote: > The main > concern I have is that kernel developers just don't tend to be the sort > of people that use webcams, printers or scanners, so we're relying on > normal users to go to the effort of reporting that their device has > stopped working

Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes

2007-08-02 Thread David Brownell
On Thursday 02 August 2007, Alan Stern wrote: > Also, building something this sweeping into a kernel driver feels like > a mistake.  It ought to be more easily configurable from userspace, say > via a sysfs file. Yeah, I could have sworn there was extensive discussion over the creation of a sysfs

Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes

2007-08-02 Thread Alan Stern
On Fri, 3 Aug 2007, Matthew Garrett wrote: > On Thu, Aug 02, 2007 at 06:15:05PM -0700, Greg KH wrote: > > > Well, if you do this, then you can pretty much delete the whole quirk > > table we have, right? > > At the moment, yes. > > > And personally, I want to do better than Windows XP when it c

Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes

2007-08-02 Thread Matthew Garrett
On Thu, Aug 02, 2007 at 06:15:05PM -0700, Greg KH wrote: > Well, if you do this, then you can pretty much delete the whole quirk > table we have, right? At the moment, yes. > And personally, I want to do better than Windows XP when it comes to > power management. This patch is only going to sus

Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes

2007-08-02 Thread Greg KH
On Fri, Aug 03, 2007 at 12:56:13AM +0100, Matthew Garrett wrote: > We're seeing a large number of problems with devices not appreciating > USB autosuspend, especially printers and scanners. According to > http://www.microsoft.com/whdc/system/bus/USB/USBFAQ_intro.mspx only a > subset of drivers s

Re: [linux-usb-devel] autosuspend for asix driver

2007-08-02 Thread David Brownell
On Wednesday 01 August 2007, Oliver Neukum wrote: > Am Dienstag 31 Juli 2007 schrieb Alan Stern: > > You assign dev->intf in both the usbnet framework driver and the > > subdriver.  Could the subdriver's assignment be removed? > > Here we go again. > > Regards > Oliver > Signe

[linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes

2007-08-02 Thread Matthew Garrett
We're seeing a large number of problems with devices not appreciating USB autosuspend, especially printers and scanners. According to http://www.microsoft.com/whdc/system/bus/USB/USBFAQ_intro.mspx only a subset of drivers support it in Windows XP, meaning that most devices are probably untested

Re: [linux-usb-devel] High speed, high bandwidth isoc

2007-08-02 Thread David Brownell
On Wednesday 01 August 2007, Steve Calfee wrote: > At 10:42 PM 7/16/2007, David Brownell wrote: > >I don't recall testing high bandwidth ISO. In general, ISO support > >is pretty weak with EHCI ... Actually, now that I think of it I *did* test it, but only using the "usbtest" driver. In fact, I

[linux-usb-devel] patch usb-avoid-using-urb-pipe-in-usbcore.patch added to gregkh-2.6 tree

2007-08-02 Thread gregkh
This is a note to let you know that I've just added the patch titled Subject: [PATCH 4/7] USB: avoid using urb->pipe in usbcore to my gregkh-2.6 tree. Its filename is usb-avoid-using-urb-pipe-in-usbcore.patch This tree can be found at http://www.kernel.org/pub/linux/kernel/peop

[linux-usb-devel] patch usb-avoid-urb-pipe-in-usbmon.patch added to gregkh-2.6 tree

2007-08-02 Thread gregkh
This is a note to let you know that I've just added the patch titled Subject: [PATCH 7/7] USB: avoid urb->pipe in usbmon to my gregkh-2.6 tree. Its filename is usb-avoid-urb-pipe-in-usbmon.patch This tree can be found at http://www.kernel.org/pub/linux/kernel/people/gregkh/greg

[linux-usb-devel] patch usb-avoid-urb-pipe-in-usbfs.patch added to gregkh-2.6 tree

2007-08-02 Thread gregkh
This is a note to let you know that I've just added the patch titled Subject: [PATCH 6/7] USB: avoid urb->pipe in usbfs to my gregkh-2.6 tree. Its filename is usb-avoid-urb-pipe-in-usbfs.patch This tree can be found at http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh

[linux-usb-devel] patch usb-address-0-handling-during-device-initialization.patch added to gregkh-2.6 tree

2007-08-02 Thread gregkh
This is a note to let you know that I've just added the patch titled Subject: [PATCH 5/7] USB: address-0 handling during device initialization to my gregkh-2.6 tree. Its filename is usb-address-0-handling-during-device-initialization.patch This tree can be found at http://www.k

[linux-usb-devel] patch usb-add-urb-ep.patch added to gregkh-2.6 tree

2007-08-02 Thread gregkh
This is a note to let you know that I've just added the patch titled Subject: [PATCH 1/7] USB: add urb->ep to my gregkh-2.6 tree. Its filename is usb-add-urb-ep.patch This tree can be found at http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/patches/ >From [EMA

[linux-usb-devel] patch usb-add-ep-enable.patch added to gregkh-2.6 tree

2007-08-02 Thread gregkh
This is a note to let you know that I've just added the patch titled Subject: [PATCH 2/7] USB: add ep->enable to my gregkh-2.6 tree. Its filename is usb-add-ep-enable.patch This tree can be found at http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/patches/ >Fro

[linux-usb-devel] patch usb-add-direction-bit-to-urb-transfer_flags.patch added to gregkh-2.6 tree

2007-08-02 Thread gregkh
This is a note to let you know that I've just added the patch titled Subject: [PATCH 3/7] USB: add direction bit to urb->transfer_flags to my gregkh-2.6 tree. Its filename is usb-add-direction-bit-to-urb-transfer_flags.patch This tree can be found at http://www.kernel.org/pub/l

Re: [linux-usb-devel] [patch 2.6.23-rc1-git 6/6] gadget zero cleanups

2007-08-02 Thread David Brownell
Clean up gadget zero, using newer APIs and conventions: - gadget_is_dualspeed() and gadget_is_otg() ... #ifdef removal - Remove many now-needless #includes - Use the VERBOSE_DEBUG convention - Some whitespace fixes. - A few comment updates - Plus a few other small cleanups: don't pass

[linux-usb-devel] [PATCH 3/3] USB: separate out endpoint queue management and DMA mapping routines

2007-08-02 Thread Alan Stern
This patch (as953) separates out three key portions from usb_hcd_submit_urb(), usb_hcd_unlink_urb(), and usb_hcd_giveback_urb() and puts them in separate functions of their own. In the next patch, these functions will be called directly by host controller drivers while holding their private spinlo

[linux-usb-devel] [PATCH 2/3] USB: update spinlock usage for root-hub URBs

2007-08-02 Thread Alan Stern
This patch (as952) adjusts the spinlock usage in the root-hub emulation part of usbcore, to make it match more closely the pattern used by regular host controller drivers. To wit: The private lock (usb_hcd_root_hub_lock) is held throughout the important parts, and it is dropped temporarily without

[linux-usb-devel] [PATCH 1/3] USB: cleanup for previous patches

2007-08-02 Thread Alan Stern
This patch (as951) cleans up a few loose ends from earlier patches. Redundant checks for non-NULL urb->dev are removed, as are checks of urb->dev->bus (which can never be NULL). Conversely, a check for non-NULL urb->ep is added to the unlink paths. A homegrown round-down-to-power-of-2 loop is sim

[linux-usb-devel] [PATCH 0/3] USB core URB handling updates

2007-08-02 Thread Alan Stern
Greg: The following short patch series includes the following: Various cleanups suggested for the previous patches; Rework the spinlock usage in the root-hub URB code; Separate out endpoint queue management and transfer-buffer DMA mapping into their own subroutin

Re: [linux-usb-devel] [PATCH] cdc-acm: fix sysfs attribute registration bug

2007-08-02 Thread Oliver Neukum
Am Donnerstag 02 August 2007 schrieb Alan Stern: > This patch (as950) fixes a bug in the cdc-acm driver.  It doesn't keep > track of which interface (control or data) the sysfs attributes get > registered for, and as a result, during disconnect it will sometimes > attempt to remove the attributes f

[linux-usb-devel] [PATCH] cdc-acm: fix sysfs attribute registration bug

2007-08-02 Thread Alan Stern
This patch (as950) fixes a bug in the cdc-acm driver. It doesn't keep track of which interface (control or data) the sysfs attributes get registered for, and as a result, during disconnect it will sometimes attempt to remove the attributes from the wrong interface. The left-over attributes can ca

Re: [linux-usb-devel] [PATCH] USB BIOS early handoff only when the we the driver is configured

2007-08-02 Thread Greg KH
On Thu, Aug 02, 2007 at 10:32:21AM -0400, Alan Stern wrote: > On Thu, 2 Aug 2007, David Engraf wrote: > > > This would be solution too, but what if someone uses the uhci controller > > and don't want the > > ehci. So a single Kconfig flag wouldn't be enough, we have to add 3 > > flags for uchi, oh

Re: [linux-usb-devel] Can't unload uhci_hcd module with 2.6.22 -- also oops

2007-08-02 Thread A. Kalten
On Thu, 2 Aug 2007 10:42:14 -0400 (EDT) Alan Stern <[EMAIL PROTECTED]> wrote: > > And here it is. You can remove the earlier diagnostic patches; this > one by itself should fix everything. > This solves the problem. I can load and unload the modules without any error. Thanks to all for your

Re: [linux-usb-devel] [patch 2.6.23-rc1-git 5/6] file storage gadget cleanups

2007-08-02 Thread David Brownell
On Thursday 02 August 2007, Alan Stern wrote: > On Thu, 2 Aug 2007, David Brownell wrote: > > > Clean up the file storage gadget, using newer APIs and conventions: > > > > - gadget_is_dualspeed() and gadget_is_otg() ... #ifdef removal > > > > - Remove many now-needless #includes > > > > - Us

[linux-usb-devel] Virus "Email.Phishing.RB-1217" gefunden

2007-08-02 Thread Inode Mailscan
Sehr geehrte Damen und Herren, in dem E-Mail mit dem Betreff '[linux-usb-devel] You've received an ecard from a Class mate!' (gesendet am Thu, 2 Aug 2007 19:38:16 +0400) mit der angegebenen Absenderadresse '"1LoveCards.Com" <[EMAIL PROTECTED]>' wurde der Virus 'Email.Phishing.RB-1217' gefunden. A

Re: [linux-usb-devel] [patch 2.6.23-rc1-git 5/6] file storage gadget cleanups

2007-08-02 Thread Alan Stern
On Thu, 2 Aug 2007, David Brownell wrote: > Clean up the file storage gadget, using newer APIs and conventions: > > - gadget_is_dualspeed() and gadget_is_otg() ... #ifdef removal > > - Remove many now-needless #includes > > - Use the DEBUG (from Kconfig+Makefile) and VERBOSE_DEBUG convention

Re: [linux-usb-devel] Can't unload uhci_hcd module with 2.6.22 -- also oops

2007-08-02 Thread Oliver Neukum
Am Donnerstag 02 August 2007 schrieb Alan Stern: > On Wed, 1 Aug 2007, Alan Stern wrote: > > > Okay, good work.  Thanks to your careful experiments, plus the hints > > from Oliver and Tejun, I figured out the problem.  In short, the > > attribute files were registered on the control interface but

Re: [linux-usb-devel] Can't unload uhci_hcd module with 2.6.22 -- also oops

2007-08-02 Thread Alan Stern
On Wed, 1 Aug 2007, Alan Stern wrote: > Okay, good work. Thanks to your careful experiments, plus the hints > from Oliver and Tejun, I figured out the problem. In short, the > attribute files were registered on the control interface but the driver > tried to delete them from the data interface.

Re: [linux-usb-devel] [PATCH] USB BIOS early handoff only when the we the driver is configured

2007-08-02 Thread Alan Stern
On Thu, 2 Aug 2007, David Engraf wrote: > This would be solution too, but what if someone uses the uhci controller > and don't want the > ehci. So a single Kconfig flag wouldn't be enough, we have to add 3 > flags for uchi, ohci and > ehci. I think this maybe a little bit difficult when configurin

Re: [linux-usb-devel] 2.6.23-rc1-mm2

2007-08-02 Thread Mariusz Kozlowski
> > > > > === > > > > > uhci_hcd :00:0c.0: dma_pool_free buffer-32, 6b6b6b6b/6b6b6b6b > > > > > (bad dma) > > > > > > Mariusz, > > > > > > I guess the patch below (which I have just added to my tree) fixes > > > that, right? Thanks. > > > > Yes - that's correct. This patch

Re: [linux-usb-devel] 2.6.23-rc1-mm2

2007-08-02 Thread Jiri Kosina
On Thu, 2 Aug 2007, Alan Stern wrote: > > > > > uhci_hcd :00:0c.0: dma_pool_free buffer-32, 6b6b6b6b/6b6b6b6b > > > > > (bad dma) > > > I guess the patch below (which I have just added to my tree) fixes that, > > > right? Thanks. > > Yes - that's correct. This patch fixes the bug. Thanks. >

Re: [linux-usb-devel] 2.6.23-rc1-mm2

2007-08-02 Thread Alan Stern
On Thu, 2 Aug 2007, Mariusz Kozlowski wrote: > > > > === > > > > uhci_hcd :00:0c.0: dma_pool_free buffer-32, 6b6b6b6b/6b6b6b6b (bad > > > > dma) > > > > Mariusz, > > > > I guess the patch below (which I have just added to my tree) fixes that, > > right? Thanks. > > Ye

Re: [linux-usb-devel] How to write a user-space application for usb_skeleton?

2007-08-02 Thread Oliver Neukum
Am Donnerstag 02 August 2007 schrieb Thaens Tim: > I've compiled the usb_skeleton.c module and loaded this module. > I've managed to open a USB-device, but that's it. > > How can I write a application that uses this module? > Can I only use the file-operations like (owner, read, write, open and >

[linux-usb-devel] How to write a user-space application for usb_skeleton?

2007-08-02 Thread Thaens Tim
I've compiled the usb_skeleton.c module and loaded this module. I've managed to open a USB-device, but that's it. How can I write a application that uses this module? Can I only use the file-operations like (owner, read, write, open and release)? Is there a way to set the baudrate, get the vendor/

Re: [linux-usb-devel] 2.6.23-rc1-mm2

2007-08-02 Thread Mariusz Kozlowski
Hello, > > > usb 2-1: USB disconnect, address 2 > > > BUG: atomic counter underflow at: > > > [] show_trace_log_lvl+0x1a/0x30 > > > [] show_trace+0x12/0x14 > > > [] dump_stack+0x15/0x17 > > > [] __free_pages+0x50/0x52 > > > [] free_pages+0x1f/0x21 > > > [] dma_free_coherent+0x43/0x9c > > >

Re: [linux-usb-devel] [PATCH] USB BIOS early handoff only when the we the driver is configured

2007-08-02 Thread David Engraf
Alan Stern schrieb: > On Wed, 1 Aug 2007, David Engraf wrote: > > >> At the moment I have a Jetway/VIA Mainboard which seems to have a problem >> with the handoff. Even >> when I wait about 20 seconds the EHCI_USBLEGSUP_BIOS flag is not cleared. >> I think this is a BIOS >> bug and I will have

Re: [linux-usb-devel] [PATCH] USB BIOS early handoff only when the we the driver is configured

2007-08-02 Thread David Engraf
Greg KH schrieb: > On Wed, Aug 01, 2007 at 09:21:12AM +0200, David Engraf wrote: > >> At the moment I have a Jetway/VIA Mainboard which seems to have a >> problem with the handoff. >> Evenwhen I wait about 20 seconds the EHCI_USBLEGSUP_BIOS flag is not >> cleared. I think this is a BIOS >> bug

[linux-usb-devel] [patch 2.6.23-rc1-git 6/6] gadget zero cleanups

2007-08-02 Thread David Brownell
Clean up gadget zero, using newer APIs and conventions: - gadget_is_dualspeed() and gadget_is_otg() ... #ifdef removal - Remove many now-needless #includes - Use the VERBOSE_DEBUG convention - Some whitespace fixes. - A few comment updates - Plus a few other small cleanups: don't pass

[linux-usb-devel] [patch 2.6.23-rc1-git 5/6] file storage gadget cleanups

2007-08-02 Thread David Brownell
Clean up the file storage gadget, using newer APIs and conventions: - gadget_is_dualspeed() and gadget_is_otg() ... #ifdef removal - Remove many now-needless #includes - Use the DEBUG (from Kconfig+Makefile) and VERBOSE_DEBUG conventions. - Remove some "sparse" warnings (it still dislikes t

[linux-usb-devel] [patch 2.6.23-rc1-git 4/6] serial gadget cleanups

2007-08-02 Thread David Brownell
Clean up the serial gadget, using newer APIs and conventions: - gadget_is_dualspeed() and gadget_is_otg() ... #ifdef removal - Remove many now-needless #includes - Use the DEBUG and VERBOSE_DEBUG conventions; turned up a bug in the original debug messaging - Various whitespace fixes. Th

[linux-usb-devel] [patch 2.6.23-rc1-git 3/6] gmidi cleanups

2007-08-02 Thread David Brownell
Clean up the midi gadget, using newer APIs and conventions: - Remove many now-needless #includes - Use the DEBUG (from Kconfig+Makefile) and VERBOSE_DEBUG conventions. - Whitespace fixes There should be no effect on object code size. Signed-off-by: David Brownell <[EMAIL PROTECTED]> Cc: Ben

[linux-usb-devel] [patch 2.6.23-rc1-git 2/6] ethernet gadget cleanups, shrinkage

2007-08-02 Thread David Brownell
Clean up the ethernet gadget, using newer APIs and conventions: - gadget_is_dualspeed() and gadget_is_otg() ... #ifdef removal - Remove many now-needless #includes - Use the VERBOSE_DEBUG convention - Minor whitespace fixes. - Fix a warning from "sparse". Surprisingly, this saved about 2

[linux-usb-devel] [patch 2.6.23-rc1-git 1/6] gadget_is_{dualspeed, otg} predicates and cleanup

2007-08-02 Thread David Brownell
This adds two small inlines to the gadget stack, which will often evaluate to compile-time constants. That can help shrink object code and remove #ifdeffery. - gadget_is_dualspeed(), currently always a compile-time constant (depending on which controller is selected). - gadget_is_otg(), usu

[linux-usb-devel] [patch 2.6.23-rc1-git 0/6] some gadget stack cleanups

2007-08-02 Thread David Brownell
Following this are several minor gadget stack updates: - Add gadget_is_dualspeed() and gadget_is_otg() predicates, which can help remove #ifdeffery and associated cleanups. Includes updates to fsl_usb2_udc, omap_udc, and gadgetfs. - Use those to cleanup the Ethernet gadget. Amazingly, th