Re: [RFC ebeam PATCH 3/3] input: misc: New USB eBeam input driver.

2012-07-27 Thread Dmitry Torokhov
Hi Yann, On Sat, Jul 28, 2012 at 02:02:34AM +0200, Yann Cantin wrote: > > Signed-off-by: Yann Cantin > --- > drivers/input/misc/Kconfig | 21 + > drivers/input/misc/Makefile |1 + > drivers/input/misc/ebeam.c | 895 > +++ > 3 files changed, 917

Re: the dreaded "needs XHCI_TRUST_TX_LENGTH quirk" returns

2012-07-27 Thread Gary E. Miller
Yo Sarah! Sorry for the dealy responding. xorg-server version 1.12.99.903 kept the workstation that I have this bug on down much of the week. On Wed, 25 Jul 2012 10:46:37 -0700 Sarah Sharp wrote: > The patch will make the printk only trigger once per USB host > controller. The USB device real

[PATCH] usb: class: Modify printk to pr_info in cdc-acm.c

2012-07-27 Thread Masanari Iida
Modify printk to pr_info in drivers/usb/class/cdc-acm.c Confirm the compile succesfully. Signed-off-by: Masanari Iida --- drivers/usb/class/cdc-acm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c index f2a120e..2330c

[RFC ebeam PATCH 2/3] hid: hid-core.c: Blackist eBeam classic device.

2012-07-27 Thread Yann Cantin
Signed-off-by: Yann Cantin --- drivers/hid/hid-core.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c index 500844f..1c03586 100644 --- a/drivers/hid/hid-core.c +++ b/drivers/hid/hid-core.c @@ -1908,6 +1908,9 @@ static const struct hid_de

[RFC ebeam PATCH 3/3] input: misc: New USB eBeam input driver.

2012-07-27 Thread Yann Cantin
Signed-off-by: Yann Cantin --- drivers/input/misc/Kconfig | 21 + drivers/input/misc/Makefile |1 + drivers/input/misc/ebeam.c | 895 +++ 3 files changed, 917 insertions(+) create mode 100644 drivers/input/misc/ebeam.c diff --git a/drivers/input

[RFC ebeam PATCH 1/3] hid: hid-ids.h: Add vendor and device ID for eBeam classic device.

2012-07-27 Thread Yann Cantin
Signed-off-by: Yann Cantin --- drivers/hid/hid-ids.h |3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index 41c34f2..8cd4b8e 100644 --- a/drivers/hid/hid-ids.h +++ b/drivers/hid/hid-ids.h @@ -271,6 +271,9 @@ #define USB_DEVICE_ID_DWAV_EGALA

[RFC ebeam PATCH 0/3] new USB eBeam input driver

2012-07-27 Thread Yann Cantin
Hi, This patchset is a test-drive for a new USB input driver for eBeam devices. Currently, only the Luidia eBeam classic projection model is supported (usb id known). Patch 1 and 2 are here to let the ebeam driver be choose to handle the device instead of the generic-usb hid one (totally useless)

RE: usb hub is n't recognised during boot up(sometimes)

2012-07-27 Thread Sadasivan Shaiju
Hi Peter, I see the following error in our system . Any idea why this is happening ? . [ 87.929163] hub 1-0:1.0: state 7 ports 6 chg evt 0004 [ 87.929178] ehci_hcd :00:1a.7: GetStatus port 2 status 001002 P

Re: [patch] USB: kaweth.c: use GFP_ATOMIC under spin_lock

2012-07-27 Thread David Miller
From: Dan Carpenter Date: Fri, 27 Jul 2012 14:46:51 +0300 > The problem is that we call this with a spin lock held. The call tree > is: > kaweth_start_xmit() holds kaweth->device_lock. > -> kaweth_async_set_rx_mode() > -> kaweth_control() > -> kaweth_internal_con

Re: Kernel OOPS in usb_submit_urb in drivers/usb/core/urb.c

2012-07-27 Thread Damjan
Maybe I read the kernel OOPS wrong. Was &ep->desc NULL or not? It's hard to tell from the information in the oops. You'd have to get an assembly listing of usb_submit_urb() and figure out what source code statement corresponds to offset 0x75 in the object code. Why did you originally think the

Re: bisected regression, v3.5 -> next-20120724: PCI PM causes USB hotplug failure

