[PATCH v2 1/1] usb: usleep_range is preferred over udelay where wakeup is flexible

2015-07-06 Thread Sunny Kumar
According to Documentation/timers/timers-howto.txt" udelay() is only called once from a place where sleeping is allowed. We can replace it with a call to usleep_range() with a reasonable upper limit. This is new version of patch created after discussion. Signed-off-by: Sunny Kumar --- drivers/u

RE: [PATCH v2 3/3] usb: dwc3: gadget: return error if command sent to DEPCMD register fails

2015-07-06 Thread Subbaraya Sundeep Bhatta
Hi Felipe, > -Original Message- > From: Felipe Balbi [mailto:ba...@ti.com] > Sent: Monday, July 06, 2015 10:38 PM > To: Subbaraya Sundeep Bhatta > Cc: ba...@ti.com; gre...@linuxfoundation.org; linux-usb@vger.kernel.org; > linux-ker...@vger.kernel.org; sta...@vger.kernel.org > Subject: Re:

Re: [PATCH v2] usb: move assignment out of if condition

2015-07-06 Thread Greg KH
On Thu, Jul 02, 2015 at 06:23:59PM -0400, Kris Borer wrote: > Fix four occurrences of checkpatch.pl error: > > ERROR: do not use assignment in if condition > > Signed-off-by: Kris Borer > --- > drivers/usb/core/hcd.c | 19 +-- > 1 file changed, 13 insertions(+), 6 deletions(-)

Re: [PATCH v2 3/3] usb: dwc3: gadget: return error if command sent to DEPCMD register fails

2015-07-06 Thread Felipe Balbi
Hi, On Tue, Jul 07, 2015 at 02:10:26AM +, John Youn wrote: > On 7/1/2015 8:00 PM, Felipe Balbi wrote: > > On Thu, Jul 02, 2015 at 02:03:14AM +, John Youn wrote: > >> On 6/29/2015 2:48 PM, Felipe Balbi wrote: > >>> Hi again, > >>> > >>> On Mon, Jun 29, 2015 at 04:47:01PM -0500, Felipe Balbi

Re: [PATCH v2 3/3] usb: dwc3: gadget: return error if command sent to DEPCMD register fails

2015-07-06 Thread John Youn
On 7/1/2015 8:00 PM, Felipe Balbi wrote: > On Thu, Jul 02, 2015 at 02:03:14AM +, John Youn wrote: >> On 6/29/2015 2:48 PM, Felipe Balbi wrote: >>> Hi again, >>> >>> On Mon, Jun 29, 2015 at 04:47:01PM -0500, Felipe Balbi wrote: On Thu, May 21, 2015 at 03:46:48PM +0530, Subbaraya Sundeep Bha

Re: [Spice-devel] [RFC PATCH 1/1] Add a usbredir kernel module to remotely connect USB devices over IP.

2015-07-06 Thread Alan Stern
On Mon, 6 Jul 2015, Jeremy White wrote: > Anything else fundamental to usbip that should inform the design of a > usbredir driver? usbip appears to be based off a 2004 vintage of > dummy_hcd. I'll look thoughtfully at the current dummy_hcd; please let > me know if there is anything else I should

Re: [Spice-devel] [RFC PATCH 1/1] Add a usbredir kernel module to remotely connect USB devices over IP.

2015-07-06 Thread Jeremy White
On 07/06/2015 03:20 AM, Oliver Neukum wrote: > On Fri, 2015-07-03 at 10:51 +0200, Krzysztof Opasiak wrote: >> Doesn't we have the same problem with functionfs/gadgetfs and >> dummy_hcd? >> Or with fuse? >> >> It's a very generic problem for all "virtualized devices" and it is >> known for quite a

Re: [REPOST PATCH 3/3] USB: dwc2: Don't turn off the usbphy in suspend if wakeup is enabled

2015-07-06 Thread Doug Anderson
Hi, On Mon, Jul 6, 2015 at 12:02 PM, Felipe Balbi wrote: > On Mon, Jul 06, 2015 at 02:58:16PM -0400, Alan Stern wrote: >> On Mon, 6 Jul 2015, Douglas Anderson wrote: >> >> > If the 'snps,need-phy-for-wake' is set in the device tree then: >> > >> > - We know that we can wakeup, so call device_set_

