RE: [PATCH 1/1] net: add dm9620 net usb driver

2013-07-02 Thread Joseph Chang
Dear Peter, This is Joseph CHANG, I am glade to contact and work with you. I am busy in some affairs else these days, and will like to be back soon. For the begin, We copy the source code 'dm9601.c' which was by you, In order to work for our new chips (DM9620, DM9621, DM9621A... series),

[PATCH 1/2] usb: misc: EHSET Test Fixture device driver for host compliance

2013-07-02 Thread Jack Pham
From: Manu Gautam An Embedded Host High-Speed Electrical Test (EHSET) test fixture is used to initiate test modes on a host controller in order to perform the high speed electrical testing procedure for USB-IF compliance. When this test fixture is connected to a host, it can enumerate as one of s

[PATCH 0/2] usb: Add support for EHSET

2013-07-02 Thread Jack Pham
PATCH 1/2 is a device driver supporting the Embedded Host High-Speed Electrical Test (EHSET) test fixture. This driver supports the set of VID/PIDs specified in the EHSET and activates the various test (TEST_J, TEST_K, et al) modes by issuing a USB_PORT_FEAT_TEST SetFeature request to the hub, or b

[RFC/PATCH 2/2] usb: ehci: Add support for SINGLE_STEP_SET_FEATURE test of EHSET

2013-07-02 Thread Jack Pham
From: Manu Gautam The USB Embedded High-speed Host Electrical Test (EHSET) defines the SINGLE_STEP_SET_FEATURE test as follows: 1) The host enumerates the test device with VID:0x1A0A, PID:0x0108 2) The host sends the SETUP stage of a GetDescriptor(Device) 3) The device ACKs the request 4) The ho

How to distinguish device classes which attached on port

2013-07-02 Thread Huang Rui
Hi, Sometimes I was encounterd an issue that auto-wakeup at second time from S3 if use mouse to wakeup OS at first time. Then I reset the port which current status is connected in PORTSC during pci resume process in host controller driver (xHCI), this issue doesn't happen. But resetting all connec

Re: How should we handle isochronous underruns?

2013-07-02 Thread Laurent Pinchart
Hi Alan, On Monday 01 July 2013 15:39:46 Alan Stern wrote: > On Mon, 1 Jul 2013, Laurent Pinchart wrote: > > > What about error recovery for insane systems? If we do get a 50-ms gap > > > in the data stream, what's the best way for the UVC driver to learn this > > > has happens and attempt to car

One USB mouse problem with runtime power management enabled

2013-07-02 Thread Peter Chen
Hi Jiri and Alan, I have run below below at 3.5.7 kernel, but after checking upstream kernel, it may also be existed. 1. evtest /dev/input/event1 & 2. Let mouse auto suspend, enable mouse as wakeup source, and let the system enters suspend. echo auto > /sys/bus/usb/devices/1-1/power/control echo

Re: [PATCH] staging: dwc2: fix thinko in dwc2_hc_set_even_odd_frame()

2013-07-02 Thread Stephen Warren
On 07/02/2013 07:47 PM, Stephen Warren wrote: > On 06/26/2013 02:20 PM, Paul Zimmerman wrote: >>> From: Stephen Warren [mailto:swar...@wwwdotorg.org] >>> Sent: Wednesday, June 26, 2013 12:13 PM >>> >>> On 06/24/2013 12:34 PM, Paul Zimmerman wrote: I screwed up the sense of this if() statement

RE: [PATCH 1/5] usb: dwc3: switch to GPL v2 only

2013-07-02 Thread Paul Zimmerman
> From: Felipe Balbi [mailto:ba...@ti.com] > Sent: Tuesday, July 02, 2013 3:33 PM > > On Tue, Jul 02, 2013 at 07:03:10PM +, Paul Zimmerman wrote: > > > From: Sebastian Andrzej Siewior > > > Sent: Tuesday, July 02, 2013 12:49 AM > > > > > > On 07/01/2013 09:18 AM, Felipe Balbi wrote: > > > > Th

Re: [PATCH] staging: dwc2: fix thinko in dwc2_hc_set_even_odd_frame()

