[PATCH net-next v6 2/3] net: huawei_cdc_ncm: Introduce the huawei_cdc_ncm driver

2013-11-04 Thread Bjørn Mork
From: Enrico Mioso mrkiko...@gmail.com This driver supports devices using the NCM protocol as an encapsulation layer for other protocols, like the E3131 Huawei 3G modem. This drivers approach was heavily inspired by the qmi_wwan/cdc_mbim approach code model. Signed-off-by: Enrico Mioso

[PATCH net-next v6 0/3] The huawei_cdc_ncm driver

2013-11-04 Thread Bjørn Mork
Enrico has been kind enough to let me repost his driver with the changes requested by Oliver Neukum during the last review of this series. The changes I have made from Enricos original v5 series to this version are: v6: - fix to avoid corrupting drvstate-pmcount - fix error return value from

[PATCH net-next v6 3/3] net: cdc_ncm: remove non-standard NCM device IDs

2013-11-04 Thread Bjørn Mork
From: Enrico Mioso mrkiko...@gmail.com Remove device IDs of NCM-like (but not NCM-conformant) devices, that are handled by the huawwei_cdc_ncm driver now. Signed-off-by: Enrico Mioso mrkiko...@gmail.com Signed-off-by: Bjørn Mork bj...@mork.no --- drivers/net/usb/cdc_ncm.c | 11 --- 1

[PATCH net-next v6 1/3] net: cdc_ncm: Export cdc_ncm_{tx,rx}_fixup functions for re-use

2013-11-04 Thread Bjørn Mork
From: Enrico Mioso mrkiko...@gmail.com Some drivers implementing NCM-like protocols, may re-use those functions, as is the case in the huawei_cdc_ncm driver. Export them via EXPORT_SYMBOL_GPL, in accordance with how other functions have been exported. Signed-off-by: Enrico Mioso

Re: [PATCH net-next 17/24] net: cdc_ncm: use netif_* and dev_* instead of pr_*

2013-11-04 Thread Bjørn Mork
Joe Perches j...@perches.com writes: You did most all the multi-line statement alignment perfectly but missed a couple. Damn! OK, that will teach me to do checkpatch --strict before submitting. Maybe in a follow-on patch. Yes, I'll fix this when I get around to the next series for this

[PATCH 3/3] USB: phy: Add OTG FSM configuration option

2013-11-04 Thread Anton Tikhomirov
This patch removes dependency on Freescale USB UTG Transceiver driver and makes OTG FSM implementation selectable. Signed-off-by: Anton Tikhomirov av.tikhomi...@samsung.com --- drivers/usb/phy/Kconfig | 10 +- drivers/usb/phy/Makefile |4 ++-- 2 files changed, 11 insertions(+), 3

[PATCH 1/3] USB: phy: replace spinlock with mutex in OTG FSM

2013-11-04 Thread Anton Tikhomirov
OTG Final State Machine calls functions which may sleep. For example, start_gadget callback implementation can use usb_gadget_vbus_connect(), whose context: can sleep. If so, mutex should be used instead of spinlock. Signed-off-by: Anton Tikhomirov av.tikhomi...@samsung.com ---

[PATCH 2/3] USB: phy: move OTG FSM header

2013-11-04 Thread Anton Tikhomirov
Other USB drivers may want to use OTG final state machine implementation, so make this header available for them. Signed-off-by: Anton Tikhomirov av.tikhomi...@samsung.com --- drivers/usb/phy/phy-fsl-usb.h |1 - drivers/usb/phy/phy-fsm-usb.c |2 - drivers/usb/phy/phy-fsm-usb.h | 236

Re: [PATCH V5 net-next 2/3] net: huawei_cdc_ncm: Introduce the huawei_cdc_ncm driver

2013-11-04 Thread Oliver Neukum
On Fri, 2013-11-01 at 12:35 +0100, Bjørn Mork wrote: So I believe we should do the update unconditionally, and but skip usb_autopm_put_interface if the get failed. Accordingly, these functions should always return 0 (not that there is anything currently checking the return anyway). I'll

Re: [PATCH] USB: pl2303: fixed handling of CS5 setting

2013-11-04 Thread Johan Hovold
On Sun, Nov 03, 2013 at 11:09:50PM +0100, Colin Leitner wrote: This patch fixes the CS5 setting on the PL2303 USB-to-serial devices. CS5 has a value of 0 and the CSIZE setting has been skipped altogether by the enclosing if. Tested on 3.11.6 and the scope shows the correct output after the

RE: [PATCH RFC 1/4] phy: Add new Exynos5 USB 3.0 PHY driver

2013-11-04 Thread Kamil Debski
Hi Kishon, From: Kishon Vijay Abraham I [mailto:kis...@ti.com] Sent: Monday, November 04, 2013 7:55 AM Hi Vivek, On Thursday 31 October 2013 01:15 PM, Vivek Gautam wrote: Add a new driver for the USB 3.0 PHY on Exynos5 series of SoCs. The new driver uses the generic PHY framework and

[PATCH] xhci: Allocate the td array and urb_priv together.

2013-11-04 Thread David Laight
Replace the array of pointers to transfer descriptors with the transfer descriptors themselves. This saves a kzalloc() call on every transfer and some memory indirections. The only possible downside is for isochronous tranfers with 64 td when the allocate is 8+4096 bytes (on 64bit systems) so