Re: [REPOST PATCH 3/3] USB: dwc2: Don't turn off the usbphy in suspend if wakeup is enabled

2015-07-06 Thread Felipe Balbi
On Mon, Jul 06, 2015 at 12:32:56PM -0700, Doug Anderson wrote: > Felipe, > > On Mon, Jul 6, 2015 at 11:34 AM, Felipe Balbi wrote: > > Hi, > > > > On Mon, Jul 06, 2015 at 11:27:04AM -0700, Douglas Anderson wrote: > >> @@ -282,6 +296,28 @@ static int dwc2_driver_probe(struct platform_device > >> *

Re: [REPOST PATCH 3/3] USB: dwc2: Don't turn off the usbphy in suspend if wakeup is enabled

2015-07-06 Thread Doug Anderson
Felipe, On Mon, Jul 6, 2015 at 11:34 AM, Felipe Balbi wrote: > Hi, > > On Mon, Jul 06, 2015 at 11:27:04AM -0700, Douglas Anderson wrote: >> @@ -282,6 +296,28 @@ static int dwc2_driver_probe(struct platform_device >> *dev) >> return retval; >> } >> >> +static bool __maybe_unused dwc2_can_p

Re: [REPOST PATCH 3/3] USB: dwc2: Don't turn off the usbphy in suspend if wakeup is enabled

2015-07-06 Thread Felipe Balbi
On Mon, Jul 06, 2015 at 02:58:16PM -0400, Alan Stern wrote: > On Mon, 6 Jul 2015, Douglas Anderson wrote: > > > If the 'snps,need-phy-for-wake' is set in the device tree then: > > > > - We know that we can wakeup, so call device_set_wakeup_capable(). > > The USB core will use this knowledge to

Re: [PATCH 1/5] usb: dwc3: ep0: use _roundup_ to calculate the transfer size

2015-07-06 Thread Felipe Balbi
On Wed, Jun 10, 2015 at 02:48:48PM +0530, Kishon Vijay Abraham I wrote: > No functional change. Used _roundup_ macro to calculate the transfer > size aligned to maxpacket in dwc3_ep0_complete_data. It also makes it > similar to how transfer size is calculated in __dwc3_ep0_do_control_data. > > Si

Re: [REPOST PATCH 3/3] USB: dwc2: Don't turn off the usbphy in suspend if wakeup is enabled

2015-07-06 Thread Alan Stern
On Mon, 6 Jul 2015, Douglas Anderson wrote: > If the 'snps,need-phy-for-wake' is set in the device tree then: > > - We know that we can wakeup, so call device_set_wakeup_capable(). > The USB core will use this knowledge to enable wakeup by default. > - We know that we should keep the PHY on dur

Re: [PATCH v5 1/3] musb: sunxi: Add support for the Allwinner sunxi musb controller

2015-07-06 Thread Felipe Balbi
On Sat, Jun 13, 2015 at 02:42:08PM +0200, Hans de Goede wrote: > This is based on initial code to get the Allwinner sunxi musb controller > supported by Chen-Yu Tsai and Roman Byshko. > > This adds support for the Allwinner sunxi musb controller in both host only > and otg mode. Peripheral only mo

Re: [PATCH v2] usb: gadget: dbgp: Supply config descriptor and accept set config command.

2015-07-06 Thread Felipe Balbi
Hi, On Thu, May 28, 2015 at 12:23:33PM +0200, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > commit 6d1e47017e322b09ee3c0d42e7c05babfc2a4cc7 > Author: Vladimir Serbinenko > Date: Tue May 26 21:35:06 2015 +0200 > > dbgp: Supply config descriptor and accept set config command. > >

Re: [REPOST PATCH 3/3] USB: dwc2: Don't turn off the usbphy in suspend if wakeup is enabled

