Re: [PATCH 00/21] On-demand device registration

2015-06-15 Thread Alexander Holler
Am 15.06.2015 um 10:58 schrieb Linus Walleij: On Sat, Jun 13, 2015 at 8:27 PM, Alexander Holler hol...@ahsoftware.de wrote: And because you've said that problem space is a bit convoluted and I disagree, here's a summary from my point of view: 1. All the necessary information (dependencies

Re: [PATCH] net:usb:cdc_ncm: fix that tag Huawei devices as wwan

2015-06-15 Thread Bjørn Mork
Marcel Holtmann mar...@holtmann.org writes: we introduced DEVTYPE in uevent a long time ago. That is what userspace should be using and not second guessing on interface names. Yes, sorry for confusing this by mentioning the device name. This is really about DEVTYPE. usbnet minidrivers use

Re: [PATCH] net:usb:cdc_ncm: fix that tag Huawei devices as wwan

2015-06-15 Thread Marcel Holtmann
Hi Bjorn, we introduced DEVTYPE in uevent a long time ago. That is what userspace should be using and not second guessing on interface names. Yes, sorry for confusing this by mentioning the device name. This is really about DEVTYPE. usbnet minidrivers use FLAG_WWAN to set both the

[PATCH 3/3][v2] drivers:usb:fsl: Introduce FSL_USB2_PHY_UTMI_DUAL macro

2015-06-15 Thread Nikhil Badola
Introduce FSL_USB2_PHY_UTMI_DUAL macro for setting phy mode in SOCs such has T4240, T1040, T2080 which have utmi dual-phy Signed-off-by: Ramneek Mehresh ramneek.mehr...@freescale.com Signed-off-by: Nikhil Badola nikhil.bad...@freescale.com --- Changes for v2 : None drivers/usb/host/ehci-fsl.c

[PATCH 2/3][v2] drivers: usb :fsl: Implement Workaround for USB Erratum A007792

2015-06-15 Thread Nikhil Badola
USB controller version-2.5 requires to enable internal UTMI phy and program PTS field in PORTSC register before asserting controller reset. This is must for successful resetting of the controller and subsequent enumeration of usb devices Signed-off-by: Nikhil Badola nikhil.bad...@freescale.com

Re: [PATCH] net:usb:cdc_ncm: fix that tag Huawei devices as wwan

2015-06-15 Thread Marcel Holtmann
Hi Bjorn, Hmm. Oliver is marked as the maintainer of the USB CDC code, but others have touched it more recently. So I'm just wildly adding people to the cc to comment on this patch and maybe apply it. Oliver/David/Ben/Bjørn? Adding Aleksander and Dan, too. The 'wwanX' vs 'usbX'

Re: [PATCH 00/21] On-demand device registration

2015-06-15 Thread Linus Walleij
On Sat, Jun 13, 2015 at 8:27 PM, Alexander Holler hol...@ahsoftware.de wrote: And because you've said that problem space is a bit convoluted and I disagree, here's a summary from my point of view: 1. All the necessary information (dependencies between drivers) already exists at compile time.

[PATCH 1/3][v2] drivers:usb:fsl: Replace macros with enumerated type

2015-06-15 Thread Nikhil Badola
Replace macros with enumerated type to represent usb ip controller version Signed-off-by: Nikhil Badola nikhil.bad...@freescale.com --- Changes for v2 : - Assigned value to each enumerator - Changed return type of function that returns controller version -

Re: Fwd: [PATCH] net:usb:cdc_ncm: fix that tag Huawei devices as wwan

2015-06-15 Thread Oliver Neukum
On Sun, 2015-06-14 at 13:51 -1000, Linus Torvalds wrote: Hmm. Oliver is marked as the maintainer of the USB CDC code, but I do CDC ACM, CDC WDM and CDC Ether, but not CDC NCM (it is a very different beast) others have touched it more recently. So I'm just wildly adding people to the cc to

Re: Fwd: [PATCH] net:usb:cdc_ncm: fix that tag Huawei devices as wwan

2015-06-15 Thread Bjørn Mork
Linus Torvalds torva...@linux-foundation.org writes: Hmm. Oliver is marked as the maintainer of the USB CDC code, but others have touched it more recently. So I'm just wildly adding people to the cc to comment on this patch and maybe apply it. Oliver/David/Ben/Bjørn? Adding Aleksander and

Re: [PATCH v2 05/22] doc: dt-binding: usb: add otg related properties

2015-06-15 Thread Li Jun
On Mon, Jun 15, 2015 at 03:25:48PM +0300, Sergei Shtylyov wrote: Hello. On 6/15/2015 10:41 AM, Li Jun wrote: [...] Regarding use of xyz-disable flags: After more thinking this, I just realize that we can use otg-rev property to handle the case of none of xyz-disable properties passed,

Re: [PATCH v2 05/22] doc: dt-binding: usb: add otg related properties

2015-06-15 Thread Sergei Shtylyov
Hello. On 6/15/2015 10:41 AM, Li Jun wrote: [...] Regarding use of xyz-disable flags: After more thinking this, I just realize that we can use otg-rev property to handle the case of none of xyz-disable properties passed, for legacy platforms, the new otg-rev not passed either, so it's a

Re: [PATCH v3 1/1] usb: core: lpm: set lpm_capable for root hub device

2015-06-15 Thread Alan Stern
On Mon, 15 Jun 2015, Lu Baolu wrote: Commit 25cd2882e2fc (usb/xhci: Change how we indicate a host supports Link PM.) removed the code to set lpm_capable for USB 3.0 super-speed root hub. The intention of that change was to avoid touching usb core internal field, a.k.a. lpm_capable, and let

Re: questions about uas

2015-06-15 Thread Alan Stern
On Mon, 15 Jun 2015, Tom Yan wrote: So I did some further investigation on the weird optimal i/o size I got from my usb3/sata adapter/ssd, started by grep'ing the size in the /sys/block/sdx/queue: [tom@localhost ~]$ grep 33553920 /sys/block/sdb/queue/* grep: /sys/block/sdb/queue/iosched:

Re: [PATCH v4 2/2] USB: io_ti: Add heartbeat to keep idle Edgeport ports from disconnecting

2015-06-15 Thread Peter Berger
Hi Johan: Again, I think I'm about ready to post a v5 patchset that addresses your comments here and elsewhere. I'll include point-by-point response notes to your comments below. Thanks! --Peter On Thu, 2015-06-11 at 12:26 +0200, Johan Hovold wrote: On Mon, Jun 08, 2015 at 02:51:37PM

Re: [PATCH v3 2/2] USB: io_ti: Fix Edgeport firmware download code

2015-06-15 Thread Peter Berger
On Thu, 2015-06-11 at 11:10 +0200, Johan Hovold wrote: On Mon, Jun 08, 2015 at 02:36:36PM -0500, Peter Berger wrote: On Fri, 2015-05-22 at 18:22 +0200, Johan Hovold wrote: On Fri, May 15, 2015 at 12:09:54AM -0500, Peter E. Berger wrote: From: Peter E. Berger pber...@brimson.com

Re: TRIM/DISCARD for usb drives?

2015-06-15 Thread Alan Stern
On Mon, 15 Jun 2015, Tom Yan wrote: I have a SanDisk Extreme USB Flash Drive (http://www.sandisk.com/products/usb/drives/extreme/), which does NOT support UASP so is running under usb-storage. According to `hdparm`, it seems to supports TRIM; and according to `sg3_opcodes`, it seems to

RE: FX3 on Linux

2015-06-15 Thread David Laight
From: philip.jos...@microchip.com Knowing that you have experts in USB 3.0 support for Linux, I am writing to your organization hoping that you may be able to offer some direction on a problem I have encountered. I am trying to track down a problem I am having with a USB 3.0 PCIe-x1 card

Re: [PATCH v4 1/2] USB: io_ti: Fix Edgeport firmware download code

2015-06-15 Thread Peter Berger
Hi Johan: I think my v5 patchset to address your review comments (here and in the other messages) is just about ready to submit. I'll include some point-by-point description details below: On Thu, 2015-06-11 at 12:03 +0200, Johan Hovold wrote: On Mon, Jun 08, 2015 at 02:51:36PM -0500, Peter E.

Re: [PATCH v2 05/22] doc: dt-binding: usb: add otg related properties

2015-06-15 Thread Li Jun
On Fri, Jun 12, 2015 at 11:31:02AM +0300, Roger Quadros wrote: On Fri, 12 Jun 2015 11:09:17 +0800 Li Jun b47...@freescale.com wrote: [...] Other than those new flags, I have not found other good way to judge whether some platform is a legacy one. Directly use dt property? Then above

[PATCH v4 1/1] usb: core: lpm: set lpm_capable for root hub device

2015-06-15 Thread Lu Baolu
Commit 25cd2882e2fc (usb/xhci: Change how we indicate a host supports Link PM.) removed the code to set lpm_capable for USB 3.0 super-speed root hub. The intention of that change was to avoid touching usb core internal field, a.k.a. lpm_capable, and let usb core to set it by checking U1 and U2

keyboard driver (anti-ghosting chip) needed for Linux - many advanced keyboards do not work with Linux

2015-06-15 Thread Zenaan Harkness
To my great surprise, Linux does not support NKRO (N-key roll-over) keyboards connected by USB, and evidently some USB-PS2 adapters (which are no use to us laptop people anyway). Definitely one for the Linux device driver bucket list. Regards Zenaan --- Firstly, we in the Linux community can

[PATCH v5 09/10] USB: io_ti: Add edge_heartbeat_reschedule() helper function

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com Add edge_heartbeat_reschedule() helper function and call it from edge_startup() and edge_resume(). Signed-off-by: Peter E. Berger pber...@brimson.com --- drivers/usb/serial/io_ti.c | 19 +++ 1 file changed, 11 insertions(+), 8

[PATCH v5 03/10] USB: io_ti: Move request_firmware() calls out of download_fw()

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com The io_ti driver fails to download firmware to Edgeport devices such as the EP/416, even when the on-disk firmware image (/lib/firmware/edgeport/down3.bin) is more current than the version on the EP/416. The current download code is broken in a few ways.

[PATCH v5 00/10] Fix idle port disconnects and firmware downloading

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com I found that when using newer Edgeport devices such as the EP/416, idle ports are regularly bounced (disconnected and then reconnected). I discussed this with the manufacturer (Digi International) and learned that some newer Edgeport devices now ship from

[PATCH v5 02/10] USB: io_ti: Add timeout parameter to ti_vsend_sync()

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com Instead of having ti_vsend_sync() decide when to use non-default timeouts, make timeout a parameter and leave the decision to the caller. Signed-off-by: Peter E. Berger pber...@brimson.com --- drivers/usb/serial/io_ti.c | 28 +---

[PATCH v5 06/10] USB: io_ti: Add heartbeat to keep idle Edgeport ports from disconnecting

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com When using newer Edgeport devices such as the EP/416, idle ports are automatically bounced (disconnected and then reconnected) approximately every 60 seconds. This breaks programs (e.g: minicom) where idle periods are common, normal and expected. I

[PATCH v5 01/10] USB: io_ti: Increase insufficient timeout for firmware downloads

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com The io_ti driver fails to download firmware to Edgeport devices such as the EP/416. One of the problems is that the default 1 second timeout in ti_vsend_sync() is insufficient for download operations. This patch increases the download timeout to 10

[PATCH v5 08/10] USB: io_ti: Call INIT_DELAYED_WORK unconditionally in edge_startup()

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com Move INIT_DELAY_WORK out of the if block. Signed-off-by: Peter E. Berger pber...@brimson.com --- drivers/usb/serial/io_ti.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/usb/serial/io_ti.c b/drivers/usb/serial/io_ti.c

[PATCH v5 10/10] USB: io_ti: Call cancel_delayed_work_sync() unconditionally

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com Call cancel_delayed_work_sync() unconditionally in edge_release() and edge_suspend(). Signed-off-by: Peter E. Berger pber...@brimson.com --- drivers/usb/serial/io_ti.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

[PATCH v5 05/10] USB: io_ti: Add comment describing the unused firmware build_number

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com The firmware has a build_number field, though it is apparently unused (according to observations of the three firmware images I have seen and confirmed by Digi Tech Support). This comment describes its structure, in case it is populated in a future

[PATCH v5 07/10] USB: io_ti: dev_dbg() and dev_err() changes in edge_startup()

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com - Replace a dev_dbg() with dev_err(). - Drop unneeded dev_dbg(). Signed-off-by: Peter E. Berger pber...@brimson.com --- drivers/usb/serial/io_ti.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/usb/serial/io_ti.c

[PATCH v5 04/10] USB: io_ti: Add fw_version field to edgeport_serial

2015-06-15 Thread Peter E. Berger
From: Peter E. Berger pber...@brimson.com The io_ti driver fails to download firmware to Edgeport devices such as the EP/416, even when the on-disk firmware image (/lib/firmware/edgeport/down3.bin) is more current than the version on the EP/416. This patch adds a fw_version field to

Re: keyboard driver (anti-ghosting chip) needed for Linux - many advanced keyboards do not work with Linux

2015-06-15 Thread Greg KH
On Tue, Jun 16, 2015 at 03:32:48PM +1000, Zenaan Harkness wrote: To my great surprise, Linux does not support NKRO (N-key roll-over) keyboards connected by USB, and evidently some USB-PS2 adapters (which are no use to us laptop people anyway). Definitely one for the Linux device driver

Re: [PATCH] usb: pass flags parameter to gpiod_get functions

2015-06-15 Thread Uwe Kleine-König
Hello, On Fri, Mar 27, 2015 at 08:30:42PM +0100, Uwe Kleine-König wrote: 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.

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

2015-06-15 Thread Andy Furniss
Andy Furniss wrote: Hi I recently set up a Asrock Q1900DC-itx Intel baytrail board as a router/nas/pvr. It has 2 usb dvb T/T2 tuners attached - pctv 290e and 292e. When running them I am getting lost dvb packets which are detected as lost via dvbs continuity counters. Have tried 3.18.14, and

Re: [PATCH v2 05/22] doc: dt-binding: usb: add otg related properties

2015-06-15 Thread Li Jun
On Fri, Jun 12, 2015 at 11:31:02AM +0300, Roger Quadros wrote: Agreed. We still need to determine legacy platform if none of the features are set. How about defining the followong enum for gadget-xyz_support enum otg_feature { OTG_FEATURE_UNDEFINED = 0,

Re: [PATCH v2 05/22] doc: dt-binding: usb: add otg related properties

2015-06-15 Thread Roger Quadros
On Mon, 15 Jun 2015 14:32:46 +0800 Li Jun b47...@freescale.com wrote: On Fri, Jun 12, 2015 at 11:31:02AM +0300, Roger Quadros wrote: On Fri, 12 Jun 2015 11:09:17 +0800 Li Jun b47...@freescale.com wrote: [...] Other than those new flags, I have not found other good way to judge

Re: [PATCH v2 05/22] doc: dt-binding: usb: add otg related properties

2015-06-15 Thread Roger Quadros
On Mon, 15 Jun 2015 15:41:02 +0800 Li Jun b47...@freescale.com wrote: On Fri, Jun 12, 2015 at 11:31:02AM +0300, Roger Quadros wrote: Agreed. We still need to determine legacy platform if none of the features are set. How about defining the followong enum for gadget-xyz_support

Wvdial problem with 3G modem connected via pl2303 (bug 99971)

2015-06-15 Thread Tim Komarov
Hello, guys! I got a modem (SIM5320), which works via UART port. I connect it to PC with USB-UART (pl2303 сonverter) cable and see, that Wvdial is not able to send AT-commands to the modem in a proper way (the output looks like the messages are corrupted), so I got no PPP-connection. At the