[PATCH] xhci: Use TRB_CYCLE instead of the constant 0x1

2013-11-04 Thread David Laight
In many cases the constant 1 is used for values that eventually get written to the hardware ring. Replace all of these with the symbolic constant TRB_CYCLE. This makes it clear that the ring-cycle_state value is used when writing to the actual ring itself. Always use ^= TRB_CYCLE to invert the

[PATCH 1/4] usb/gadget: composite: redirect setup requests

2013-11-04 Thread Andrzej Pietrasiewicz
If there are setup requests not directed to an endpont or an interface, current config's setup() has been attempted so far. This patch, in case the above fails, adds code which tries the setup() of configuration's function if there is only one function in the configuration. This behavior is

[PATCH 3/4] usb/gadget: f_loopback: add configfs support

2013-11-04 Thread Andrzej Pietrasiewicz
Add support for using the loopback USB function in gadgets composed with configfs. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- .../ABI/testing/configfs-usb-gadget-loopback |8 ++ drivers/usb/gadget/Kconfig

[PATCH 2/4] usb/gadget: factor out alloc_ep_req

2013-11-04 Thread Andrzej Pietrasiewicz
alloc_ep_req() is a function repeated in several modules. Make a common implementation and use it. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/Makefile |2 +- drivers/usb/gadget/f_hid.c

[PATCH 4/4] usb/gadget: f_sourcesink: add configfs support

2013-11-04 Thread Andrzej Pietrasiewicz
Add support for using the sourcesink function in gadgets composed with configfs. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- .../ABI/testing/configfs-usb-gadget-sourcesink | 12 + drivers/usb/gadget/Kconfig

[PATCH 0/4] Equivalent of g_zero with configfs

2013-11-04 Thread Andrzej Pietrasiewicz
This series aims at integrating configfs into gadget zero, the way it has been done for acm, ncm, ecm, eem, ecm subset, rndis, obex, phonet, mass_storage and FunctionFS. It contains everything that is required to provide the equivalent of g_zero.ko with configfs. BACKWARD COMPATIBILITY

about libusbg (formerly libgadget)

2013-11-04 Thread Andrzej Pietrasiewicz
Hi Matt, Thank you for the library: http://thread.gmane.org/gmane.linux.kernel/1556137 I'd like to make some comments you might want to consider. I don't like too much the idea of hardcoding the set of available functions. When new functions' conversions to use configfs are available (or

Re: Read transaction not work for EHCI host controller

2013-11-04 Thread ys6 land
Hi, greg I have used the minicom program which calls the open system call in blocking mode to access the usb device, and it seems there is no problem with it. So, the only thing I can suspect is that there is a bug in open system call implementation. Do you know which directory containing the

Re: [PATCH RFC 1/4] phy: Add new Exynos5 USB 3.0 PHY driver

2013-11-04 Thread Tomasz Figa
Hi Kishon, On Monday 04 of November 2013 12:24:42 Kishon Vijay Abraham I wrote: Hi Vivek, On Thursday 31 October 2013 01:15 PM, Vivek Gautam wrote: Add a new driver for the USB 3.0 PHY on Exynos5 series of SoCs. The new driver uses the generic PHY framework and will interact with DWC3

RE: Gadget tool proposition

2013-11-04 Thread Krzysztof Opasiak
Hi, -Original Message- From: Felipe Balbi [mailto:ba...@ti.com] Sent: Thursday, October 31, 2013 4:23 PM To: Krzysztof Opasiak Cc: ba...@ti.com; linux-usb@vger.kernel.org; 'Matt Porter' Subject: Re: Gadget tool proposition Hi, On Thu, Oct 31, 2013 at 01:35:45PM +0100,

RE: Gadget tool proposition

2013-11-04 Thread Krzysztof Opasiak
Hi, -Original Message- From: 'Matt Porter' [mailto:matt.por...@linaro.org] Sent: Saturday, November 02, 2013 6:59 PM To: Krzysztof Opasiak Cc: ba...@ti.com; linux-usb@vger.kernel.org Subject: Re: Gadget tool proposition On Thu, Oct 31, 2013 at 01:35:45PM +0100, Krzysztof Opasiak

Re: Read transaction not work for EHCI host controller

2013-11-04 Thread Greg KH
A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Mon, Nov 04, 2013 at 07:58:42PM +0800, ys6 land wrote: Hi, greg I have used the minicom program which calls the open system call in blocking mode to access the usb device, and it seems there

[PATCH][bugfix] usb/gadget: f_mass_storage: fix mass storage dependency

2013-11-04 Thread Andrzej Pietrasiewicz
Legacy gadgets supporting mass storage (g_mass_storage, g_acm_ms, g_multi) all depend on BLOCK. Make the standalone compilation of f_mass_storage (without any legacy gadget) dependent no BLOCK, too. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park

[PATCH] USB: serial: option: blacklist intf1 for Huawei E173s-6

2013-11-04 Thread Gustavo Zacarias
Interface 1 on this device isn't for option to bind to otherwise an oops on usb_wwan with log flooding will happen: tty_release: ttyUSB1: read/write wait queue active! And it doesn't seem to respond to QMI if it's added to qmi_wwan so don't add it there. Signed-off-by: Gustavo Zacarias

USB 3.0 SDR / Ettus Research B200/210

2013-11-04 Thread Moritz Fischer
Hi guys, This is my first post to this list, so let me quickly introduce myself. I'm a software engineer for Ettus Research (National Instruments). One of our new products [1] is a USB 3.0 (using Cypress FX3) based SDR that currently ships with a libusb based open source driver integrated in our

Re: [PATCH RFC 1/4] phy: Add new Exynos5 USB 3.0 PHY driver

2013-11-04 Thread Kishon Vijay Abraham I
On Monday 04 November 2013 05:56 PM, Tomasz Figa wrote: Hi Kishon, On Monday 04 of November 2013 12:24:42 Kishon Vijay Abraham I wrote: Hi Vivek, On Thursday 31 October 2013 01:15 PM, Vivek Gautam wrote: Add a new driver for the USB 3.0 PHY on Exynos5 series of SoCs. The new driver uses the

Re: [PATCH RFC 1/4] phy: Add new Exynos5 USB 3.0 PHY driver

2013-11-04 Thread Kishon Vijay Abraham I
Hi, On Monday 04 November 2013 03:45 PM, Kamil Debski wrote: Hi Kishon, From: Kishon Vijay Abraham I [mailto:kis...@ti.com] Sent: Monday, November 04, 2013 7:55 AM Hi Vivek, On Thursday 31 October 2013 01:15 PM, Vivek Gautam wrote: Add a new driver for the USB 3.0 PHY on Exynos5 series of

[PATCH] usb: usbtest: add a test case to support bos for queue control

2013-11-04 Thread Huang Rui
In Test 10 of usbtest module, it queues multiple control messages and thereby tests control message queuing, protocol stalls, short reads, and fault handling. And this patch add a test case to support queue BOS control request for USB 3.0 SPEC. Signed-off-by: Huang Rui ray.hu...@amd.com ---

RE: about libusbg (formerly libgadget)

2013-11-04 Thread Krzysztof Opasiak
Hi, -Original Message- From: Andrzej Pietrasiewicz [mailto:andrze...@samsung.com] Sent: Monday, November 04, 2013 12:52 PM To: matt.por...@linaro.org Cc: jeng...@inai.de; linux-usb@vger.kernel.org; Marek Szyprowski; kyungmin.p...@samsung.com; Krzysztof Opasiak Subject: about

Re: [PATCH] USB: ehci-atmel: add usb_clk for transition to CCF

2013-11-04 Thread Nicolas Ferre
On 18/10/2013 21:26, Boris BREZILLON : The AT91 PMC (Power Management Controller) provides a USB clock used by the different USB controllers (ehci, ohci and udc). The atmel-ehci driver must configure the usb clock rate to 48Mhz in order to get a fully functionnal USB host controller. This

[PATCH 2/2] added fputs() fgets() error handling

2013-11-04 Thread Stanislaw Wadas
Change-Id: I787380dda981c7cee6508a4ff566d7ca9fb273cf Signed-off-by: Stanislaw Wadas s.wa...@samsung.com --- src/gadget.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/src/gadget.c b/src/gadget.c index faed675..ff4f3d5 100644 --- a/src/gadget.c +++

libusbg clean up and error handling

2013-11-04 Thread Stanislaw Wadas
In reference to the message sent by Andrzej Pietrasiewicz (about libusbg (formerly libgadget)) I would like to propose some changes to libusbg. Best Regards Stanislaw Wadas Samsung RD Institute Poland Samsung Electronics s.wa...@samsung.com -- To unsubscribe from this list: send the line

[PATCH 1/2] Moved mkdir functions, added MAX_LENGHT MAX_PATH_LENGHT

2013-11-04 Thread Stanislaw Wadas
mkdir() function are now called after successfull function creation. Added inline to gadget_write_string(). 256 hard coded value has been replaced by two defined constants MAX_LENGHT and MAX_PATH_LENGHT Change-Id: Ifcfaf9cf95da6558c176ce3367cd553cea54e871 Signed-off-by: Stanislaw Wadas

Re: [PATCH 1/2] Moved mkdir functions, added MAX_LENGHT MAX_PATH_LENGHT

2013-11-04 Thread Andrzej Pietrasiewicz
W dniu 04.11.2013 14:55, Stanislaw Wadas pisze: mkdir() function are now called after successfull function creation. Added inline to gadget_write_string(). 256 hard coded value has been replaced by two defined constants MAX_LENGHT and MAX_PATH_LENGHT LENGHT - LENGTH Change-Id:

Re: [PATCH net-next 13/13] scripts/checkpatch.pl: Add dev_kfree_skb*(NULL) check to checkpatch