2015-07-06 Thread Felipe Balbi
Hi, On Mon, Jul 06, 2015 at 11:27:04AM -0700, Douglas Anderson wrote: > @@ -282,6 +296,28 @@ static int dwc2_driver_probe(struct platform_device *dev) > return retval; > } > > +static bool __maybe_unused dwc2_can_poweroff_phy(struct dwc2_hsotg *dwc2) > +{ > + struct usb_device *root_h

[REPOST PATCH 1/3] USB: Export usb_wakeup_enabled_descendants()

2015-07-06 Thread Douglas Anderson
In (e583d9d USB: global suspend and remote wakeup don't mix) we introduced wakeup_enabled_descendants() as a static function. We'd like to use this function in USB controller drivers to know if we should keep the controller on during suspend time, since doing so has a power impact. Signed-off-by:

[REPOST PATCH 0/3] dwc2 patches to allow wakeup on Rockchip rk3288

2015-07-06 Thread Douglas Anderson
This series of patches, together with from Chris Zhong and a dts change allow us to wake up from a USB device on rk3288 boards. The patches were tested on rk3288-jerry in the chromeos-3.14 kernel. The chromeos-3.14 kernel tested included a full set of d

[REPOST PATCH 3/3] USB: dwc2: Don't turn off the usbphy in suspend if wakeup is enabled

2015-07-06 Thread Douglas Anderson
If the 'snps,need-phy-for-wake' is set in the device tree then: - We know that we can wakeup, so call device_set_wakeup_capable(). The USB core will use this knowledge to enable wakeup by default. - We know that we should keep the PHY on during suspend if something on our root hub needs remote

[REPOST PATCH 2/3] Documentation: dt-bindings: Add snps,need-phy-for-wake for dwc2 USB

2015-07-06 Thread Douglas Anderson
Some SoCs with a dwc2 USB controller may need to keep the PHY on to support remote wakeup. Allow specifying this as a device tree property. Signed-off-by: Douglas Anderson --- Documentation/devicetree/bindings/usb/dwc2.txt | 4 1 file changed, 4 insertions(+) diff --git a/Documentation/de

Re: [PATCH 0/3] dwc2 patches to allow wakeup on Rockchip rk3288

2015-07-06 Thread Felipe Balbi
On Mon, Jul 06, 2015 at 11:06:35AM -0700, Doug Anderson wrote: > Felipe, > > On Mon, Jul 6, 2015 at 10:48 AM, Felipe Balbi wrote: > > On Mon, Jun 22, 2015 at 04:52:21PM -0700, Douglas Anderson wrote: > >> This series of patches, together with > >> fro

Re: [PATCH] usb: gadget: nokia: Add mass storage driver to g_nokia

2015-07-06 Thread Felipe Balbi
Hi, On Mon, Jul 06, 2015 at 08:07:25PM +0200, Pali Rohár wrote: > On Monday 06 July 2015 19:44:21 Felipe Balbi wrote: > > On Mon, Jul 06, 2015 at 12:40:49PM -0500, Felipe Balbi wrote: > > > On Mon, Jul 06, 2015 at 01:59:18PM +0200, Pali Rohár wrote: > > > > On Monday 08 June 2015 18:08:19 Felipe B

Re: [PATCH] usb: gadget: nokia: Add mass storage driver to g_nokia

2015-07-06 Thread Pali Rohár
On Monday 06 July 2015 19:44:21 Felipe Balbi wrote: > On Mon, Jul 06, 2015 at 12:40:49PM -0500, Felipe Balbi wrote: > > On Mon, Jul 06, 2015 at 01:59:18PM +0200, Pali Rohár wrote: > > > On Monday 08 June 2015 18:08:19 Felipe Balbi wrote: > > > > On Mon, Jun 08, 2015 at 08:20:05AM +0200, Pali Rohár

Re: [PATCH 0/3] dwc2 patches to allow wakeup on Rockchip rk3288

2015-07-06 Thread Doug Anderson
Felipe, On Mon, Jul 6, 2015 at 10:48 AM, Felipe Balbi wrote: > On Mon, Jun 22, 2015 at 04:52:21PM -0700, Douglas Anderson wrote: >> This series of patches, together with >> from Chris Zhong and a >> dts change allow us to wake up from a USB device on

Re: [PATCH 0/3] dwc2 patches to allow wakeup on Rockchip rk3288

2015-07-06 Thread Felipe Balbi
On Mon, Jun 22, 2015 at 04:52:21PM -0700, Douglas Anderson wrote: > This series of patches, together with > from Chris Zhong and a > dts change allow us to wake up from a USB device on rk3288 boards. > The patches were tested on rk3288-jerry in the chro

Re: [PATCH] usb: gadget: nokia: Add mass storage driver to g_nokia

2015-07-06 Thread Felipe Balbi
On Mon, Jul 06, 2015 at 12:40:49PM -0500, Felipe Balbi wrote: > On Mon, Jul 06, 2015 at 01:59:18PM +0200, Pali Rohár wrote: > > On Monday 08 June 2015 18:08:19 Felipe Balbi wrote: > > > On Mon, Jun 08, 2015 at 08:20:05AM +0200, Pali Rohár wrote: > > > > This patch adds removable mass storage suppor

Re: [PATCH] usb: gadget: nokia: Add mass storage driver to g_nokia

2015-07-06 Thread Felipe Balbi
On Mon, Jul 06, 2015 at 01:59:18PM +0200, Pali Rohár wrote: > On Monday 08 June 2015 18:08:19 Felipe Balbi wrote: > > On Mon, Jun 08, 2015 at 08:20:05AM +0200, Pali Rohár wrote: > > > This patch adds removable mass storage support to g_nokia gadget > > > (for N900). It means that at runtime block d

Re: [PATCH v5 5/5] usb: gadget: udc-core: independent registration of gadgets and gadget drivers

2015-07-06 Thread Felipe Balbi
On Sat, Jun 27, 2015 at 07:47:28PM -0400, Alan Stern wrote: > On Sun, 28 Jun 2015, Ruslan Bilovol wrote: > > > > Weren't you going to replace this loop with a simple list_del()? IIRC, > > > this is the third time I have asked you to make this change. > > > > I understand the improvement that rep

Re: net2280 and UAS not working

2015-07-06 Thread Felipe Balbi
On Mon, Jul 06, 2015 at 06:58:52PM +0200, Paul Jones wrote: > Hi, > > On 05 Jul 2015, at 20:43, Felipe Balbi wrote: > > > Hi, > > > > On Sun, Jul 05, 2015 at 06:56:29PM +0200, Paul Jones wrote: > >> Ricardo, > >> > >> I’m trying to get the 3380 to work in UAS mode on the 4.0.1 and/or > >> 4.1

Re: [PATCH 1/5] usb: dwc3: omap: Replace deprecated API of extcon

2015-07-06 Thread Felipe Balbi
On Wed, Jul 01, 2015 at 01:11:30PM +0900, Chanwoo Choi wrote: > This patch removes the deprecated notifier API of extcon framwork > and then use the new extcon API with the unique id to indicate > the each external connector (USB, USB-HOST). > > Alter deprecated API as following: > - extcon_regist

Re: net2280 and UAS not working

2015-07-06 Thread Ricardo Ribalda Delgado
Hello Paul Thanks for your info, I will not be able to test anything until next week. Regards! -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2 3/3] usb: dwc3: gadget: return error if command sent to DEPCMD register fails

2015-07-06 Thread Felipe Balbi
On Wed, Jul 01, 2015 at 07:29:28AM +, Subbaraya Sundeep Bhatta wrote: > Hi Felipe, > > > -Original Message- > > From: Felipe Balbi [mailto:ba...@ti.com] > > Sent: Tuesday, June 30, 2015 3:17 AM > > To: Subbaraya Sundeep Bhatta > > Cc: ba...@ti.com; gre...@linuxfoundation.org; linux-usb

Re: net2280 and UAS not working

2015-07-06 Thread Paul Jones
Hi, On 05 Jul 2015, at 20:43, Felipe Balbi wrote: > Hi, > > On Sun, Jul 05, 2015 at 06:56:29PM +0200, Paul Jones wrote: >> Ricardo, >> >> I’m trying to get the 3380 to work in UAS mode on the 4.0.1 and/or 4.1.0-rc8 >> kernels. >> I’m using the following script (derived from Sebastian’s post):

Re: Data loss with usb dvb tuners that is "fixed" by spinning a cpu - how to debug?

2015-07-06 Thread Alan Stern
On Sun, 5 Jul 2015, Andy Furniss wrote: > Alan Stern wrote: > > >> Is 0u the wrong thing to do? > > > > It depends on what you want. If you want complete details (all the > > packets, all the data) then you need to use the binary interface, > > i.e., wireshark or dumpcap. > > > > But if it helps

Re: [PATCH] usb-storage: ignore ZTE MF 823 card reader in mode 0x1225

2015-07-06 Thread Lars Melin
On 2015-07-06 18:12, Oliver Neukum wrote: This device automatically switches itself to another mode (0x1405) unless the specific access pattern of Windows is followed in its initial mode. That makes a dirty unmount of the internal storage devices inevitable if they are mounted. So the card reader

Re: bug#41752 on kernel.org's bugzilla

2015-07-06 Thread Alan Stern
On Mon, 6 Jul 2015, Oliver Neukum wrote: > Hi, > > there was an old patch from Sarah: > https://bugzilla.kernel.org/show_bug.cgi?id=41752 > > drivers/usb/core/hub.c | 12 +++- > 1 files changed, 11 insertions(+), 1 deletions(-) > --- a/drivers/usb/core/hub.c > +++ a/drivers/usb/core

Re: [PATCH 1/1] usb: gadget: composite: only control read may need zero length packet

2015-07-06 Thread Alan Stern
On Mon, 6 Jul 2015, Peter Chen wrote: > On Fri, Jul 03, 2015 at 10:03:22AM -0400, Alan Stern wrote: > > On Fri, 3 Jul 2015, Peter Chen wrote: > > > > > According to the USB 2.0 Spec CH5.5.3 and CH8.5.3.2, only control > > > read (IN) needs zero length packet. > > > > > > Cc: Alan Stern > > > Si

[PATCH 1/2] extcon: fix hang and extcon_get/set_cable_state().

2015-07-06 Thread Roger Quadros
Users of find_cable_index_by_name() will cause a kernel hang as the while loop counter is never incremented and end condition is never reached. extcon_get_cable_state() and extcon_set_cable_state() are broken because they use cable index instead of cable id. This causes the first cable state (cabl

[PATCH 2/2] extcon: Fix extcon_cable_get_state() from getting old state after notification

2015-07-06 Thread Roger Quadros
Currently the extcon code notifiers the interested listeners before it updates the extcon state with the new state. This will cause the listeners that use extcon_cable_get_state() to get the stale state and loose the new state. Fix this by first changing the extcon state variable and then notifyin

[PATCH 0/2] extcon: fixes for v4.2-rc1

2015-07-06 Thread Roger Quadros
Hi, First patch fixes kernel hang and incorrect states when extcon_get/set_cable_state() is used. Second patch fixes extcon_cable_get_state() users from getting old state after notifier callback. cheers, -roger Roger Quadros (2): extcon: fix hang and extcon_get/set_cable_state(). extcon: Fi

[PATCH] USB: serial: ftdi_sio: Fix broken URL in comment

2015-07-06 Thread Nik Nyby
This fixes a typo in the URL: http://zeitcontrol.de/ Signed-off-by: Nik Nyby --- drivers/usb/serial/ftdi_sio_ids.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/serial/ftdi_sio_ids.h b/drivers/usb/serial/ftdi_sio_ids.h index 792e054..cb12b31 100644 --- a/driver

bug#41752 on kernel.org's bugzilla

2015-07-06 Thread Oliver Neukum
Hi, there was an old patch from Sarah: https://bugzilla.kernel.org/show_bug.cgi?id=41752 drivers/usb/core/hub.c | 12 +++- 1 files changed, 11 insertions(+), 1 deletions(-) --- a/drivers/usb/core/hub.c +++ a/drivers/usb/core/hub.c @@ -2033,6 +2033,8 @@ static unsigned hub_is_wu

Re: [PATCH] extcon: Fix kernel hang for find_cable_index_by_name() users

2015-07-06 Thread Roger Quadros
Hi, On 06/07/15 15:31, Roger Quadros wrote: Users of find_cable_index_by_name() will cause a kernel hang as the while loop counter is never incremented and end condition is never reached. Fixes: commit 73b6ecdb93e8 ("extcon: Redefine the unique id of supported external connectors without 'enum

Re: PROBLEM: ch341 module fails with EPROTO when I plug-in Arduino board

2015-07-06 Thread Peter Stuge
Johan Hovold wrote: > On Sat, Jul 04, 2015 at 08:03:57PM +0300, Evgen Druzhynin wrote: > > [1.] One line summary of the problem: > > ch341 module fails on 4.1.1 kernel > > > > [2.] Full description of the problem/report: > > It's not possible to work with Arduino board after installing 4.1.1. > >

Re: [PATCH 1/2] usb: chipidea: Reduce ULPI PHY reset pulse to datasheet spec of 1us

2015-07-06 Thread Mike Looijmans
On 02-07-15 11:39, David Laight wrote: From: Peter Chen Sent: 30 June 2015 03:06 On Fri, Jun 26, 2015 at 03:47:03PM +0200, Mike Looijmans wrote: The datasheet for the 334x PHY mentions that a reset can be performed: "... by bringing the pin low for a minimum of 1 microsecond and then high." A d

[PATCH] extcon: Fix kernel hang for find_cable_index_by_name() users

2015-07-06 Thread Roger Quadros
Users of find_cable_index_by_name() will cause a kernel hang as the while loop counter is never incremented and end condition is never reached. Fixes: commit 73b6ecdb93e8 ("extcon: Redefine the unique id of supported external connectors without 'enum extcon' type") Cc: Greg Kroah-Hartman Signed-

Re: [PATCH] usb: gadget: nokia: Add mass storage driver to g_nokia

2015-07-06 Thread Pali Rohár
On Monday 08 June 2015 18:08:19 Felipe Balbi wrote: > On Mon, Jun 08, 2015 at 08:20:05AM +0200, Pali Rohár wrote: > > This patch adds removable mass storage support to g_nokia gadget > > (for N900). It means that at runtime block device can be exported > > or unexported. So it does not export anyth

Re: [PATCH V2] cdc_ncm: Add support for moving NDP to end of NCM frame

2015-07-06 Thread Enrico Mioso
Just to be clear - this happens on the real machine as well, but here the trace is difficult to extract, because even with the help of someone seeing the screen, I noticed the screen doesn't get updated. I am using vesa right now. -- To unsubscribe from this list: send the line "unsubscribe linu

Re: [PATCH V2] cdc_ncm: Add support for moving NDP to end of NCM frame

2015-07-06 Thread Enrico Mioso
Sure Oliver! Here it is. And - I tried with various approach. I tired also kzallocating the needed memory inside the tx_fixup function using the GFP_ATOMIC flag due to the fact I am in an interrupt handler. At some point, the problem started manifesting in a memset call that whasn't in my patc

Re: [PATCH V2 1/1] usb:serial:f81534 Add F81532/534 Driver

2015-07-06 Thread Johan Hovold
On Thu, Jun 25, 2015 at 10:06:07AM +0200, Johan Hovold wrote: > Hi Peter, > > On Thu, Jun 25, 2015 at 01:16:56PM +0800, Peter Hung wrote: > > Hello Johan, > > > > Peter Hung 於 2015/6/15 上午 09:54 寫道: > > > This driver is for Fintek F81532/F81534 USB to Serial Ports IC. > > > > > > Features: > > >

Re: [PATCH] usb-storage: ignore ZTE MF 823 in mode 0x1225

2015-07-06 Thread Oliver Neukum
On Thu, 2015-07-02 at 20:33 +0700, Lars Melin wrote: > Yes but there are other 19d2:1225 devices which don't have dual-mode > (19d2:1403 RNDIS or 19d2:1405 ECM) they do only have one of the modes > and they don't auto-flip so they need usb_modeswitch. > > This is the first 3G dongle I have seen

[PATCH] usb-storage: ignore ZTE MF 823 card reader in mode 0x1225

2015-07-06 Thread Oliver Neukum
This device automatically switches itself to another mode (0x1405) unless the specific access pattern of Windows is followed in its initial mode. That makes a dirty unmount of the internal storage devices inevitable if they are mounted. So the card reader of such a device should be ignored, lest an

Re: [PATCH] USB-mxuport: Delete an unnecessary check before the function call "release_firmware"

2015-07-06 Thread Johan Hovold
On Sun, Jun 28, 2015 at 03:09:00PM +0200, SF Markus Elfring wrote: > From: Markus Elfring > Date: Sun, 28 Jun 2015 14:59:04 +0200 > > The release_firmware() function tests whether its argument is NULL and then > returns immediately. Thus the test around the call is not needed. > > This issue was

Re: [PATCH V2] cdc_ncm: Add support for moving NDP to end of NCM frame

2015-07-06 Thread Oliver Neukum
On Sun, 2015-07-05 at 12:21 +0200, Enrico Mioso wrote: > When sending lots of small packets, this patch will generate an "Unable to > handle kernel paging request" in the memset call: > ndp16 = memset(ctx->delayed_ndp16, 0, ctx->max_ndp_size); > And I don't know why. > Any comment or suggestion wo

Re: PROBLEM: ch341 module fails with EPROTO when I plug-in Arduino board

2015-07-06 Thread Johan Hovold
On Sat, Jul 04, 2015 at 08:03:57PM +0300, Evgen Druzhynin wrote: > [1.] One line summary of the problem: > ch341 module fails on 4.1.1 kernel > > [2.] Full description of the problem/report: > It's not possible to work with Arduino board after installing 4.1.1. > ch341 fails and does not create /d

[PATCH gpio-for-next 08/10] usb: dwc3: pci: make better use of gpiod API

2015-07-06 Thread Uwe Kleine-König
Since 39b2bbe3d715 (gpio: add flags argument to gpiod_get*() functions) which appeared in v3.17-rc1, the gpiod_get* functions take an additional parameter that allows to specify direction and initial value for output. Use this additional parameter and the _optional variant to simplify the driver a

[PATCH gpio-for-next 09/10] usb: pass flags parameter to gpiod_get functions

2015-07-06 Thread Uwe Kleine-König
Since 39b2bbe3d715 (gpio: add flags argument to gpiod_get*() functions) which appeared in v3.17-rc1, the gpiod_get* functions take an additional parameter that allows to specify direction and initial value for output. Currently this parameter is made optional with the help of a cpp trick. To allow

[PATCH gpio-for-next 07/10] phy: tusb1210: make better use of gpiod API

2015-07-06 Thread Uwe Kleine-König
Since 39b2bbe3d715 (gpio: add flags argument to gpiod_get*() functions) which appeared in v3.17-rc1, the gpiod_get* functions take an additional parameter that allows to specify direction and initial value for output. Furthermore there is devm_gpiod_get_optional which is designed to get optional g

[PULL gpio-for-next] gpio: make flags mandatory for gpiod_get functions

2015-07-06 Thread Uwe Kleine-König
Hello, now that all patches that were in next hit Linus Torvalds' tree and v4.2-rc1 is out here comes the promised pull request that makes usage of the flags parameter mandatory for gpiod_get et al: The following changes since commit d770e558e21961ad6cfdf0ff7df0eb5d7d4f0754: Linux 4.2-rc1 (201

Re: [Spice-devel] [RFC PATCH 1/1] Add a usbredir kernel module to remotely connect USB devices over IP.

2015-07-06 Thread Oliver Neukum
On Fri, 2015-07-03 at 10:51 +0200, Krzysztof Opasiak wrote: > Doesn't we have the same problem with functionfs/gadgetfs and > dummy_hcd? > Or with fuse? > > It's a very generic problem for all "virtualized devices" and it is > known for quite a long time. This is why many tutorials about swap >

Re: [PATCH v3] usb: renesas_usbhs: Allow an OTG PHY driver to provide VBUS

2015-07-06 Thread Laurent Pinchart
Hi Phil, Thank you for the patch. On Thursday 02 July 2015 11:26:33 Phil Edworthy wrote: > These changes allow a PHY driver to trigger a VBUS interrupt and > to provide the value of VBUS. > > Signed-off-by: Phil Edworthy > > --- > v3: > - Changed how indirect vbus is plumbed in. > - Remove

RE: [PATCH v3] usb: renesas_usbhs: Allow an OTG PHY driver to provide VBUS

2015-07-06 Thread Yoshihiro Shimoda
Hi Phil-san, > Sent: Thursday, July 02, 2015 7:27 PM > > These changes allow a PHY driver to trigger a VBUS interrupt and > to provide the value of VBUS. > > Signed-off-by: Phil Edworthy Thank you for the patch! However, if I tested this patch, a kernel panic happened when I did "rmmod g_mass

RE: [PATCH v2] phy: rcar-gen2 usb: Add Host/Function switching for USB0

2015-07-06 Thread Yoshihiro Shimoda
Hi Phil-san, Thank you very much for the patch! > Sent: Thursday, July 02, 2015 5:06 PM < snip > > +/* VBUS change IRQ handler */ > +static irqreturn_t gpio_vbus_irq(int irq, void *data) > +{ > + struct rcar_gen2_channel *channel = data; > + > + /* Wait 20ms before doing anything as VBUS