2012-07-27 Thread Rafael J. Wysocki
On Friday, July 27, 2012, Alan Stern wrote: > On Fri, 27 Jul 2012, Rafael J. Wysocki wrote: > > > On Friday, July 27, 2012, Alan Stern wrote: > > > On Fri, 27 Jul 2012, Huang Ying wrote: > > > > > > > --- a/drivers/pci/pci-driver.c > > > > +++ b/drivers/pci/pci-driver.c > > > > @@ -280,8 +280,12

Re: Kernel OOPS in usb_submit_urb in drivers/usb/core/urb.c

2012-07-27 Thread Alan Stern
On Fri, 27 Jul 2012, Damjan wrote: > > No, that's not right at all, for a couple of reasons. > > > > First, &ep->desc is the address of a member of a structure. Even if ep > > itself were null, &ep->desc would not be NULL unless desc happened to > > be the first member of the ep structure (it is

Re: [PATCH 0/3] Fixing usb_wwan NULL pointer dereferencing

2012-07-27 Thread Thomas Schäfer
Am Freitag, 27. Juli 2012, 20:37:38 schrieben Sie: > Ah, right. I don't really use those scripts myself anymore. I just use > the qmicli tool from libqmi. And I assume that works for you as well? > It did not. But I gave up in an early state. But this project has its own mailing list. We ca

Re: bisected regression, v3.5 -> next-20120724: PCI PM causes USB hotplug failure

2012-07-27 Thread Alan Stern
On Fri, 27 Jul 2012, Rafael J. Wysocki wrote: > On Friday, July 27, 2012, Alan Stern wrote: > > On Fri, 27 Jul 2012, Huang Ying wrote: > > > > > --- a/drivers/pci/pci-driver.c > > > +++ b/drivers/pci/pci-driver.c > > > @@ -280,8 +280,12 @@ static long local_pci_probe(void *_ddi) > > > { > > >

Re: Kernel OOPS in usb_submit_urb in drivers/usb/core/urb.c

2012-07-27 Thread Damjan
No, that's not right at all, for a couple of reasons. First, &ep->desc is the address of a member of a structure. Even if ep itself were null, &ep->desc would not be NULL unless desc happened to be the first member of the ep structure (it is now, but there's no guarantee that it always will be).

Re: bisected regression, v3.5 -> next-20120724: PCI PM causes USB hotplug failure

