Re: [PATCH] usb: chipidea: udc: clear vbus_active flag in udc_stop

2015-10-20 Thread Jiada Wang
Hi On 10/20/2015 03:28 PM, Peter Chen wrote: On Tue, Oct 20, 2015 at 03:09:02PM +0900, Jiada Wang wrote: Hi On 10/20/2015 03:01 PM, Peter Chen wrote: On Tue, Oct 20, 2015 at 11:29:18AM +0900, Jiada Wang wrote: Currently in udc_stop, if vbus_active flag is true, all USB activities will be

Re: [PATCH] usb: chipidea: udc: clear vbus_active flag in udc_stop

2015-10-20 Thread Peter Chen
On Tue, Oct 20, 2015 at 03:09:02PM +0900, Jiada Wang wrote: > Hi > > On 10/20/2015 03:01 PM, Peter Chen wrote: > >On Tue, Oct 20, 2015 at 11:29:18AM +0900, Jiada Wang wrote: > >>Currently in udc_stop, if vbus_active flag is true, all USB activities > >>will be stopped, but vbus_active flag is

Re: [PATCH] usb: chipidea: udc: clear vbus_active flag in udc_stop

2015-10-20 Thread Peter Chen
On Tue, Oct 20, 2015 at 03:33:52PM +0900, Jiada Wang wrote: > Hi > > On 10/20/2015 03:28 PM, Peter Chen wrote: > >On Tue, Oct 20, 2015 at 03:09:02PM +0900, Jiada Wang wrote: > >>Hi > >> > >>On 10/20/2015 03:01 PM, Peter Chen wrote: > >>>On Tue, Oct 20, 2015 at 11:29:18AM +0900, Jiada Wang wrote:

[PATCH v5 2/2] usb: chipidea: imx: refine clock operations to adapt for all platforms

2015-10-20 Thread Peter Chen
Some i.mx platforms need three clocks to let controller work, but others only need one, refine clock operation to adapt for all platforms, it fixes a regression found at i.mx27. Signed-off-by: Peter Chen Tested-by: Fabio Estevam Cc:

[PATCH v5 1/2] ARM: dts: imx27.dtsi: change the clock information for usb

2015-10-20 Thread Peter Chen
For imx27, it needs three clocks to let the controller work, the old code is wrong, and usbmisc has not included clock handling code any more. Without this patch, it will cause below data abort when accessing usbmisc registers. usbcore: registered new interface driver usb-storage Unhandled fault:

[PATCH v5 0/2] usb: change clock information for chipidea

2015-10-20 Thread Peter Chen
This patch set changes usb clock information for legacy i.mx platforms. At these platforms, they needs three clocks to let controller work. Fabio, would you please help to test again, I am sorry to let you test again. Changes for v5: - Using one line code for two devm_clk_get error message -

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Rafael J. Wysocki
On Monday, October 19, 2015 05:58:40 PM Rob Herring wrote: > On Mon, Oct 19, 2015 at 4:40 PM, Rafael J. Wysocki wrote: > > On Monday, October 19, 2015 10:58:25 AM Rob Herring wrote: > >> On Mon, Oct 19, 2015 at 10:29 AM, David Woodhouse > >> wrote: > >>

Re: [PATCH] usb: chipidea: udc: clear vbus_active flag in udc_stop

2015-10-20 Thread Jiada Wang
Hi On 10/20/2015 03:01 PM, Peter Chen wrote: On Tue, Oct 20, 2015 at 11:29:18AM +0900, Jiada Wang wrote: Currently in udc_stop, if vbus_active flag is true, all USB activities will be stopped, but vbus_active flag is still left to be true, this causes issue, when afterwards driver tries to

Re: [PATCH] usb: chipidea: udc: clear vbus_active flag in udc_stop

2015-10-20 Thread Peter Chen
On Tue, Oct 20, 2015 at 11:29:18AM +0900, Jiada Wang wrote: > Currently in udc_stop, if vbus_active flag is true, all USB activities > will be stopped, but vbus_active flag is still left to be true, > this causes issue, when afterwards driver tries to connect gadget > device to host, But due to

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread David Woodhouse
On Mon, 2015-10-19 at 16:43 +0100, Russell King - ARM Linux wrote: > On Mon, Oct 19, 2015 at 04:29:40PM +0100, David Woodhouse wrote: > > I don't know that there *is* a coherent plan here to address it > > all. > > > > Certainly, we *will* need subsystems to have firmware-specific > > knowledge