2013-07-02 Thread Stephen Warren
On 06/26/2013 02:20 PM, Paul Zimmerman wrote: >> From: Stephen Warren [mailto:swar...@wwwdotorg.org] >> Sent: Wednesday, June 26, 2013 12:13 PM >> >> On 06/24/2013 12:34 PM, Paul Zimmerman wrote: >>> I screwed up the sense of this if() statement while porting our >>> vendor driver to create the dwc

Re: How should we handle isochronous underruns?

2013-07-02 Thread Ming Lei
On Tue, Jul 2, 2013 at 11:12 PM, Alan Stern wrote: > On Tue, 2 Jul 2013, Ming Lei wrote: > >> On Mon, Jul 1, 2013 at 10:48 PM, Alan Stern >> wrote: >> > On Mon, 1 Jul 2013, Ming Lei wrote: >> > The fact that the delay is small doesn't matter -- what's important is >> > that it will be > 0 wherea

Re: [PATCH v4 4/5] USB: EHCI: improve interrupt qh unlink

2013-07-02 Thread Ming Lei
On Wed, Jul 3, 2013 at 1:46 AM, Alan Stern wrote: >> >> - >> /* Poll the STS_ASS status bit; see when it agrees with CMD_ASE */ >> static void ehci_poll_ASS(struct ehci_hcd *ehci) >> { > > This blank line should remain intact. Apart from that, the patch is > okay. Sorry for missing this, will

Re: [PATCH 1/5] usb: dwc3: switch to GPL v2 only

2013-07-02 Thread Felipe Balbi
On Tue, Jul 02, 2013 at 07:03:10PM +, Paul Zimmerman wrote: > > From: Sebastian Andrzej Siewior > > Sent: Tuesday, July 02, 2013 12:49 AM > > > > On 07/01/2013 09:18 AM, Felipe Balbi wrote: > > > This is a Linux-only driver which makes use > > > of GPL-only symbols. It makes no sense to > > >

Re: [RFC v2 0/3] changes from v1

2013-07-02 Thread Sarah Sharp
Hi Xenia! This patchset looks great, aside from the subject line of your cover letter. It probably should have been "[RFC v2 0/3] xhci: remove USB_XHCI_HCD_DEBUGGING config option" instead of "[RFC v2 0/3] changes from v1". Don't worry about resending the patchset though. :) The patches look go

Re: Hardware bug in Intel USB-2 hub?

2013-07-02 Thread Sarah Sharp
On Tue, Jul 02, 2013 at 10:47:13AM -0400, Alan Stern wrote: > I think reverting the "global suspend" commit would be pretty easy. > But there is a problem. > > The commit was added at the time when I removed the CONFIG_USB_SUSPEND > option. Apparently some embedded systems were normally built w

[PATCH 3/3] USB: isp1362: move debug files from proc to debugfs

2013-07-02 Thread Greg Kroah-Hartman
Drivers should not be putting debug files in /proc/ that is what debugfs is for, so move the isp1362 driver's debug file to debugfs. Cc: Felipe Balbi Signed-off-by: Greg Kroah-Hartman --- drivers/usb/host/isp1362-hcd.c | 45 +++--- drivers/usb/host/isp1362.h

[PATCH 1/3] USB: remove unneeded idr.h include

2013-07-02 Thread Greg Kroah-Hartman
None of these USB files need idr.h, so don't include it. Cc: Alexander Shishkin Cc: Felipe Balbi Signed-off-by: Greg Kroah-Hartman --- drivers/usb/chipidea/core.c | 1 - drivers/usb/core/endpoint.c | 1 - drivers/usb/musb/musb_core.c | 1 - 3 files changed, 3 deletions(-) diff --git a/drive

[PATCH 2/3] USB: sl811: move debug files from proc to debugfs

2013-07-02 Thread Greg Kroah-Hartman
Drivers should not be putting debug files in /proc/ that is what debugfs is for, so move the sl811 driver's debug file to debugfs. Cc: Felipe Balbi Signed-off-by: Greg Kroah-Hartman --- drivers/usb/host/sl811-hcd.c | 38 +++--- drivers/usb/host/sl811.h | 2 +

RE: [PATCH 1/5] usb: dwc3: switch to GPL v2 only

2013-07-02 Thread Paul Zimmerman
> From: Sebastian Andrzej Siewior > Sent: Tuesday, July 02, 2013 12:49 AM > > On 07/01/2013 09:18 AM, Felipe Balbi wrote: > > This is a Linux-only driver which makes use > > of GPL-only symbols. It makes no sense to > > maintain Dual BSD/GPL licensing for this driver. > > > > Considering that the

Re: Regarding g_mass_storage.ko module configuration.

2013-07-02 Thread Sebastian Andrzej Siewior
On Fri, Jun 28, 2013 at 06:37:07PM +0200, humanshu bhatia wrote: > Hello , Hi, > > > I am using g_mass_storage for USB 3.0 testing for that I use below commands: > > modprobe g_mass_storage file=/tmp/ramdisk/disk > > kernel log output: > > usb 1-4: New USB device found, idVendor=0525, idProdu

Re: [PATCH] USB: OHCI: make ohci-da8xx a separate driver

2013-07-02 Thread Sekhar Nori
On 7/2/2013 10:50 PM, Kevin Hilman wrote: > On 07/02/2013 08:14 AM, Manjunath Goudar wrote: >> >> >> On 2 July 2013 20:20, Sergei Shtylyov >> > > wrote: >> >> Hello. >> >> >> On 02-07-2013 15:36, Manjunath Goudar wrote: >> >> Separate the D

Re: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Felipe Balbi
Hi, On Tue, Jul 02, 2013 at 06:04:45PM +, Paul Zimmerman wrote: > > From: linux-usb-ow...@vger.kernel.org > > [mailto:linux-usb-ow...@vger.kernel.org] On Behalf Of Felipe Balbi > > Sent: Tuesday, July 02, 2013 2:31 AM > > > > Signed-off-by: Felipe Balbi > > > > diff --git a/Documentati

Re: EHCI force full speed (chipidea ip) interrupt transfer

2013-07-02 Thread Greg KH
On Tue, Jul 02, 2013 at 01:58:29PM +0800, Rong Wang wrote: > Hi all, > > We are developing an embedded product which can connect ipod and play > its contents. > But for some particular reason we need to force the ipod to connect at > full-speed. > > Chipidea usb ip did not provide an companion HC

RE: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Paul Zimmerman
> From: linux-usb-ow...@vger.kernel.org > [mailto:linux-usb-ow...@vger.kernel.org] On Behalf Of Felipe Balbi > Sent: Tuesday, July 02, 2013 2:31 AM > > Signed-off-by: Felipe Balbi > > diff --git a/Documentation/devicetree/bindings/usb/generic.txt > b/Documentation/devicetree/bindings/usb/g

Re: [PATCH v4 4/5] USB: EHCI: improve interrupt qh unlink

2013-07-02 Thread Alan Stern
On Tue, 2 Jul 2013, Ming Lei wrote: > ehci-hcd currently unlinks an interrupt QH when it becomes empty, that > is, after its last URB completes. This works well because in almost > all cases, the completion handler for an interrupt URB resubmits the > URB; therefore the QH doesn't become empty an

Re: [PATCH v4 3/5] USB: EHCI: improve ehci_endpoint_disable

2013-07-02 Thread Alan Stern
On Tue, 2 Jul 2013, Ming Lei wrote: > The patch does the below improvement: > > - think QH_STATE_COMPLETING as unlinking state since all URBs on the > endpoint should be in unlinking or unlinked when doing endpoint_disable() > > - add "WARN_ON(!list_empty(&qh->qtd_list));" if qh->qh_state is > Q

Re: Scanner fails on USB3 port

2013-07-02 Thread Alan Stern
On Tue, 2 Jul 2013, Martin van Es wrote: > On Tue, Jul 2, 2013 at 6:47 PM, Alan Stern wrote: > > On Tue, 2 Jul 2013, Martin van Es wrote: > > > > Maybe that's the explanation: The scanner isn't able to cope when the > > packets arrive too rapidly. It _is_ clear from the usbmon data that > > the

Re: [PATCH] USB: OHCI: make ohci-da8xx a separate driver

2013-07-02 Thread Kevin Hilman
On 07/02/2013 08:14 AM, Manjunath Goudar wrote: > > > On 2 July 2013 20:20, Sergei Shtylyov > > wrote: > > Hello. > > > On 02-07-2013 15:36, Manjunath Goudar wrote: > > Separate the Davinci OHCI host controller driver from ohci-hcd >

Re: [RFC PATCH 4/6] USB: ehci-omap: Suspend the controller during bus suspend

2013-07-02 Thread Alan Stern
On Tue, 2 Jul 2013, Roger Quadros wrote: > On 07/02/2013 12:01 AM, Alan Stern wrote: > > On Mon, 1 Jul 2013, Felipe Balbi wrote: > > > >>> I don't know what Pad wakeup is. The wakeup signal has to originate > >>> from the EHCI controller, doesn't it? If not, how does the Pad know > >>> when a

Re: Enable xhci_hcd

2013-07-02 Thread Sultana
Greg KH writes: > Those are ehci controllers, not xhci ones (i.e. USB 2, not USB 3), so > the kernel is working as designed. > > sorry, > > greg k-h > -- Thanks for the information. I have to continue my work with EHCI software then :( -- To unsubscribe from this list: send the line "unsubs

Re: Scanner fails on USB3 port

2013-07-02 Thread Martin van Es
On Tue, Jul 2, 2013 at 6:47 PM, Alan Stern wrote: > On Tue, 2 Jul 2013, Martin van Es wrote: > > Maybe that's the explanation: The scanner isn't able to cope when the > packets arrive too rapidly. It _is_ clear from the usbmon data that > the scanner is at least slightly buggy. So the scanner mi

Re: Linux USB file storage gadget with new UDC

2013-07-02 Thread Alan Stern
On Tue, 2 Jul 2013, Victor Yeo wrote: > Hi, > > > No, that's not right. Set-Config has only two stages, Setup and > > Status; there is no Data stage. The flow is: > > > > Host Device > > - Setup Packet ---> | > > - Data0 Packet ---> |== Setup stage > > <

Re: Scanner fails on USB3 port

2013-07-02 Thread Alan Stern
On Tue, 2 Jul 2013, Martin van Es wrote: > Here's the usbmon output on my old USB2 laptop, kernel 3.9.1 > Including the plug/unplug events. scanimage -L succesfully reports the > scanner and returns the prompt. The two usbmon traces are essentially identical (including the -71 errors!) right up t

Re: Proposed modification to PL2303 driver

2013-07-02 Thread Frank Schäfer
Am 28.06.2013 15:53, schrieb Frank Schäfer: ... > Am 27.06.2013 22:13, schrieb Reinhard Max: >> But I am interested in improving this driver generally, not only to >> get it working on my particular device, which BTW is a data cable for >> Siemens mobile phones as it is often used by hobbyists to c

Re: [PATCH 09/24] USB: pl2303: clean up baud-rate handling

2013-07-02 Thread Frank Schäfer
Hi Johan, first of all: I see you've sent lots of great fixes and improvements for the USB-serial drivers during last months. I would like to thank you for all your work in this area ! One remark concerning this patch: Am 26.06.2013 16:47, schrieb Johan Hovold: > Clean up baud-rate handling som

Re: Linux USB file storage gadget with new UDC

2013-07-02 Thread Victor Yeo
Hi, > No, that's not right. Set-Config has only two stages, Setup and > Status; there is no Data stage. The flow is: > > Host Device > - Setup Packet ---> | > - Data0 Packet ---> |== Setup stage > < Ack Packet -- | > > - In Packet -->

Re: your mail

2013-07-02 Thread Alan Stern
On Mon, 1 Jul 2013, Devin Heitmueller wrote: > Hi all, > > I've been doing some debugging of a video corruption problem in the > em28xx video capture driver, and after a couple of weeks of digging > in, I think I might have exposed some sort race condition in the USB > core. > > http://devinheit

Re: [PATCH V2] USB: OHCI: make ohci-nxp a separate driver

2013-07-02 Thread Alan Stern
On Tue, 2 Jul 2013, Manjunath Goudar wrote: > Separate the OHCI NXP host controller driver from ohci-hcd > host code so that it can be built as a separate driver module. > This work is part of enabling multi-platform kernels on ARM. > > Many place function name and struct name started with usb, >

Re: How should we handle isochronous underruns?

2013-07-02 Thread Alan Stern
On Tue, 2 Jul 2013, Ming Lei wrote: > On Mon, Jul 1, 2013 at 10:48 PM, Alan Stern wrote: > > On Mon, 1 Jul 2013, Ming Lei wrote: > > The fact that the delay is small doesn't matter -- what's important is > > that it will be > 0 whereas now there is no delay between completion > > and resubmission

Re: omap4 ehci sporadic resume issue

2013-07-02 Thread Roger Quadros
On 07/02/2013 05:49 PM, Michael Trimarchi wrote: > Hi Roger > > On 07/02/2013 04:42 PM, Roger Quadros wrote: >> On 06/28/2013 07:47 PM, Michael Trimarchi wrote: >>> Hi >>> When you said earlier that the problem doesn't happen when one of the ULPIs is used did you try both of

Re: omap4 ehci sporadic resume issue

2013-07-02 Thread Nishanth Menon
On 16:49-20130702, Michael Trimarchi wrote: > Last question: > If one domain is in RET mode and not OFF mode what happen during SAR restore > in OFF mode? Without going to the depth of what TRM says already: SAR comes into play only if device-off sequence was triggered. Depending on whi

Re: Audio I/O parameters

2013-07-02 Thread Alan Stern
On Tue, 2 Jul 2013, Clemens Ladisch wrote: > Alan Stern wrote: > >256 samples/period / (44100 samples/second) * 8000 microframes/second > > = 46.44 microframes/period. > > > > Therefore I would expect to see snd-usb-audio submitting isochronous > > URBs with 46 or 47 packets, with a pipeli

[RFC v2 0/3] changes from v1

2013-07-02 Thread Xenia Ragiadakou
This patchset comes as the version 2 of the following patch: [RFC] xhci: remove USB_XHCI_HCD_DEBUGGING config option Changes from version 1: Before removing the USB_XHCI_HCD_DEBUGGING config option and the unused code related to it, this patchset first replaces all instances of xhci_info() and pr

[RFC v2 1/3] xhci: replace xhci_info() with xhci_dbg()

2013-07-02 Thread Xenia Ragiadakou
This patch replaces the calls to xhci_info() with calls to xhci_dbg() and removes the unused xhci_info() definition from xhci-hcd. By replacing the xhci_info() with xhci_dbg(), the calls to dev_info() are replaced with calls to dev_dbg() so that their output can be dynamically controlled via the d

[RFC v2 2/3] xhci: replace printk(KERN_DEBUG ...)

2013-07-02 Thread Xenia Ragiadakou
This patch replaces the calls to printk(KERN_DEBUG ...) with either calls to xhci_dbg() or calls to pr_debug(), depending on whether the xhci_hcd structure is available at callsite, so that the correspoding debugging messages are not enabled by default when CONFIG_DYNAMIC_DEBUG option is set but ra

[RFC v2 3/3] xhci: remove CONFIG_USB_XHCI_HCD_DEBUGGING and unused code

2013-07-02 Thread Xenia Ragiadakou
CONFIG_USB_XHCI_HCD_DEBUGGING option is used to enable verbose debugging output for the xHCI host controller driver. In the current version of the xhci-hcd driver, this option must be turned on, in order for the debugging log messages to be displayed, and users may need to recompile the linux kern

Re: [PATCH] USB: OHCI: make ohci-da8xx a separate driver

2013-07-02 Thread Sergei Shtylyov
Hello. On 02-07-2013 15:36, Manjunath Goudar wrote: Separate the Davinci OHCI host controller driver from ohci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; it would be nice to have in 3.11. One preexisting err

Re: omap4 ehci sporadic resume issue

2013-07-02 Thread Michael Trimarchi
Hi Roger On 07/02/2013 04:42 PM, Roger Quadros wrote: > On 06/28/2013 07:47 PM, Michael Trimarchi wrote: >> Hi >> >> On Fri, Jun 28, 2013 at 02:46:11PM +0300, Roger Quadros wrote: >>> On 06/28/2013 02:33 PM, Michael Trimarchi wrote: Hi Roger On Thu, Jun 27, 2013 at 11:07:11PM +0300,

Re: Hardware bug in Intel USB-2 hub?

2013-07-02 Thread Alan Stern
On Mon, 1 Jul 2013, Greg KH wrote: > On Mon, Jul 01, 2013 at 08:25:20PM -0700, Sarah Sharp wrote: > > On Mon, Jun 24, 2013 at 02:14:04PM -0400, Alan Stern wrote: > > > If the hardware doesn't expect global suspend, that's a hardware issue. > > > After all, global suspend is part of the USB-2 spe

Re: omap4 ehci sporadic resume issue

2013-07-02 Thread Roger Quadros
On 06/28/2013 07:47 PM, Michael Trimarchi wrote: > Hi > > On Fri, Jun 28, 2013 at 02:46:11PM +0300, Roger Quadros wrote: >> On 06/28/2013 02:33 PM, Michael Trimarchi wrote: >>> Hi Roger >>> >>> On Thu, Jun 27, 2013 at 11:07:11PM +0300, Ruslan Bilovol wrote: On Thu, Jun 27, 2013 at 10:24 PM, M

Re: Detecting start/stop streaming for UVC webcam with bulk transfer mode

2013-07-02 Thread Pratyush Anand
On 6/20/2013 11:20 AM, Chetan Nanda wrote: Hi Laurent, Thanks for your mail, On Tue, Jun 18, 2013 at 6:12 PM, Laurent Pinchart wrote: Hi Chetan, On Tuesday 18 June 2013 11:17:40 Chetan Nanda wrote: Hi, I am currently working with UVC camera device which send data using bulk transfer for pr

[PATCH V2] USB: OHCI: make ohci-nxp a separate driver

2013-07-02 Thread Manjunath Goudar
Separate the OHCI NXP host controller driver from ohci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM. Many place function name and struct name started with usb, current scenario replaced usb with ohci for proper namin

[PATCH] USB: storage: Add MicroVault Flash Drive to unusual_devs

2013-07-02 Thread Oskar Andero
From: Ren Bigcren The device report an error capacity when read_capacity_16(). Using read_capacity_10() can get the correct capacity. Signed-off-by: Ren Bigcren Cc: Matthew Dharm Signed-off-by: Oskar Andero --- drivers/usb/storage/unusual_devs.h | 7 +++ 1 file changed, 7 insertions(+)

[PATCH] USB: OHCI: make ohci-da8xx a separate driver

2013-07-02 Thread Manjunath Goudar
Separate the Davinci OHCI host controller driver from ohci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; it would be nice to have in 3.11. One preexisting error: "da8xx_syscfg0_base" [drivers/usb/host/ohci-da8xx.ko]

Re: How should we handle isochronous underruns?

2013-07-02 Thread Ming Lei
On Mon, Jul 1, 2013 at 10:48 PM, Alan Stern wrote: > On Mon, 1 Jul 2013, Ming Lei wrote: > The fact that the delay is small doesn't matter -- what's important is > that it will be > 0 whereas now there is no delay between completion > and resubmission (in fact, the resubmission occurs before the >

Re: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Sebastian Andrzej Siewior
On 07/02/2013 12:00 PM, Felipe Balbi wrote: > On Tue, Jul 02, 2013 at 11:35:02AM +0200, Sebastian Andrzej Siewior > wrote: >> On 07/02/2013 11:31 AM, Felipe Balbi wrote: - we seem to use usb-phy and phys. Couldn't we just stick to one? >>> >>> phys comes from the new PHY layer which isn

Re: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Felipe Balbi
On Tue, Jul 02, 2013 at 11:35:02AM +0200, Sebastian Andrzej Siewior wrote: > On 07/02/2013 11:31 AM, Felipe Balbi wrote: > >> - we seem to use usb-phy and phys. Couldn't we just stick to > >> one? > > > > phys comes from the new PHY layer which isn't merged yet. > > So we end up having both? Is t

Re: [PATCH 0/2] *** SUBJECT HERE ***

2013-07-02 Thread Johan Hovold
On Tue, Jul 02, 2013 at 01:22:01AM +0200, Anders Hammarquist wrote: > In a message of Fri, 28 Jun 2013 12:23:33 +0200, Johan Hovold writes: > >> I did a quick check of adding the device id though sysfs, and although > >> it partly works, it doesn't find the correct firmware (it ends up trying > >>

Re: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Sebastian Andrzej Siewior
On 07/02/2013 11:31 AM, Felipe Balbi wrote: >> - we seem to use usb-phy and phys. Couldn't we just stick to >> one? > > phys comes from the new PHY layer which isn't merged yet. So we end up having both? Is there are a rule which one should be used? I see ehci is using usb-phy while the omap thin

Re: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Felipe Balbi
On Tue, Jul 02, 2013 at 10:42:12AM +0200, Sebastian Andrzej Siewior wrote: > On 07/02/2013 10:26 AM, Felipe Balbi wrote: > > alright, how about this ? I'll split dr_mode documentation to a > > separate patch soon and resend with devicetree-discuss in the > > loop. > > Okay. > > > diff --git a/Doc

Re: Scanner fails on USB3 port

2013-07-02 Thread Martin van Es
Here's the usbmon output on my old USB2 laptop, kernel 3.9.1 Including the plug/unplug events. scanimage -L succesfully reports the scanner and returns the prompt. M. On Mon, Jul 1, 2013 at 10:14 PM, Alan Stern wrote: > On Mon, 1 Jul 2013, Martin van Es wrote: > >> Here's the usbmon log on bus 0

Re: [PATCH v2] net: cdc_ether: allow combined control and data interface

2013-07-02 Thread Oliver Neukum
On Monday 01 July 2013 23:38:17 David Miller wrote: > From: Bjørn Mork > Date: Sat, 29 Jun 2013 12:03:06 +0200 > > > Some Icera based Huawei modems handled by this driver are not > > completely CDC ECM compliant, using the same USB interface for both > > control and data. The CDC functional descr

Re: [PATCH v2] net: cdc_ether: allow combined control and data interface

2013-07-02 Thread David Miller
From: Oliver Neukum Date: Tue, 02 Jul 2013 10:17:51 +0200 > On Saturday 29 June 2013 12:03:06 Bjørn Mork wrote: >> Some Icera based Huawei modems handled by this driver are not >> completely CDC ECM compliant, using the same USB interface for both >> control and data. The CDC functional descripto

Re: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Sebastian Andrzej Siewior
On 07/02/2013 10:26 AM, Felipe Balbi wrote: > alright, how about this ? I'll split dr_mode documentation to a > separate patch soon and resend with devicetree-discuss in the > loop. Okay. > diff --git a/Documentation/devicetree/bindings/usb/generic.txt > b/Documentation/devicetree/bindings/usb/ge

Re: [PATCH 4/5] usb: dwc3: let non-DT platforms pass tx-fifo-resize flag;

2013-07-02 Thread Sebastian Andrzej Siewior
On 07/02/2013 10:24 AM, Felipe Balbi wrote: > nobody, frankly, but still, if we add something to DT, we should > add the equivalent platform_data, should some platform out there > want to use it. No, we should add code for use case which exist and not might exist. dwc3 is only included in new hard

Re: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Felipe Balbi
Hi, On Tue, Jul 02, 2013 at 10:14:08AM +0200, Sebastian Andrzej Siewior wrote: > >> This looks tricky. If you copy that property unconditionally than > >> you will end up with UNKNOWN speed if that property is missing > >> and no gadget will load, right? If so, maybe a second argument > >> would b

Re: [PATCH 4/5] usb: dwc3: let non-DT platforms pass tx-fifo-resize flag;

2013-07-02 Thread Felipe Balbi
Hi, On Tue, Jul 02, 2013 at 10:08:37AM +0200, Sebastian Andrzej Siewior wrote: > On 07/01/2013 09:18 AM, Felipe Balbi wrote: > > in case we're not in a DT boot, we should > > still be able to tell the driver how to behave. > > > > In order to be able to pass flags to the driver, > > we introduce

Re: [RFC PATCH 4/6] USB: ehci-omap: Suspend the controller during bus suspend

2013-07-02 Thread Roger Quadros
On 07/02/2013 12:01 AM, Alan Stern wrote: > On Mon, 1 Jul 2013, Felipe Balbi wrote: > >>> I don't know what Pad wakeup is. The wakeup signal has to originate >>> from the EHCI controller, doesn't it? If not, how does the Pad know >>> when a wakeup is needed? >> >> That's really an OMAP thing,

Re: [PATCH v2] net: cdc_ether: allow combined control and data interface

2013-07-02 Thread Oliver Neukum
On Saturday 29 June 2013 12:03:06 Bjørn Mork wrote: > Some Icera based Huawei modems handled by this driver are not > completely CDC ECM compliant, using the same USB interface for both > control and data. The CDC functional descriptors include a Union > naming this interface as both master and sla

Re: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Sebastian Andrzej Siewior
On 07/02/2013 10:10 AM, Felipe Balbi wrote: > Hi, Hi Felipe, > On Tue, Jul 02, 2013 at 10:06:23AM +0200, Sebastian Andrzej Siewior > wrote: >> On 07/01/2013 09:18 AM, Felipe Balbi wrote: >>> diff --git a/drivers/usb/usb-common.c >>> b/drivers/usb/usb-common.c index 675384d..b87f202 100644 --- >>>

Re: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Felipe Balbi
Hi, On Tue, Jul 02, 2013 at 10:06:23AM +0200, Sebastian Andrzej Siewior wrote: > On 07/01/2013 09:18 AM, Felipe Balbi wrote: > > diff --git a/drivers/usb/usb-common.c b/drivers/usb/usb-common.c > > index 675384d..b87f202 100644 > > --- a/drivers/usb/usb-common.c > > +++ b/drivers/usb/usb-common.c

Re: [PATCH 4/5] usb: dwc3: let non-DT platforms pass tx-fifo-resize flag;

2013-07-02 Thread Sebastian Andrzej Siewior
On 07/01/2013 09:18 AM, Felipe Balbi wrote: > in case we're not in a DT boot, we should > still be able to tell the driver how to behave. > > In order to be able to pass flags to the driver, > we introduce platform_data structure which the > core driver should use. Now you are getting me curious.

Re: [PATCH 3/5] usb: common: introduce of_usb_get_maximum_speed()

2013-07-02 Thread Sebastian Andrzej Siewior
On 07/01/2013 09:18 AM, Felipe Balbi wrote: > diff --git a/drivers/usb/usb-common.c b/drivers/usb/usb-common.c > index 675384d..b87f202 100644 > --- a/drivers/usb/usb-common.c > +++ b/drivers/usb/usb-common.c > @@ -112,6 +112,41 @@ enum usb_dr_mode of_usb_get_dr_mode(struct device_node > *np) >

[PATCH v3] usb: clamp bInterval to allowed range

2013-07-02 Thread Felipe Balbi
bInterval must be within the range 1 - 16 when running at High/Super speed, and within the range 1 - 255 when running at Full/Low speed. In order to catch drivers passing a too large bInterval on Super/High speed scenarios (thus overflowing urb->interval), let's clamp() the argument to the allowed

Re: [PATCH 1/5] usb: dwc3: switch to GPL v2 only

2013-07-02 Thread Sebastian Andrzej Siewior
On 07/01/2013 09:18 AM, Felipe Balbi wrote: > This is a Linux-only driver which makes use > of GPL-only symbols. It makes no sense to > maintain Dual BSD/GPL licensing for this driver. > > Considering that the amount of work to use this > driver in any different operating system would likely > be

Re: [PATCH] net: ipheth: Add USB ID for iPad mini

2013-07-02 Thread David Miller
From: Aaron Marburg Date: Tue, 2 Jul 2013 10:00:06 +1200 > Adds the USB device ID (0x12ab) to the ipheth network-over-USB-tethering > driver for iOS devices. Applied and tested against mainline tag v3.10 > (as well as 3.8.x and 3.6.y kernel for Raspbian on Raspberry pi) > > > Signed-off-by: Aa

Re: Audio I/O parameters

2013-07-02 Thread Clemens Ladisch
Alan Stern wrote: >256 samples/period / (44100 samples/second) * 8000 microframes/second > = 46.44 microframes/period. > > Therefore I would expect to see snd-usb-audio submitting isochronous > URBs with 46 or 47 packets, with a pipeline depth of 2 URBs. > > However, that's not what actua