2012-07-27 Thread Rafael J. Wysocki
On Friday, July 27, 2012, Alan Stern wrote: > On Fri, 27 Jul 2012, Huang Ying wrote: > > > --- a/drivers/pci/pci-driver.c > > +++ b/drivers/pci/pci-driver.c > > @@ -280,8 +280,12 @@ static long local_pci_probe(void *_ddi) > > { > > struct drv_dev_and_id *ddi = _ddi; > > struct device *dev

Re: [PATCH 0/3] Fixing usb_wwan NULL pointer dereferencing

2012-07-27 Thread Bjørn Mork
Thomas Schäfer writes: > I tried this patch-set today, applied to linux-next-20120727. > > As the RFC "Fixing usb_wwan NULL pointer dereferencing" set from yesterday it > did help to avoid oopses while unplugging the stick. > > That is the good thing. > > Bu

Re: usb scheduler

2012-07-27 Thread Alexey Filin
On Fri, Jul 27, 2012 at 9:08 PM, Peter Stuge wrote: >> 2. To provide link to external bus adapter with 1 us delay and use the >> same USB link from first use case. Not possible with USB. :( > > You didn't expand on the details, as I asked for, so I can't really > comment much more. > > The point I

Re: [PATCH 0/3] Fixing usb_wwan NULL pointer dereferencing

2012-07-27 Thread Thomas Schäfer
I tried this patch-set today, applied to linux-next-20120727. As the RFC "Fixing usb_wwan NULL pointer dereferencing" set from yesterday it did help to avoid oopses while unplugging the stick. That is the good thing. But there is still a problem - wwan0 doesn't work. /dev/c

Re: [PATCH] Add Etron XHCI quirk to avoid warning spam

2012-07-27 Thread Sarah Sharp
On Fri, Jul 27, 2012 at 02:40:56PM +0800, Daniel J Blueman wrote: > On 27 July 2012 14:14, Sarah Sharp wrote: > > On Fri, Jul 27, 2012 at 12:03:44PM +0800, Daniel J Blueman wrote: > >> When various USB3 devices with Etron XHCI controllers, we see a bunch of > >> warnings: > >> xhci_hcd :02:00.

Re: usb scheduler

2012-07-27 Thread Peter Stuge
Alexey Filin wrote: > >> For 2-byte read/write we could use either two bulk IN/OUT > >> endpoints (131 * 8kHz ~ 1 MHz) or one control endpoint (table 5-3, 42 > >> * 8 kHz = 336 kHz). > > > > If you can expand further on the details of your application then you > > will be able to get possibly unpar

[PATCH v2] ene_ub6250: Use macros for firmware names

2012-07-27 Thread Tim Gardner
Advertise firmware files using MODULE_FIRMWARE macros. Fix a debug string: SD_RDWR_PATTERN --> SD_RW_PATTERN Cc: Greg Kroah-Hartman Cc: linux-usb@vger.kernel.org Cc: usb-stor...@lists.one-eyed-alien.net Signed-off-by: Tim Gardner --- drivers/usb/storage/ene_ub6250.c | 28

Re: usb scheduler

2012-07-27 Thread Alexey Filin
On Fri, Jul 27, 2012 at 7:10 PM, Alan Stern wrote: > On Fri, 27 Jul 2012, Alexey Filin wrote: > >> So USB is not a "classic" bus and not universal, it is a network with >> non-constant delay. >> >> Are there USB controllers with the "synchronous transfers" reported >> not on microframe boundary, s

Re: usb scheduler

2012-07-27 Thread Alexey Filin
On Fri, Jul 27, 2012 at 5:04 PM, Peter Stuge wrote: > Alexey Filin wrote: >> So USB is not a "classic" bus and not universal, it is a network >> with non-constant delay. > > Whatever "classic" bus means.. USB is a packet-based communications > bus with well-defined timing characteristics. > > It i

Re: usb scheduler

2012-07-27 Thread Hans Petter Selasky
On Friday 27 July 2012 17:12:51 Alan Stern wrote: > On Fri, 27 Jul 2012, Hans Petter Selasky wrote: > > On Friday 27 July 2012 10:52:11 Alexey Filin wrote: > > > USB is a good choice for some bus adapters, it is used widely, cheap, > > > simple. For 2-byte read/write we could use either two bulk IN

Re: usb scheduler

2012-07-27 Thread Alan Stern
On Fri, 27 Jul 2012, Hans Petter Selasky wrote: > On Friday 27 July 2012 10:52:11 Alexey Filin wrote: > > USB is a good choice for some bus adapters, it is used widely, cheap, > > simple. For 2-byte read/write we could use either two bulk IN/OUT > > endpoints (131 * 8kHz ~ 1 MHz) or one control en

Re: usb scheduler

2012-07-27 Thread Alan Stern
On Fri, 27 Jul 2012, Alexey Filin wrote: > So USB is not a "classic" bus and not universal, it is a network with > non-constant delay. > > Are there USB controllers with the "synchronous transfers" reported > not on microframe boundary, so we can get response less 125 us? > Is there any USB2.0 sp

Re: bisected regression, v3.5 -> next-20120724: PCI PM causes USB hotplug failure

2012-07-27 Thread Alan Stern
On Fri, 27 Jul 2012, Huang Ying wrote: > --- a/drivers/pci/pci-driver.c > +++ b/drivers/pci/pci-driver.c > @@ -280,8 +280,12 @@ static long local_pci_probe(void *_ddi) > { > struct drv_dev_and_id *ddi = _ddi; > struct device *dev = &ddi->dev->dev; > + struct device *parent = dev->

Re: usb scheduler

2012-07-27 Thread Hans Petter Selasky
On Friday 27 July 2012 10:52:11 Alexey Filin wrote: > USB is a good choice for some bus adapters, it is used widely, cheap, > simple. For 2-byte read/write we could use either two bulk IN/OUT > endpoints (131 * 8kHz ~ 1 MHz) or one control endpoint (table 5-3, 42 > * 8 kHz = 336 kHz). Ethernet is o

Re: Kernel OOPS in usb_submit_urb in drivers/usb/core/urb.c

2012-07-27 Thread Alan Stern
On Thu, 26 Jul 2012, Damjan wrote: > On чет, 26 јул 2012 23:32:48 CEST, Greg KH wrote: > > On Thu, Jul 26, 2012 at 11:25:44PM +0200, Damjan wrote: > >> While playing with my custom made i2c-tiny-usb adapter I triggered > >> this kernel OOPS in usb_submit_urb in the drivers/usb/core/urb.c > >> file

Re: [PATCH V2] usb:musb: Fix for Full Speed issue

2012-07-27 Thread Felipe Balbi
On Fri, Jul 27, 2012 at 01:01:07PM +0200, Ajay M JAWADE wrote: > After sending the USB Bus reset, Host waits for High Speed Chirps. > If the soft-disconnect and soft-connect happens during this chirp time, > then Host doesn't detect this disconnect. > Host treats this as Full Speed Device as there

Re: usb scheduler

2012-07-27 Thread Peter Stuge
Alexey Filin wrote: > So USB is not a "classic" bus and not universal, it is a network > with non-constant delay. Whatever "classic" bus means.. USB is a packet-based communications bus with well-defined timing characteristics. It is obviously not a local bus, as you know from PC architecture the

[patch] USB: kaweth.c: use GFP_ATOMIC under spin_lock

2012-07-27 Thread Dan Carpenter
The problem is that we call this with a spin lock held. The call tree is: kaweth_start_xmit() holds kaweth->device_lock. -> kaweth_async_set_rx_mode() -> kaweth_control() -> kaweth_internal_control_msg() The kaweth_internal_control_msg() function is only c

RE: [PATCH] usb:musb: Fix for Full Speed issue

2012-07-27 Thread Ajay M JAWADE
Hi, > -Original Message- > From: Sergei Shtylyov [mailto:sshtyl...@mvista.com] > Sent: Thursday, July 26, 2012 5:55 PM > To: Ajay M JAWADE > Cc: linux-usb@vger.kernel.org > Subject: Re: [PATCH] usb:musb: Fix for Full Speed issue > > Hello. > > On 26-07-2012 10:03, Ajay M JAWADE wrote: >

[PATCH V2] usb:musb: Fix for Full Speed issue

2012-07-27 Thread Ajay M JAWADE
After sending the USB Bus reset, Host waits for High Speed Chirps. If the soft-disconnect and soft-connect happens during this chirp time, then Host doesn't detect this disconnect. Host treats this as Full Speed Device as there are no chirps. To avoid this Full Speed Enumeration, soft-disconnect an

RE: [PATCH] usb:musb: Fix for Full Speed issue

2012-07-27 Thread Ajay M JAWADE
Hi, > -Original Message- > From: Felipe Balbi [mailto:ba...@ti.com] > Sent: Thursday, July 26, 2012 1:02 PM > To: Ajay M JAWADE > Cc: linux-usb@vger.kernel.org > Subject: Re: [PATCH] usb:musb: Fix for Full Speed issue > > Hi, > > On Thu, Jul 26, 2012 at 08:03:11AM +0200, Ajay M JAWADE wr

Re: bisected regression, v3.5 -> next-20120724: PCI PM causes USB hotplug failure

2012-07-27 Thread Bjørn Mork
Huang Ying writes: > Do you have time to try the following patch? > > Best Regards, > Huang Ying > > --- > drivers/pci/pci-driver.c |6 ++ > 1 file changed, 6 insertions(+) > > --- a/drivers/pci/pci-driver.c > +++ b/drivers/pci/pci-driver.c > @@ -280,8 +280,12 @@ static long local_pci_pr

Re: usb scheduler

2012-07-27 Thread Alexey Filin
On Thu, Jul 26, 2012 at 7:59 PM, Hans Petter Selasky wrote: > On Thursday 26 July 2012 17:47:11 Alan Stern wrote: >> To get maximum throughput you should submit multiple URBs originally, >> and then submit a new URB whenever an URB completes. I recommand using >> a pipeline depth of 10-20 ms. Fo

[PATCH v6 09/11] usb: musb: dsps: remove explicit NOP device creation

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta As NOP device node is now added in am33xx tree so remove the call which creates the NOP platform_device. Signed-off-by: Ajay Kumar Gupta Signed-off-by: Ravi B --- drivers/usb/musb/musb_dsps.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/dri

[PATCH v6 06/11] arm/dts: am33xx: Add dt data for usbss

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta Added device tree data for usbss on am33xx. There are two musb controllers on am33xx platform so have port0_mode and port1_mode additional data. Signed-off-by: Ajay Kumar Gupta Signed-off-by: Ravi B --- arch/arm/boot/dts/am33xx.dtsi | 11 +++ 1 files changed,

[PATCH v6 10/11] usb: musb: dsps: get the PHY using phandle api

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta AM33xx has two PHY of same type used by each musb controller so use phandle of phy nodes to get the phy pointer. Signed-off-by: Ajay Kumar Gupta Signed-off-by: Ravi B --- .../devicetree/bindings/usb/am33xx-usb.txt |2 ++ drivers/usb/musb/musb_dsps.c

[PATCH v6 02/11] usb: musb: kill global and static for multi instance

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta Moved global variable "musb_debugfs_root" and static variable "old_state" to 'struct musb' to help support multi instance of musb controller as present on AM335x platform. Also removed the global variable "orig_dma_mask" and filled the dev->dma_mask with parent device's dm

[PATCH v6 01/11] usb: musb: add musb_ida for multi instance support

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta Added musb_ida in musb_core.c to manage the multi core ids. Signed-off-by: Ajay Kumar Gupta Signed-off-by: Ravi B --- drivers/usb/musb/am35x.c | 42 -- drivers/usb/musb/blackfin.c | 26 -- drivers/

[PATCH v6 11/11] arm/dts: am33xx: add phy phandle to usbss

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta Added NOP PHY phandle to usbss device node as same will be used to get the phy from otg framework. Signed-off-by: Ajay Kumar Gupta Signed-off-by: Ravi B --- arch/arm/boot/dts/am33xx.dtsi |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/arch/arm/

[PATCH v6 03/11] usb: musb: am335x: add support for dual instance

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta AM335x and TI81xx platform has dual musb controller so updating the musb_dspc.c to support the same. Changes: - Moved otg_workaround timer to glue structure - Moved static local variable last_timer to glue structure - PHY on/off related cleanups Si

[PATCH v6 04/11] usb: otg: nop: add support for multiple tranceiver

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta Currently we have one single nop transceiver support as same is defined as a global variable in drivers/usb/otg/nop-usb-xceiv.c. This need to be changed to support multiple otg controller each using nop transceiver on a platform such as am335x. Signed-off-by: Ajay Kumar Gu

[PATCH v6 05/11] usb: musb: dsps: add dt support

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta Added device tree support for dsps musb glue driver and updated the Documentation with device tree binding information. Signed-off-by: Ajay Kumar Gupta Signed-off-by: Ravi B --- .../devicetree/bindings/usb/am33xx-usb.txt | 14 + drivers/usb/musb/musb_dsps.

[PATCH v6 08/11] arm/dts: am33xx: add dt data for usb nop phy

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta AM33xx has two musb controller and they have one NOP PHY each. Added the device tree data for NOP PHY. Signed-off-by: Ajay Kumar Gupta Signed-off-by: Ravi B --- arch/arm/boot/dts/am33xx.dtsi |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a

[PATCH v6 07/11] usb: otg: nop: add dt support

2012-07-27 Thread Ravi B
From: Ajay Kumar Gupta Added device tree support for nop transceiver driver and updated the Documentation with device tree binding information for am33xx platform. Signed-off-by: Ajay Kumar Gupta Signed-off-by: Ravi B --- .../devicetree/bindings/usb/am33xx-usb.txt |3 +++ drivers/

[PATCH v6 00/11] usb: musb: adding multi instance support

2012-07-27 Thread Ravi B
This series of patches adds, a) Multi instances support in musb driver b) DT support for musb_dsps glue layer c) DT support for NOP transceiver AM33xx and TI81xx has dual musb controller and has two usb PHY of same type. This patch series uses 'phandle' based API devm_usb_get_phy_by_phandle() to g

Re: usb : mass storage : short_not_ok for non usb3 udc

2012-07-27 Thread Felipe Balbi
On Thu, Jul 26, 2012 at 11:17:38AM -0400, Alan Stern wrote: > On Thu, 26 Jul 2012, Felipe Balbi wrote: > > > Hi, > > > > On Wed, Jul 25, 2012 at 02:53:00PM +0200, Rajaram REGUPATHY wrote: > > > The short_not_ok field is used by class drivers to indicate udc whether > > > short packet is expected