2013-11-04 Thread Joe Perches
On Sat, 2013-11-02 at 19:17 +0530, Govindarajulu Varadarajan wrote: Signed-off-by: Govindarajulu Varadarajan govindarajul...@gmail.com [] diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl [] @@ -3719,7 +3719,7 @@ sub process { # check for needless if (foo) fn(foo) uses

Re: USB 3.0 SDR / Ettus Research B200/210

2013-11-04 Thread Moritz Fischer
On Mon, Nov 4, 2013 at 2:21 PM, Greg KH gre...@linuxfoundation.org wrote: Where specifically in the kernel? Busywaiting for data transferrs to complete, or doing something else? Are you keeping the pipeline full to the device? You should be able to hit USB line speed from userspace if you

Re: [PATCH v2 1/1] usb: hcd: move controller wakeup setting initialization to individual driver

2013-11-04 Thread Alan Stern
On Sun, 3 Nov 2013, Peter Chen wrote: diff --git a/drivers/usb/host/ehci-fsl.c b/drivers/usb/host/ehci-fsl.c index a06d501..bf405fd 100644 --- a/drivers/usb/host/ehci-fsl.c +++ b/drivers/usb/host/ehci-fsl.c @@ -139,6 +139,7 @@ static int usb_hcd_fsl_probe(const struct hc_driver

Re: kernel NULL pointer dereference at (null) - inside hub_disconnect

2013-11-04 Thread Alan Stern
On Mon, 4 Nov 2013, Luke-Jr wrote: On Tuesday, October 29, 2013 2:39:14 PM Alan Stern wrote: On Mon, 28 Oct 2013, Luke-Jr wrote: https://bugzilla.kernel.org/show_bug.cgi?id=63961 Kernel version 3.10.15 [1774470.503558] hub 2-1.3:1.0: hub_port_status failed (err = -71)

[PATCH 1/2](fix) Moved mkdir functions, added MAX_LENGTH MAX_PATH_LENGTH

2013-11-04 Thread Stanislaw Wadas
mkdir() function are now called after successfull function creation. Added inline to gadget_write_string(). 256 hard coded value has been replaced by two defined constants MAX_LENGTH and MAX_PATH_LENGTH Change-Id: Ifcfaf9cf95da6558c176ce3367cd553cea54e871 Signed-off-by: Stanislaw Wadas

[PATCH 2/2](fix) added fputs() fgets() error handling

2013-11-04 Thread Stanislaw Wadas
Change-Id: I787380dda981c7cee6508a4ff566d7ca9fb273cf Signed-off-by: Stanislaw Wadas s.wa...@samsung.com --- src/gadget.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/src/gadget.c b/src/gadget.c index faed675..ff4f3d5 100644 --- a/src/gadget.c +++

Re: [PATCH 1/2] Moved mkdir functions, added MAX_LENGHT MAX_PATH_LENGHT

2013-11-04 Thread Stanislaw Wadas
On 04.11.2013 15:10, Andrzej Pietrasiewicz wrote: W dniu 04.11.2013 14:55, Stanislaw Wadas pisze: mkdir() function are now called after successfull function creation. Added inline to gadget_write_string(). 256 hard coded value has been replaced by two defined constants MAX_LENGHT and

Re: USB 3.0 SDR / Ettus Research B200/210

2013-11-04 Thread Alan Stern
On Mon, 4 Nov 2013, Moritz Fischer wrote: Hi guys, This is my first post to this list, so let me quickly introduce myself. I'm a software engineer for Ettus Research (National Instruments). One of our new products [1] is a USB 3.0 (using Cypress FX3) based SDR that currently ships with

Re: kernel NULL pointer dereference at (null) - inside hub_disconnect

2013-11-04 Thread Luke-Jr
On Monday, November 04, 2013 3:07:18 PM Alan Stern wrote: On Mon, 4 Nov 2013, Luke-Jr wrote: On Tuesday, October 29, 2013 2:39:14 PM Alan Stern wrote: On Mon, 28 Oct 2013, Luke-Jr wrote: https://bugzilla.kernel.org/show_bug.cgi?id=63961 Kernel version 3.10.15

[PATCH] usb: xhci: Simplify the way the TRB_CYCLE bit is set on the first trb

2013-11-04 Thread David Laight
Updating the TRB_CYCLE bit in the first trb has to be done after all the subsequent trb have been written. The code to do this was overly complicated. When initially writing the start_trb invert its TRB_CYCLE bit. In giveback_first_trb() invert the TRB_CYCLE bit again. Tested with a USB

Re: Large USB HID transfers

2013-11-04 Thread Cliff Brake
On Fri, Nov 1, 2013 at 5:04 PM, Alan Stern st...@rowland.harvard.edu wrote: On Fri, 1 Nov 2013, Cliff Brake wrote: On Thu, Oct 31, 2013 at 2:25 PM, Alan Stern st...@rowland.harvard.edu wrote: What host controller driver are you using? Its the USB EHCI host in the TI DM3730, so it uses

Re: kernel NULL pointer dereference at (null) - inside hub_disconnect

2013-11-04 Thread Alan Stern
On Mon, 4 Nov 2013, Luke-Jr wrote: Perhaps I phrased that wrong: I suspect this is a hardware issue or firmware bug, triggering bad error handling in Linux. After upgrading to 3.12, Linux indeed survives the hub being plugged in, and the hub is usable, but I still see various errors in

Re: [PATCH 2/2](fix) added fputs() fgets() error handling

2013-11-04 Thread Matt Porter
On Mon, Nov 04, 2013 at 04:09:39PM +0100, Stanislaw Wadas wrote: Change-Id: I787380dda981c7cee6508a4ff566d7ca9fb273cf Signed-off-by: Stanislaw Wadas s.wa...@samsung.com Need a commit log entry for this one. Can you please follow kernel conventions when posting/reposting? Submit the updated

Re: kernel NULL pointer dereference at (null) - inside hub_disconnect

2013-11-04 Thread Luke-Jr
On Monday, November 04, 2013 4:22:04 PM Alan Stern wrote: On Mon, 4 Nov 2013, Luke-Jr wrote: Perhaps I phrased that wrong: I suspect this is a hardware issue or firmware bug, triggering bad error handling in Linux. After upgrading to 3.12, Linux indeed survives the hub being plugged in, and

Re: libusbg clean up and error handling

2013-11-04 Thread Matt Porter
On Mon, Nov 04, 2013 at 02:55:46PM +0100, Stanislaw Wadas wrote: In reference to the message sent by Andrzej Pietrasiewicz (about libusbg (formerly libgadget)) I would like to propose some changes to libusbg. Along the lines in my other reply, it would be preferred if you could send this type

[PATCH] usb: gadget: mass storage: fix return of delayed status

2013-11-04 Thread Pratyush Anand
Mass storage gadget returns DELAYED_STATUS in stead of USB_GADGET_DELAYED_STATUS while handling bulk reset request. Since peripheral driver uses USB_GADGET_DELAYED_STATUS for delayed status handling, therefore replace DELAYED_STATUS by USB_GADGET_DELAYED_STATUS in mass storage driver. Since,

Re: khubd timed out on ep0in len=0/64 with 3.4 kernel

2013-11-04 Thread Prasad Koya
Hi I'm attaching dmesg with old_scheme_first=1 and CONFIG_USB_STORAGE_DEBUG enabled. I compiled code with old_scheme_first set to 1. I see that it doesn't do get descriptor early in the loop so it doesn't run into khubd time out. But as I mentioned earlier usb-storage runs into some sort of error

Re: Large USB HID transfers

2013-11-04 Thread Alan Stern
On Mon, 4 Nov 2013, Cliff Brake wrote: As far as I can tell, the big blockage occurs when the machine has to analyze an input report from which it extracts 767 fields and ends up calling hid_process_event about 2300 times. Since each call takes around 60-80 microseconds, you end up with

Re: CSW endpoint status returned STALL after BOT MSC Reset

2013-11-04 Thread Pratyush Anand
On Sat, Nov 02, 2013 at 04:18:05AM +0800, Paul Zimmerman wrote: From: Alan Stern Sent: Friday, November 01, 2013 7:09 AM Now DELAYED_STATUS isn't used anywhere. You should add the following hunk to this patch: Index: usb-3.12/drivers/usb/gadget/storage_common.c

Re: [PATCH 2/2] added fputs() fgets() error handling

2013-11-04 Thread Daniele Forsi
2013/11/4 Stanislaw Wadas: @@ -127,6 +132,7 @@ static void gadget_write_buf(char *path, char *name, char *file, char *buf) { char p[MAX_LENGHT]; FILE *fp; + int ret; this new variable isn't used in the hunk that follows: sprintf(p, %s/%s/%s, path, name,

Re: [PATCH] usb: musb: dsps: polling ID pin status only in otg mode

2013-11-04 Thread Sebastian Andrzej Siewior
On 10/30/2013 03:38 PM, Bin Liu wrote: Only start the otg_timer in dual role mode; otherwise in peripheral mode when musb is disconnected from the host port, otg_timer starts and continuously toggles the session, which causes VBUS pulse. Hmm. Okay, so one piece of the pulsing should be gone

Re: kernel NULL pointer dereference at (null) - inside hub_disconnect

2013-11-04 Thread Alan Stern
On Mon, 4 Nov 2013, Luke-Jr wrote: On Monday, November 04, 2013 4:22:04 PM Alan Stern wrote: On Mon, 4 Nov 2013, Luke-Jr wrote: Perhaps I phrased that wrong: I suspect this is a hardware issue or firmware bug, triggering bad error handling in Linux. After upgrading to 3.12, Linux

Re: khubd timed out on ep0in len=0/64 with 3.4 kernel

2013-11-04 Thread Alan Stern
On Mon, 4 Nov 2013, Prasad Koya wrote: Hi I'm attaching dmesg with old_scheme_first=1 and CONFIG_USB_STORAGE_DEBUG enabled. I compiled code with old_scheme_first set to 1. I see that it doesn't do get descriptor early in the loop so it doesn't run into khubd time out. But as I mentioned

Re: khubd timed out on ep0in len=0/64 with 3.4 kernel

2013-11-04 Thread Alan Stern
On Mon, 4 Nov 2013, Alan Stern wrote: On Mon, 4 Nov 2013, Prasad Koya wrote: Hi I'm attaching dmesg with old_scheme_first=1 and CONFIG_USB_STORAGE_DEBUG enabled. I compiled code with old_scheme_first set to 1. I see that it doesn't do get descriptor early in the loop so it doesn't

Re: [PATCH v3] kconfig/symbol.c: handle choice_values that depend on 'm' symbols

2013-11-04 Thread Sebastian Andrzej Siewior
* Dirk Gouders | 2013-11-01 00:39:46 [+0100]: This patch sets choice_values' visibility that depend on symbols set to 'm' to 'n' if the corresponding choice is set to 'y'. This makes them disappear from the choice list and will also cause the choice_values' value set to 'n' in sym_calc_value()

Re: [PATCH] usb: musb: dsps: polling ID pin status only in otg mode

2013-11-04 Thread Bin Liu
Sebastian, On Mon, Nov 4, 2013 at 11:13 AM, Sebastian Andrzej Siewior bige...@linutronix.de wrote: On 10/30/2013 03:38 PM, Bin Liu wrote: Only start the otg_timer in dual role mode; otherwise in peripheral mode when musb is disconnected from the host port, otg_timer starts and continuously

Re: [PATCH] usb: musb: dsps: polling ID pin status only in otg mode

2013-11-04 Thread Bin Liu
Sebastian, On Mon, Nov 4, 2013 at 11:40 AM, Bin Liu binml...@gmail.com wrote: Sebastian, On Mon, Nov 4, 2013 at 11:13 AM, Sebastian Andrzej Siewior bige...@linutronix.de wrote: On 10/30/2013 03:38 PM, Bin Liu wrote: Only start the otg_timer in dual role mode; otherwise in peripheral mode

Re: [PATCH] usb: musb: dsps: polling ID pin status only in otg mode

2013-11-04 Thread Sebastian Andrzej Siewior
On 11/04/2013 06:43 PM, Bin Liu wrote: Sebastian, Hi Bin, Sorry, please ignore my message above. I misinterpreted your response. Yeah, probably not easy to get rid of starting otg_timer in dsps_interrupt(), otg mode still needs it. Yes. It may work with the try_idle() thingy that we have in

[PATCH] USB: pl2303: fixed handling of CS5 setting

2013-11-04 Thread Colin Leitner
This patch fixes the CS5 setting on the PL2303 USB-to-serial devices. CS5 has a value of 0 and the CSIZE setting has been skipped altogether by the enclosing if. Tested on 3.11.6 and the scope shows the correct output after the fix has been applied. Tagged to be added to stable, because it fixes

Re: [PATCH] USB: serial: option: blacklist intf1 for Huawei E173s-6

2013-11-04 Thread Sergei Shtylyov
Hello. On 11/04/2013 04:01 PM, Gustavo Zacarias wrote: Interface 1 on this device isn't for option to bind to otherwise an oops on usb_wwan with log flooding will happen: tty_release: ttyUSB1: read/write wait queue active! And it doesn't seem to respond to QMI if it's added to qmi_wwan

Re: [PATCH 2/2] added fputs() fgets() error handling

2013-11-04 Thread Sergei Shtylyov
Hello. On 11/04/2013 04:55 PM, Stanislaw Wadas wrote: Change-Id: I787380dda981c7cee6508a4ff566d7ca9fb273cf This line shouldn't be present in the upstream patches. Signed-off-by: Stanislaw Wadas s.wa...@samsung.com --- src/gadget.c | 12 +++- 1 file changed, 11

Re: [PATCH] USB: pl2303: fixed handling of CS5 setting

2013-11-04 Thread Johan Hovold
On Mon, Nov 04, 2013 at 07:40:43PM +0100, Colin Leitner wrote: This patch fixes the CS5 setting on the PL2303 USB-to-serial devices. CS5 has a value of 0 and the CSIZE setting has been skipped altogether by the enclosing if. Tested on 3.11.6 and the scope shows the correct output after the

Re: [PATCH net-next 02/13] driver: net: remove unnecessary skb NULL check before calling dev_kfree_skb_irq

2013-11-04 Thread David Miller
From: Govindarajulu Varadarajan govindarajul...@gmail.com Date: Sat, 2 Nov 2013 19:17:43 +0530 @@ -1030,10 +1030,8 @@ static void ni65_xmit_intr(struct net_device *dev,int csr0) } #ifdef XMT_VIA_SKB - if(p-tmd_skb[p-tmdlast]) { -

Re: [RFC] Revert sierra_net: keep status interrupt URB active

2013-11-04 Thread Dan Williams
On Fri, 2013-11-01 at 13:53 +0100, Bjørn Mork wrote: This reverts commit 7b0c5f21f348a66de495868b8df0284e8dfd6bbf. It's not easy to create a driver for all the various firmware bugs out there. This change caused regressions for a number of devices, which started to fail link detection and

Re: [PATCH v3] kconfig/symbol.c: handle choice_values that depend on 'm' symbols

2013-11-04 Thread Yann E. MORIN
Sebastian, All, On 2013-11-04 18:27 +0100, Sebastian Andrzej Siewior spake thusly: * Dirk Gouders | 2013-11-01 00:39:46 [+0100]: This patch sets choice_values' visibility that depend on symbols set to 'm' to 'n' if the corresponding choice is set to 'y'. This makes them disappear from the

Re: [PATCH net-next 0/5] cdc_mbim + qmi_wwan trivial fixes

2013-11-04 Thread David Miller
From: Bjørn Mork bj...@mork.no Date: Fri, 1 Nov 2013 14:18:51 +0100 This series fixes three problems Oliver pointed out during the review of the new huawei_cdc_ncm driver: http://patchwork.ozlabs.org/patch/278903/ That innocent driver only used cdc_mbim as a blueprint, and all the blame

Re: [PATCH net v2 2/3] r8152: modify the tx flow

2013-11-04 Thread David Miller
From: hayeswang hayesw...@realtek.com Date: Thu, 31 Oct 2013 13:52:38 +0800 From: David Miller [mailto:da...@davemloft.net] Sent: Thursday, October 31, 2013 5:05 AM From: Hayes Wang hayesw...@realtek.com Date: Wed, 30 Oct 2013 15:13:39 +0800 [...] Basically, your driver will now queue up

Re: [PATCH 13/16] usb/gadget: FunctionFS: convert to new function interface with backward compatibility

2013-11-04 Thread Michal Nazarewicz
On Wed, Oct 23 2013, Andrzej Pietrasiewicz wrote: This is required in order to integrate configfs support. f_fs needs to be a separately compiled module and so it needs to use the new interface. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park

Re: [PATCH 14/16] usb/gadget: g_ffs: convert to new interface of f_fs

2013-11-04 Thread Michal Nazarewicz
On Wed, Oct 23 2013, Andrzej Pietrasiewicz wrote: Prepare for configfs integration. Use the new interface so that f_fs can be made a module. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/Kconfig |

Re: [PATCH 15/16] usb/gadget: FunctionFS: Remove compatibility layer

2013-11-04 Thread Michal Nazarewicz
On Wed, Oct 23 2013, Andrzej Pietrasiewicz wrote: There are no old function interface users left, so the old interface can be removed. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com Acked-by: Michal Nazarewicz

Re: [PATCH 16/16] usb/gadget: FunctionFS: add configfs support

2013-11-04 Thread Michal Nazarewicz
On Wed, Oct 23 2013, Andrzej Pietrasiewicz wrote: Add support for using FunctionFS in configfs-based USB gadgets. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com Acked-by: Michal Nazarewicz min...@mina86.com -- Best regards,

Re: [PATCH 2/2] added fputs() fgets() error handling

2013-11-04 Thread Greg KH
On Mon, Nov 04, 2013 at 02:55:48PM +0100, Stanislaw Wadas wrote: Change-Id: I787380dda981c7cee6508a4ff566d7ca9fb273cf Signed-off-by: Stanislaw Wadas s.wa...@samsung.com --- src/gadget.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/src/gadget.c

[PATCH v4 1/4] usb: gadget: move bitflags to the end of usb_gadget struct

2013-11-04 Thread David Cohen
This patch moves all bitflags to the end of usb_gadget struct in order to improve readability. Signed-off-by: David Cohen david.a.co...@linux.intel.com --- include/linux/usb/gadget.h | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git

[PATCH v4 2/4] usb: gadget: add quirk_ep_out_aligned_size field to struct usb_gadget

2013-11-04 Thread David Cohen
Due to USB controllers may have different restrictions, usb gadget layer needs to provide a generic way to inform gadget functions to complain with non-standard requirements. This patch adds 'quirk_ep_out_aligned_size' field to struct usb_gadget to inform when controller's epout requires buffer

[PATCH v4 4/4] usb: dwc3: add quirk USB_GADGET_QUIRK_EP_OUT_ALIGNED_SIZE to gadget driver

2013-11-04 Thread David Cohen
DWC3 requires epout to have buffer size aligned to MaxPacketSize value. This patch adds necessary quirk for it. Signed-off-by: David Cohen david.a.co...@linux.intel.com --- drivers/usb/dwc3/gadget.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/usb/dwc3/gadget.c

[PATCH v4 3/4] usb: ffs: check quirk to pad epout buf size when not aligned to maxpacketsize

2013-11-04 Thread David Cohen
Check gadget.quirk_ep_out_aligned_size to decide if buffer size requires to be aligned to maxpacketsize of an out endpoint. ffs_epfile_io() needs to pad epout buffer to match above condition if quirk is found. Signed-off-by: David Cohen david.a.co...@linux.intel.com --- drivers/usb/gadget/f_fs.c

[PATCH v4 0/4] add gadget quirk to adapt f_fs for DWC3

2013-11-04 Thread David Cohen
Hi, These patches are a proposal to add gadget quirks in an immediate objective to adapt f_fs when using DWC3 controller. But the quirk solution is generic and can be used by other controllers to adapt gadget functions to their non-standard restrictions. This change is necessary to make

[PATCH v4.1 4/4] usb: dwc3: set gadget's quirk ep_out_align_size

2013-11-04 Thread David Cohen
DWC3 requires epout to have buffer size aligned to MaxPacketSize value. This patch sets necessary quirk for it. Signed-off-by: David Cohen david.a.co...@linux.intel.com --- Changes from v4 to v4.1: - just updated patch's subject. No actual code changed. drivers/usb/dwc3/gadget.c | 6 ++ 1

Re: Large USB HID transfers

2013-11-04 Thread Cliff Brake
On Mon, Nov 4, 2013 at 11:51 AM, Alan Stern st...@rowland.harvard.edu wrote: On Mon, 4 Nov 2013, Cliff Brake wrote: We have the same system running on a older PXA270 version of the product (vs the current TI DM3730). It has an older OHCI 12Mbit USB host controller (vs the current TI EHCI

[PATCH v3 00/11] Add power management support for MXS PHY

2013-11-04 Thread Peter Chen
Hi Felipe Shawn, The serial adds power management support for MXS PHY, it includes: - Add three common PHY APIs, .set_wakeup, .notify_suspend, notify_resume. - Related above API implementation at mxs phy driver - misc changes for mxs phy to support low power mode and wakeup. My github tree:

[PATCH v3 08/11] usb: phy-mxs: Enable IC fixes for related SoCs

2013-11-04 Thread Peter Chen
Some PHY bugs are fixed by IC logic, but these bits are not enabled by default, so we enable them at driver. Signed-off-by: Peter Chen peter.c...@freescale.com --- drivers/usb/phy/phy-mxs-usb.c | 32 ++-- 1 files changed, 30 insertions(+), 2 deletions(-) diff --git

[PATCH v3 01/11] usb: phy-mxs: Add platform judgement code

2013-11-04 Thread Peter Chen
The mxs-phy has several bugs and features at different versions, the driver code can get it through of_device_id.data. Signed-off-by: Peter Chen peter.c...@freescale.com --- drivers/usb/phy/phy-mxs-usb.c | 59 ++-- 1 files changed, 50 insertions(+), 9

[PATCH v3 03/11] usb: doc: phy-mxs: update binding for adding anatop phandle

2013-11-04 Thread Peter Chen
Add anatop phandle which is used to access anatop registers to control PHY's power and other USB operations. Signed-off-by: Peter Chen peter.c...@freescale.com --- Documentation/devicetree/bindings/usb/mxs-phy.txt |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git

[PATCH v3 05/11] usb: phy-mxs: Add anatop regmap

2013-11-04 Thread Peter Chen
It is needed by imx6 SoC serial, but not for imx23 and imx28. Signed-off-by: Peter Chen peter.c...@freescale.com --- drivers/usb/phy/phy-mxs-usb.c | 15 +++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/drivers/usb/phy/phy-mxs-usb.c b/drivers/usb/phy/phy-mxs-usb.c

[PATCH v3 07/11] usb: phy-mxs: Add implementation of nofity_suspend and notify_resume

2013-11-04 Thread Peter Chen
Implementation of notify_suspend and notify_resume will be different according to mxs_phy_platform_flag-flags. Signed-off-by: Peter Chen peter.c...@freescale.com --- drivers/usb/phy/phy-mxs-usb.c | 54 +--- 1 files changed, 50 insertions(+), 4 deletions(-)

[PATCH v3 10/11] usb: phy-mxs: Add implementation of set_wakeup

2013-11-04 Thread Peter Chen
When we need the PHY can be waken up by external signals, we can call this API. Besides, we call mxs_phy_disconnect_line at this API to close the connection between USB PHY and controller, after that, the line state from controller is SE0. Once the PHY is out of power, without calling

[PATCH v3 04/11] ARM: dts: imx6: add anatop phandle for usbphy

2013-11-04 Thread Peter Chen
Add anatop phandle for usbphy Signed-off-by: Peter Chen peter.c...@freescale.com --- arch/arm/boot/dts/imx6qdl.dtsi |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi index 226ce75..bb79678 100644 ---

[PATCH v3 11/11] usb: phy-mxs: Add system suspend/resume API

2013-11-04 Thread Peter Chen
We need this to keep PHY's power on or off during the system suspend mode. If we need to enable USB wakeup, then we must keep PHY's power being on during the system suspend mode. Otherwise, we need to keep PHY's power being off to save power. Signed-off-by: Peter Chen peter.c...@freescale.com ---

[PATCH v3 02/11] usb: phy-mxs: Add auto clock and power setting

2013-11-04 Thread Peter Chen
With the auto setting, the PHY's clock and power can be recovered correctly from low power mode, it is ganranteed by IC logic. Signed-off-by: Peter Chen peter.c...@freescale.com --- drivers/usb/phy/phy-mxs-usb.c | 20 +--- 1 files changed, 17 insertions(+), 3 deletions(-) diff

[PATCH v3 09/11] usb: phy: Add set_wakeup API

2013-11-04 Thread Peter Chen
This API is used to set wakeup enable at PHY registers, in that case, the PHY can be waken up from suspend due to external events, like vbus change, dp/dm change and id change. Signed-off-by: Peter Chen peter.c...@freescale.com --- include/linux/usb/phy.h | 16 1 files

[PATCH v3 00/10] Add power management support for chipidea

2013-11-04 Thread Peter Chen
Hi Greg, This serial adds power management (system runtime) for chipidea core. With this, the chipidea controller can be at low power mode when it is not in use, and the chipidea controller can be the system wakeup source. It needs to depend on my patch[1], since [1]-Add power management support

[PATCH v3 07/10] usb: chipidea: imx: Enable runtime pm support for imx6q

2013-11-04 Thread Peter Chen
Currently, only imx6q adds wakeup logic, so only enable runtime pm for imx6q. Signed-off-by: Peter Chen peter.c...@freescale.com --- drivers/usb/chipidea/ci_hdrc_imx.c | 13 - 1 files changed, 12 insertions(+), 1 deletions(-) diff --git a/drivers/usb/chipidea/ci_hdrc_imx.c

[PATCH v3 06/10] usb: chipidea: imx: call set_wakeup when necessary

2013-11-04 Thread Peter Chen
- Disable wakeup after probe - Enable wakeup during the suspend - Disable wakeup after controller is active Signed-off-by: Peter Chen peter.c...@freescale.com --- drivers/usb/chipidea/ci_hdrc_imx.c | 36 1 files changed, 36 insertions(+), 0 deletions(-)

[PATCH v3 02/10] usb: chipidea: imx: add power management support

2013-11-04 Thread Peter Chen
Add system and runtime power management support for imx gluy layer. Signed-off-by: Peter Chen peter.c...@freescale.com --- drivers/usb/chipidea/ci_hdrc_imx.c | 94 ++- 1 files changed, 91 insertions(+), 3 deletions(-) diff --git

  1   2   >