[PATCH 05/15] usb: gadget: tcm: split string definitions into function and device

2015-10-20 Thread Andrzej Pietrasiewicz
Prepare for factoring out f_tcm from a legacy gadget. Signed-off-by: Andrzej Pietrasiewicz --- drivers/usb/gadget/legacy/tcm_usb_gadget.c | 25 + drivers/usb/gadget/legacy/tcm_usb_gadget.h | 3 +-- 2 files changed, 22 insertions(+), 6 deletions(-)

[PATCH 12/15] usb: gadget: f_tcm: remove compatibility layer

2015-10-20 Thread Andrzej Pietrasiewicz
There are no old function interface users left. Signed-off-by: Andrzej Pietrasiewicz --- drivers/usb/gadget/function/f_tcm.c | 87 +++-- 1 file changed, 6 insertions(+), 81 deletions(-) diff --git a/drivers/usb/gadget/function/f_tcm.c

[PATCH 09/15] usb: gadget: tcm: factor out f_tcm

2015-10-20 Thread Andrzej Pietrasiewicz
Prepare for converting tcm to new function registration interface. Signed-off-by: Andrzej Pietrasiewicz --- drivers/usb/gadget/function/f_tcm.c| 2168 drivers/usb/gadget/function/tcm.h | 132 ++

[PATCH 14/15] usb: gadget: f_tcm: use usb_gstrings_attach

2015-10-20 Thread Andrzej Pietrasiewicz
Do not directly use file static strings definitions in instances of f_tcm. Instead use usb_gstrings_attach. Signed-off-by: Andrzej Pietrasiewicz --- drivers/usb/gadget/function/f_tcm.c | 18 +++--- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git

[PATCH 11/15] usb: gadget: tcm: convert to use new function registration interface

2015-10-20 Thread Andrzej Pietrasiewicz
Convert the only user of old tcm function interface so that the old interface can be removed. Signed-off-by: Andrzej Pietrasiewicz --- drivers/usb/gadget/legacy/Kconfig | 1 + drivers/usb/gadget/legacy/tcm_usb_gadget.c | 61 +- 2

Re: [PATCH 00/15] Equivalent of tcm_usb_gadget with configfs

2015-10-20 Thread Sebastian Andrzej Siewior
On 10/20/2015 02:32 PM, Andrzej Pietrasiewicz wrote: > Dear All, > > This series adds support to tcm usb gadget for composing it with configfs. oh great. With this you could have two tcm functions right? (that was one of the limitations of the legacy gadget due to the API we had). I will try to

Re: [PATCH 00/15] Equivalent of tcm_usb_gadget with configfs

2015-10-20 Thread Andrzej Pietrasiewicz
W dniu 20.10.2015 o 14:40, Sebastian Andrzej Siewior pisze: On 10/20/2015 02:32 PM, Andrzej Pietrasiewicz wrote: Dear All, This series adds support to tcm usb gadget for composing it with configfs. oh great. With this you could have two tcm functions right? (that was one of the limitations

Re: [PATCH 1/2] USB: serial: cp210x: Implement 16-bit register access functions

2015-10-20 Thread Konstantin Shkolnyy
On Tue, Oct 20, 2015 at 1:45 AM, Oliver Neukum wrote: > On Mon, 2015-10-19 at 17:01 -0500, Konstantin Shkolnyy wrote: [...] >> +static int cp210x_read_u16_reg(struct usb_serial *serial, u8 req, u16 *pval) >> +{ >> + struct cp210x_serial_private *spriv =

Re: [PATCH 1/2] USB: serial: cp210x: Implement 16-bit register access functions

2015-10-20 Thread Konstantin Shkolnyy
On Tue, Oct 20, 2015 at 2:45 AM, Johan Hovold wrote: [...] > Instead of adding the new helpers to read u16 as a prerequisite for > fixing the broken cp2108 support, just reuse the current config register > helpers for now (in order to keep the fixes minimal and potentially >

[PATCH 02/15] tcm_usb_gadget: Don't strip off nexus WWPN prefix

2015-10-20 Thread Andrzej Pietrasiewicz
From: Nicholas Bellinger Avoid stripping off the 'naa.' I_T nexus prefix from configfs attribute store input, so that user-space will get back what it originaly wrote into ../usb_gadget/$WWPN/$TPGT/nexus. Note the SCSI initiator WWPN is purely symbolic for UAS + BOT, so it

[PATCH 10/15] usb: gadget: f_tcm: convert to new function interface with backward compatibility

2015-10-20 Thread Andrzej Pietrasiewicz
Converting tcm to the new function interface requires converting USB tcm's function code and its users. This patch converts the f_tcm.c to the new function interface. The file can be now compiled into a separate module usb_f_tcm.ko. The old function interface is provided by means of

[PATCH 00/15] Equivalent of tcm_usb_gadget with configfs

2015-10-20 Thread Andrzej Pietrasiewicz
Dear All, This series adds support to tcm usb gadget for composing it with configfs. @target-devel folks: You might be wondering why add configfs for something which already supports configfs. In tcm_usb_gadget configfs has beeen used for configuring the SCSI target part, but the usb gadget part

[PATCH 15/15] usb: gadget: f_tcm: add configfs support

2015-10-20 Thread Andrzej Pietrasiewicz
Allow using the tcm function as a component of a gadget composed with ConfigFS. Signed-off-by: Andrzej Pietrasiewicz --- Documentation/ABI/testing/configfs-usb-gadget-tcm | 6 ++ drivers/usb/gadget/Kconfig| 14 +

[PATCH 01/15] fs: configfs: Add unlocked version of configfs_depend_item()

2015-10-20 Thread Andrzej Pietrasiewicz
From: Krzysztof Opasiak This change is necessary for the SCSI target usb gadget composed with configfs. In this case configfs will be used for two different purposes: to compose a usb gadget and to configure the target part. If an instance of tcm function is created in

[PATCH 07/15] usb: gadget: tcm: use strtobool for a boolean value

2015-10-20 Thread Andrzej Pietrasiewicz
Simplify the function. Signed-off-by: Andrzej Pietrasiewicz --- drivers/usb/gadget/legacy/tcm_usb_gadget.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/usb/gadget/legacy/tcm_usb_gadget.c

[PATCH 04/15] tcm_usb_gadget: Fix enabled attribute failure

2015-10-20 Thread Andrzej Pietrasiewicz
From: Nicholas Bellinger Fix up tcm_usbg_tpg_store_enable() return value to propagate usbg_attach() failure up to user-space if no HDC is found. Reported-by: Andrzej Pietrasiewicz Cc: Andrzej Pietrasiewicz Cc: Sebastian

[PATCH 08/15] usb: gadget: tcm: simplify attribute store function

2015-10-20 Thread Andrzej Pietrasiewicz
Simplify function code. Signed-off-by: Andrzej Pietrasiewicz --- drivers/usb/gadget/legacy/tcm_usb_gadget.c | 22 +++--- 1 file changed, 7 insertions(+), 15 deletions(-) diff --git a/drivers/usb/gadget/legacy/tcm_usb_gadget.c

[PATCH 06/15] usb: gadget: tcm: follow naming conventions

2015-10-20 Thread Andrzej Pietrasiewicz
Prepare for splitting tcm_usb_gadget into legacy gadget proper and f_tcm. Signed-off-by: Andrzej Pietrasiewicz --- drivers/usb/gadget/legacy/tcm_usb_gadget.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git

[PATCH 03/15] tcm_usb_gadget: Fix nexus leak

2015-10-20 Thread Andrzej Pietrasiewicz
From: Nicholas Bellinger This patch adds the missing tcm_usbg_drop_nexus() to properly release tcm_usbg_nexus memory during typical ->fabric_drop_tpg() callback shutdown. Reported-by: Andrzej Pietrasiewicz Cc: Andrzej Pietrasiewicz

[PATCH 13/15] usb: gadget: f_tcm: remove redundant singleton

2015-10-20 Thread Andrzej Pietrasiewicz
The only instance is guaranteed with TPG_INSTANCES defined to 1. Signed-off-by: Andrzej Pietrasiewicz --- drivers/usb/gadget/function/f_tcm.c | 9 - drivers/usb/gadget/function/tcm.h | 2 -- 2 files changed, 11 deletions(-) diff --git

Re: [PATCH 1/2] USB: serial: cp210x: Implement 16-bit register access functions

2015-10-20 Thread Oliver Neukum
On Mon, 2015-10-19 at 17:01 -0500, Konstantin Shkolnyy wrote: > Existing register access functions cp210x_get_config and cp210x_set_config > are cumbersome to use. This change introduces new functions specifically > for 16-bit registers that read and write simple u16 values. Hi, I am afraid

Re: [PATCH 1/2] USB: serial: cp210x: Implement 16-bit register access functions

2015-10-20 Thread Johan Hovold
On Mon, Oct 19, 2015 at 05:01:24PM -0500, Konstantin Shkolnyy wrote: > Existing register access functions cp210x_get_config and cp210x_set_config > are cumbersome to use. This change introduces new functions specifically > for 16-bit registers that read and write simple u16 values. There's a bit

Re: [PATCH] usb: chipidea: set unique names to requested irqs

2015-10-20 Thread Peter Chen
On Mon, Oct 19, 2015 at 07:00:26PM +0300, Vladimir Zapolskiy wrote: > This change allows to get a driver instance of usb controller, which > registers an irq, now the interrupt names are unique: > > # cat /proc/interrupts | grep ci_ > 72: 0 0 0 0GIC 72

Re: [PATCH v9 4/5] xhci: mediatek: support MTK xHCI host controller

2015-10-20 Thread chunfeng yun
hi, On Mon, 2015-10-19 at 14:25 +0300, Mathias Nyman wrote: > >> > >> So basically we are trying to use as many microframes as possible with as > >> few packets > >> per microframe as possible. > >> > >> Did I understand this correctly? > > Yes, you are right. > > > >> How will devices react if

Re: [PATCH 1/1] usb: chipidea: otg: don't wait vbus drops below BSV when starts host

2015-10-20 Thread Peter Chen
On Fri, Oct 16, 2015 at 01:53:20PM +0800, Li Jun wrote: > Some HW design may use ID pin state to control vbus for otg port, > so before host role start, the vbus is already turned on, in this > case, we do not need wait vbus dropping below BSV. > Yes, only gadget need to consider vbus. >

Re: [PATCH 2/2] USB: serial: cp210x: Workaround for occasional cp2108 hangs.

2015-10-20 Thread Johan Hovold
On Mon, Oct 19, 2015 at 05:01:40PM -0500, Konstantin Shkolnyy wrote: > This patch can only be applied after the patch titled > "USB: serial: cp210x: Implement 16-bit register access functions" No need to put this in the commit message, ordering is assumed to be correct when you submit a series of

Re: [PATCH v9 4/5] xhci: mediatek: support MTK xHCI host controller

2015-10-20 Thread Mathias Nyman
On 20.10.2015 09:29, chunfeng yun wrote: hi, On Mon, 2015-10-19 at 14:25 +0300, Mathias Nyman wrote: So basically we are trying to use as many microframes as possible with as few packets per microframe as possible. Did I understand this correctly? Yes, you are right. How will devices

Re: [PATCH 1/2] USB: serial: cp210x: Implement 16-bit register access functions

2015-10-20 Thread Konstantin Shkolnyy
On Tue, Oct 20, 2015 at 8:02 AM, Johan Hovold wrote: > On Tue, Oct 20, 2015 at 07:52:31AM -0500, Konstantin Shkolnyy wrote: >> On Tue, Oct 20, 2015 at 2:45 AM, Johan Hovold wrote: >> [...] >> > Instead of adding the new helpers to read u16 as a prerequisite

Fwd: linux-image-4.3.0-rc5-amd64 fails to attach usb storage device

2015-10-20 Thread Jos van Wolput
Forwarded Message Subject: linux-image-4.3.0-rc5-amd64 fails to attach usb storage device Date: Tue, 20 Oct 2015 12:01:41 +0800 From: Jos van Wolput Reply-To: wol...@on.nl To: Debian Bug Tracking System Package:

Fwd: [Bug 106321] linux-kernel-4.3-rc5 fails to attach usb storage device

2015-10-20 Thread Jos van Wolput
Forwarded Message Subject: [Bug 106321] linux-kernel-4.3-rc5 fails to attach usb storage device Date: Tue, 20 Oct 2015 13:48:54 + From: bugzilla-dae...@bugzilla.kernel.org To: wol...@on.nl https://bugzilla.kernel.org/show_bug.cgi?id=106321 --- Comment #1 from Greg

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Alan Stern
On Tue, 20 Oct 2015, Rob Herring wrote: > > The probe ordering is not the entire picture, though. > > > > Even if you get the probe ordering right, the problem is going to show up in > > multiple other places: system suspend/resume, runtime PM, system shutdown, > > unbinding of drivers. In all

Re: Fwd: linux-image-4.3.0-rc5-amd64 fails to attach usb storage device

2015-10-20 Thread Alan Stern
On Tue, 20 Oct 2015, Jos van Wolput wrote: > Forwarded Message > Subject: linux-image-4.3.0-rc5-amd64 fails to attach usb storage device > Date: Tue, 20 Oct 2015 12:01:41 +0800 > From: Jos van Wolput > Reply-To: wol...@on.nl > To: Debian Bug Tracking

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Rob Herring
On Tue, Oct 20, 2015 at 2:56 AM, Rafael J. Wysocki wrote: > On Monday, October 19, 2015 05:58:40 PM Rob Herring wrote: >> On Mon, Oct 19, 2015 at 4:40 PM, Rafael J. Wysocki >> wrote: >> > On Monday, October 19, 2015 10:58:25 AM Rob Herring wrote: >> >> On

Re: [PATCH 1/2] USB: serial: cp210x: Implement 16-bit register access functions

2015-10-20 Thread Konstantin Shkolnyy
On Tue, Oct 20, 2015 at 11:22 AM, Johan Hovold wrote: > On Tue, Oct 20, 2015 at 09:19:05AM -0500, Konstantin Shkolnyy wrote: >> On Tue, Oct 20, 2015 at 8:02 AM, Johan Hovold wrote: >> > On Tue, Oct 20, 2015 at 07:52:31AM -0500, Konstantin Shkolnyy wrote: >> >>

[PATCH 3/3] uwb: rsv: Use setup_timer

2015-10-20 Thread Muhammad Falak R Wani
Use timer API function setup_timer instead of init_timer to initialize the timer. Signed-off-by: Muhammad Falak R Wani --- drivers/uwb/rsv.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/uwb/rsv.c b/drivers/uwb/rsv.c index

[PATCH 2/3] uwb: neh: Use setup_timer

2015-10-20 Thread Muhammad Falak R Wani
Use timer API function setup_timer instead of init_timer to initialize the timer. Signed-off-by: Muhammad Falak R Wani --- drivers/uwb/neh.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/uwb/neh.c b/drivers/uwb/neh.c index 8cb71bb..36b5cb6

Re: Alternative approach to solve the deferred probe (was: [GIT PULL] On-demand device probing)

2015-10-20 Thread Russell King - ARM Linux
On Mon, Oct 19, 2015 at 06:21:40PM +0200, Geert Uytterhoeven wrote: > Hi Russell, > > On Mon, Oct 19, 2015 at 5:35 PM, Russell King - ARM Linux > wrote: > >> > What you can do is print those devices which have failed to probe at > >> > late_initcall() time - possibly

Re: Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Alan Stern
On Tue, 20 Oct 2015, Bin Liu wrote: > Hi, > > On Tue, Oct 20, 2015 at 11:57 AM, Peter Stuge wrote: > > Greg K-H wrote: > >> >> Userspace still has the port open, so the number is not reused until > >> >> userspace closes the device node. Fix your userspace programs to > >> >>

Re: [PATCH] usb: musb: dsps: handle the otg_state_a_wait_vrise_timeout case

2015-10-20 Thread Gregory CLEMENT
Hi Felipe, On lun., oct. 05 2015, Felipe Balbi wrote: >> So after many tests on different devices, 200ms is enough for most of >> them, but for one, 2000ms (2s) was needed! >> >> I see several option: >> - adding a sysfs entry to setup the time >> - adding a debugs entry entry

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Mark Brown
On Tue, Oct 20, 2015 at 01:14:46PM -0400, Alan Stern wrote: > On Tue, 20 Oct 2015, Tomeu Vizoso wrote: > > This iteration of the series would make this quite easy, as > > dependencies are calculated before probes are attempted: > > https://lkml.org/lkml/2015/6/17/311 > But what Rafael is

Re: [PATCH 1/2] USB: serial: cp210x: Implement 16-bit register access functions

2015-10-20 Thread Konstantin Shkolnyy
On Tue, Oct 20, 2015 at 11:22 AM, Johan Hovold wrote: > On Tue, Oct 20, 2015 at 09:19:05AM -0500, Konstantin Shkolnyy wrote: >> On Tue, Oct 20, 2015 at 8:02 AM, Johan Hovold wrote: >> > On Tue, Oct 20, 2015 at 07:52:31AM -0500, Konstantin Shkolnyy wrote: >> >>

Re: [PATCH] Fix data loss in cdc-acm

2015-10-20 Thread Peter Hurley
On 08/05/2015 01:36 PM, Peter Hurley wrote: > On 07/22/2015 06:53 PM, Sven Brauch wrote: >> On 23/07/15 00:12, Peter Hurley wrote: >>> The premature unthrottle actually leads to the data loss but the throttling >>> with a mere 2K left is _way too late_. >> Ok, yes, I think so too. >> >>> 10ms is a

[GIT PULL] USB patches for v4.4 merge window

2015-10-20 Thread Felipe Balbi
Hi Greg, Here's the large gadget pull request for v4.4. All patches have been in next for a while and should be safe to apply. Let me know if you want any changes to this pull request. cheers The following changes since commit 9ffecb10283508260936b96022d4ee43a7798b4c: Linux 4.3-rc3

Re: Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Krzysztof Opasiak
On 10/20/2015 07:35 PM, Alan Stern wrote: On Tue, 20 Oct 2015, Bin Liu wrote: Hi, On Tue, Oct 20, 2015 at 11:57 AM, Peter Stuge wrote: Greg K-H wrote: Userspace still has the port open, so the number is not reused until userspace closes the device node. Fix your

Re: [PATCH] extcon: add MAX3355 driver

2015-10-20 Thread Sergei Shtylyov
Hello. On 12/18/2014 12:58 AM, Sergei Shtylyov wrote: MAX3355E chip integrates a charge pump and comparators to enable a system with an integrated USB OTG dual-role transceiver to function as a USB OTG dual-role device. In addition to sensing/controlling Vbus, the chip also passes thru the ID

Re: Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Peter Stuge
Krzysztof Opasiak wrote: >> There's still a bit of a race condition here, isn't there? >> >> Is there any good way to deal with that? >> >> Yes, there is a race condition. If the program closes the device node >> before the device is plugged in again, the ttyUSB number won't change;

[PATCH 1/3] uwb: drp: Use setup_timer

2015-10-20 Thread Muhammad Falak R Wani
Use timer API function setup_timer instead of init_timer to initialize a kernel timer. Signed-off-by: Muhammad Falak R Wani --- drivers/uwb/drp.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/uwb/drp.c b/drivers/uwb/drp.c index

Re: Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Greg KH
On Tue, Oct 20, 2015 at 05:20:23PM +0200, Erwin Van de Velde wrote: > Hi all, > > Due to some hardware issues, I have to reset USB modems and a USB to > serial converter on some of my servers. What I noticed is that > /dev/ttyUSBXX devices are never reused: ttyUSB1 becomes ttyUSB3 e.g. > and so

Re: Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Bin Liu
Hi, On Tue, Oct 20, 2015 at 11:57 AM, Peter Stuge wrote: > Greg K-H wrote: >> >> Userspace still has the port open, so the number is not reused until >> >> userspace closes the device node. Fix your userspace programs to >> >> properly listen to the hangup signal to know to

Re: [PATCH 01/15] fs: configfs: Add unlocked version of configfs_depend_item()

2015-10-20 Thread Joel Becker
On Tue, Oct 20, 2015 at 02:32:32PM +0200, Andrzej Pietrasiewicz wrote: > From: Krzysztof Opasiak > > This change is necessary for the SCSI target usb gadget composed with > configfs. In this case configfs will be used for two different purposes: > to compose a usb gadget

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Alan Stern
On Tue, 20 Oct 2015, Mark Brown wrote: > On Tue, Oct 20, 2015 at 10:40:03AM -0400, Alan Stern wrote: > > > Furthermore, that applies only to devices that use synchronous suspend. > > Async suspend is becoming common, and there the only restrictions are > > parent-child relations plus whatever

[BUG] mfd: rtsx_usb data corruption with SDCX microsd

2015-10-20 Thread Marcus Overhagen
I tested again with a 4.2 kernel but the bug is still present and happens more often. So far nobody has responded. I don't know what to do, and whether it's related to usb, mmc or mfd. Please advise. When using the RTS5129 card reader (USB2 but internal to Samsung NP7230U3E series 7 ultrabook)

Re: Udoo support for chipidea

2015-10-20 Thread Fabio Estevam
Hi Peter, On Mon, Oct 19, 2015 at 12:50 AM, Peter Chen wrote: > Add linux-usb. > > Patryk, your problem is you may need to open 24M OSC for HUB 2514 > manually, and you have used IMX6QDL_CLK_CKO for it in the design, > but this clock is not controller's clock,

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Tomeu Vizoso
On 20 October 2015 at 18:04, Alan Stern wrote: > On Tue, 20 Oct 2015, Mark Brown wrote: > >> On Tue, Oct 20, 2015 at 10:40:03AM -0400, Alan Stern wrote: >> >> > Furthermore, that applies only to devices that use synchronous suspend. >> > Async suspend is becoming

Re: [PATCH 1/2] USB: serial: cp210x: Implement 16-bit register access functions

2015-10-20 Thread Johan Hovold
On Tue, Oct 20, 2015 at 09:19:05AM -0500, Konstantin Shkolnyy wrote: > On Tue, Oct 20, 2015 at 8:02 AM, Johan Hovold wrote: > > On Tue, Oct 20, 2015 at 07:52:31AM -0500, Konstantin Shkolnyy wrote: > >> On Tue, Oct 20, 2015 at 2:45 AM, Johan Hovold wrote: > >>

Re: Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Peter Stuge
Greg KH wrote: > Userspace still has the port open, so the number is not reused until > userspace closes the device node. Fix your userspace programs to > properly listen to the hangup signal to know to release the device node > and you should be fine. There's still a bit of a race condition

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Alan Stern
On Tue, 20 Oct 2015, Tomeu Vizoso wrote: > On 20 October 2015 at 18:04, Alan Stern wrote: > > On Tue, 20 Oct 2015, Mark Brown wrote: > > > >> On Tue, Oct 20, 2015 at 10:40:03AM -0400, Alan Stern wrote: > >> > >> > Furthermore, that applies only to devices that use

Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Erwin Van de Velde
Hi all, Due to some hardware issues, I have to reset USB modems and a USB to serial converter on some of my servers. What I noticed is that /dev/ttyUSBXX devices are never reused: ttyUSB1 becomes ttyUSB3 e.g. and so on. This looks fine, but we have to do it rather often, so after a while we reach

[PATCH] usb: gadget: composite: remove redundant bcdUSB setting in legacy

2015-10-20 Thread Krzysztof Opasiak
From: Igor Kotrasinski Since composite now overwrites bcdUSB for any gadget, remove setting it in legacy gadgets. All legacy gadgets set 0x0200, the same as the value additionally set by composite, so there is no behaviour change. Signed-off-by: Igor Kotrasinski

Re: Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Peter Stuge
Hello, Bin Liu wrote: > >> >There's still a bit of a race condition here, isn't there? > >> > > >> >Is there any good way to deal with that? > >> > >> What race condition are you seeing here? > > > > USB cable is unplugged > > Assuming /dev/ttyUSB0 is opened by process ATM. > > > Process is

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Mark Brown
On Tue, Oct 20, 2015 at 10:40:03AM -0400, Alan Stern wrote: > Furthermore, that applies only to devices that use synchronous suspend. > Async suspend is becoming common, and there the only restrictions are > parent-child relations plus whatever explicit requirements that drivers > impose by

Re: Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Johan Hovold
On Tue, Oct 20, 2015 at 05:20:23PM +0200, Erwin Van de Velde wrote: > Hi all, > > Due to some hardware issues, I have to reset USB modems and a USB to > serial converter on some of my servers. What I noticed is that > /dev/ttyUSBXX devices are never reused: ttyUSB1 becomes ttyUSB3 e.g. > and so

Re: Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Greg K-H
On October 20, 2015 9:38:52 AM PDT, Peter Stuge wrote: >Greg KH wrote: >> Userspace still has the port open, so the number is not reused until >> userspace closes the device node. Fix your userspace programs to >> properly listen to the hangup signal to know to release the device

Re: Resetting USB modems and USB to serial converters increases ttyUSBXX counter

2015-10-20 Thread Peter Stuge
Greg K-H wrote: > >> Userspace still has the port open, so the number is not reused until > >> userspace closes the device node. Fix your userspace programs to > >> properly listen to the hangup signal to know to release the device > >node > >> and you should be fine. > > > >There's still a bit

[PATCH] usb: dwc2: host: Fix ahbcfg for rk3066

2015-10-20 Thread Douglas Anderson
The comment for ahbcfg for rk3066 parameters (also used for rk3288) claimed that ahbcfg was INCR16, but it wasn't. Since the bits weren't shifted properly, the 0x7 ended up being masked and we ended up programming 0x3 for the HBstLen. Let's set it to INCR16 properly. As per Wu Liang Feng at

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Rafael J. Wysocki
On Tuesday, October 20, 2015 09:15:01 AM Rob Herring wrote: > On Tue, Oct 20, 2015 at 2:56 AM, Rafael J. Wysocki wrote: > > On Monday, October 19, 2015 05:58:40 PM Rob Herring wrote: > >> On Mon, Oct 19, 2015 at 4:40 PM, Rafael J. Wysocki > >> wrote: > >>

Re: Overly conservative xHCI bandwidth estimation

2015-10-20 Thread Steinar H. Gunderson
On Mon, Sep 28, 2015 at 02:32:13PM +0200, Steinar H. Gunderson wrote: > Just so that it doesn't get lost: I've reported issues with this specific > device and LPM not too long ago. It's entirely possible that the device > somehow is broken, although it works in Windows 7/8/10 and OS X, from what I

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Rafael J. Wysocki
On Tuesday, October 20, 2015 08:35:28 PM Mark Brown wrote: > > --7fVr/IRGAG9sAW4J > Content-Type: text/plain; charset=us-ascii > Content-Disposition: inline > Content-Transfer-Encoding: quoted-printable > > On Tue, Oct 20, 2015 at 01:14:46PM -0400, Alan Stern wrote: > > On Tue, 20 Oct 2015,

Re: Overly conservative xHCI bandwidth estimation

2015-10-20 Thread Lu, Baolu
I could spend some time on this issue a week later. I'd like to check whether there is any bugs in the driver itself. Otherwise, blacklist this specific device for LPM. Thanks, Baolu On 10/21/2015 07:05 AM, Steinar H. Gunderson wrote: On Mon, Sep 28, 2015 at 02:32:13PM +0200, Steinar H.

[PATCH 1/1] usb: chipidea: otg: gadget module load and unload support

2015-10-20 Thread Peter Chen
From: Li Jun This patch is to support load and unload gadget driver in full OTG mode. Signed-off-by: Li Jun Signed-off-by: Peter Chen Cc: Jiada Wang (Peter: it fixes a bug that the unload/reload

Fwd: [Bug 106321] linux-kernel-4.3-rc5 fails to attach usb storage device

2015-10-20 Thread Jos van Wolput
Forwarded Message Subject: [Bug 106321] linux-kernel-4.3-rc5 fails to attach usb storage device Date: Tue, 20 Oct 2015 13:48:54 + From: bugzilla-dae...@bugzilla.kernel.org To: wol...@on.nl https://bugzilla.kernel.org/show_bug.cgi?id=106321 --- Comment #1 from Greg

Re: Udoo support for chipidea

2015-10-20 Thread Peter Chen
On Tue, Oct 20, 2015 at 02:09:38PM -0200, Fabio Estevam wrote: > Hi Peter, > > On Mon, Oct 19, 2015 at 12:50 AM, Peter Chen wrote: > > > Add linux-usb. > > > > Patryk, your problem is you may need to open 24M OSC for HUB 2514 > > manually, and you have used

Re: [PATCH] extcon: add MAX3355 driver

2015-10-20 Thread Chanwoo Choi
Hi Sergei, I think this patch is too much delay. I recommend you better to develop this driver based on latest extcon-next branch[1]. [1] https://git.kernel.org/cgit/linux/kernel/git/chanwoo/extcon.git/log/?h=extcon-next Thanks, Chanwoo Choi On 2015년 10월 21일 03:20, Sergei Shtylyov wrote: >

Re: Alternative approach to solve the deferred probe

2015-10-20 Thread Frank Rowand
On 10/20/2015 8:46 AM, Russell King - ARM Linux wrote: > On Mon, Oct 19, 2015 at 06:21:40PM +0200, Geert Uytterhoeven wrote: >> Hi Russell, >> >> On Mon, Oct 19, 2015 at 5:35 PM, Russell King - ARM Linux >> wrote: > What you can do is print those devices which have

Re: [PATCH 1/2] USB: serial: cp210x: Implement 16-bit register access functions

2015-10-20 Thread Johan Hovold
On Tue, Oct 20, 2015 at 07:52:31AM -0500, Konstantin Shkolnyy wrote: > On Tue, Oct 20, 2015 at 2:45 AM, Johan Hovold wrote: > [...] > > Instead of adding the new helpers to read u16 as a prerequisite for > > fixing the broken cp2108 support, just reuse the current config

Re: [PATCH] usb: chipidea: set unique names to requested irqs

2015-10-20 Thread Vladimir Zapolskiy
On 20.10.2015 09:05, Peter Chen wrote: > On Mon, Oct 19, 2015 at 07:00:26PM +0300, Vladimir Zapolskiy wrote: >> This change allows to get a driver instance of usb controller, which >> registers an irq, now the interrupt names are unique: >> >> # cat /proc/interrupts | grep ci_ >> 72: