Re: [PATCH v3 3/3] usb: phy: hold wakeupsource when USB is enumerated in peripheral mode

2015-07-20 Thread Sumit Semwal
On 13 July 2015 at 12:30, Peter Chen wrote: > On Mon, Jul 13, 2015 at 01:26:23PM +0530, Sumit Semwal wrote: >> Hi Peter, >> >> On 8 June 2015 at 14:02, Peter Chen wrote: >> > On Mon, Jun 08, 2015 at 12:37:00PM +0530, Sumit Semwal wrote: >> >> H Peter, Felipe, >> >> >> >> On 14 January 2015 at 06:

Re: [PATCH v3 3/3] usb: phy: hold wakeupsource when USB is enumerated in peripheral mode

2015-07-20 Thread Peter Chen
On Mon, Jul 20, 2015 at 01:18:35PM +0530, Sumit Semwal wrote: > On 13 July 2015 at 12:30, Peter Chen wrote: > > On Mon, Jul 13, 2015 at 01:26:23PM +0530, Sumit Semwal wrote: > >> Hi Peter, > >> > >> On 8 June 2015 at 14:02, Peter Chen wrote: > >> > On Mon, Jun 08, 2015 at 12:37:00PM +0530, Sumit

Re: is xHCI support for AMD APU chipsets known to be broken?

2015-07-20 Thread Mathias Nyman
Hi, I'm temporary back from vacation, for a week or two. On 08.07.2015 05:38, James Ralston wrote: > Before I submit a full-blown bug report, could anyone tell me whether > there are already known issues with the xHCI driver on AMD and/or > ASUSTek systems, specifically involving AMD APU chipsets

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

2015-07-20 Thread Johan Hovold
On Mon, Jul 06, 2015 at 10:40:32AM -0400, Nik Nyby wrote: > This fixes a typo in the URL: http://zeitcontrol.de/ > > Signed-off-by: Nik Nyby Applied, thanks. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More

[PATCH] qcserial: Add support for Dell Wireless 5809e 4G Modem

2015-07-20 Thread Pieter Hollants
Added the USB IDs 0x413c:0x81b1 for the "Dell Wireless 5809e Gobi(TM) 4G LTE Mobile Broadband Card", a Dell-branded Sierra Wireless EM7305 LTE card in M.2 form factor, used eg. in Dell's Latitude E7540 Notebook series. Signed-off-by: Pieter Hollants --- drivers/usb/serial/qcserial.c | 1 + 1 fil

Re: [PATCH v2] USB: sierra: add 1199:68AB device ID

2015-07-20 Thread Johan Hovold
[ +CC: Bjørn and Dan ] On Mon, Jul 20, 2015 at 08:14:19AM +0200, Dirk Behme wrote: > Add support for the Sierra Wireless AR8550 device with > USB descriptor 0x1199, 0x68AB. For this, lsusb reports: Thanks for the patch. This modem business is a bit of a mess and it's not always apparent what driv

Re: [PATCH] USB: qcserial/option: make AT URCs work for Sierra Wireless MC7305/MC7355

2015-07-20 Thread Johan Hovold
On Tue, Jul 14, 2015 at 10:55:06PM +0200, Reinhard Speyerer wrote: > Tests with a Sierra Wireless MC7355 have shown that 1199:9041 devices > also require the option_send_setup() code to be used on the USB > interface for the AT port to make unsolicited response codes work > correctly. Move these de

Re: [PATCH] qcserial: Add support for Dell Wireless 5809e 4G Modem

2015-07-20 Thread Johan Hovold
On Mon, Jul 20, 2015 at 10:33:21AM +0200, Pieter Hollants wrote: > Added the USB IDs 0x413c:0x81b1 for the "Dell Wireless 5809e Gobi(TM) 4G > LTE Mobile Broadband Card", a Dell-branded Sierra Wireless EM7305 LTE > card in M.2 form factor, used eg. in Dell's Latitude E7540 Notebook > series. > > Si

[PATCH v2] qcserial: Add support for Dell Wireless 5809e 4G Modem

2015-07-20 Thread Pieter Hollants
Added the USB IDs 0x413c:0x81b1 for the "Dell Wireless 5809e Gobi(TM) 4G LTE Mobile Broadband Card", a Dell-branded Sierra Wireless EM7305 LTE card in M.2 form factor, used eg. in Dell's Latitude E7540 Notebook series. "lsusb -v" output for this device: Bus 002 Device 003: ID 413c:81b1 Dell Compu

Re: [PATCH v2] qcserial: Add support for Dell Wireless 5809e 4G Modem

2015-07-20 Thread Johan Hovold
On Mon, Jul 20, 2015 at 11:56:17AM +0200, Pieter Hollants wrote: > Added the USB IDs 0x413c:0x81b1 for the "Dell Wireless 5809e Gobi(TM) 4G > LTE Mobile Broadband Card", a Dell-branded Sierra Wireless EM7305 LTE > card in M.2 form factor, used eg. in Dell's Latitude E7540 Notebook > series. > > "l

Re: [PATCH v3 4/5] usb: gadget: mass_storage: Use static array for luns

2015-07-20 Thread Krzysztof Opasiak
On 07/08/2015 04:00 PM, Michal Nazarewicz wrote: On Tue, Jul 07 2015, Krzysztof Opasiak wrote: This patch replace dynamicly allocated luns array with static one. This simplifies the code of mass storage function and modules. It also fix issue with reporting wrong number of LUNs in GET_MAX_LUN

Re: Renesas upd7202 (xhci): can't setup

2015-07-20 Thread Mathias Nyman
On 15.07.2015 14:54, Jean-Francois SIMON wrote: > Hi, > > We have designed a h/w system with the Renesas USB 3.0 upd7202 controller. > We are seeing that sometimes xhci initialization fails with a "can't > setup" error message. > As a result lsusb doesn't report the USB3.0 ports. > > Adding trace

Re: [PATCH v2] USB: pl2303: Rewrite pl2303_encode_baud_rate_divisor

2015-07-20 Thread Johan Hovold
On Wed, Jul 15, 2015 at 01:35:58PM +0200, Michał Pecio wrote: > > This commit fixes the following issues: > > 1. The 9th bit of buf was believed to be the LSB of divisor's > exponent, but the hardware interprets it as MSB (9th bit) of the > mantissa. The exponent is actually one bit shorter and a

External USB3 hard drive is sometimes not recognized (race condition?) - Event TRB for slot 14 ep 2 with no TDs queued

2015-07-20 Thread Kamil Paral
Hello folks, I have troubles with my external USB3 hard drive (a standard laptop drive put into an USB3 enclosure) and I have reported a bug about it here: https://bugzilla.kernel.org/show_bug.cgi?id=101661 I was asked to re-send it to this mailing list, so here it is. I quote the initial bug d

Re: [PATCH v3 3/3] usb: phy: hold wakeupsource when USB is enumerated in peripheral mode

2015-07-20 Thread Sumit Semwal
Hi Peter, On 20 July 2015 at 12:45, Peter Chen wrote: > On Mon, Jul 20, 2015 at 01:18:35PM +0530, Sumit Semwal wrote: >> On 13 July 2015 at 12:30, Peter Chen wrote: >> > On Mon, Jul 13, 2015 at 01:26:23PM +0530, Sumit Semwal wrote: >> >> Hi Peter, >> >> >> >> On 8 June 2015 at 14:02, Peter Chen

Re: [PATCH v3 4/5] usb: gadget: mass_storage: Use static array for luns

2015-07-20 Thread Michal Nazarewicz
> On 07/08/2015 04:00 PM, Michal Nazarewicz wrote: >> On Tue, Jul 07 2015, Krzysztof Opasiak wrote: >>> This patch replace dynamicly allocated luns array with static one. >>> This simplifies the code of mass storage function and modules. >>> >>> It also fix issue with reporting wrong number of LUNs

Re: [PATCH] xhci: Calculate old endpoints correctly on device reset

2015-07-20 Thread Mathias Nyman
On 19.07.2015 18:25, Brian Campbell wrote: > > When resetting a device the number of active TTs may need to be > corrected by xhci_update_tt_active_eps, but the number of old active > endpoints supplied to it was always zero, so the number of TTs and the > bandwidth reserved for them was not updat

Re: [PATCH v3 4/5] usb: gadget: mass_storage: Use static array for luns

2015-07-20 Thread Krzysztof Opasiak
On 07/20/2015 03:17 PM, Michal Nazarewicz wrote: On 07/08/2015 04:00 PM, Michal Nazarewicz wrote: On Tue, Jul 07 2015, Krzysztof Opasiak wrote: This patch replace dynamicly allocated luns array with static one. This simplifies the code of mass storage function and modules. It also fix issue

Re: [PATCH v2 2/5] dt-bindings: Add a binding for Mediatek xHCI host controller

2015-07-20 Thread chunfeng yun
On Fri, 2015-07-10 at 11:31 +0300, Roger Quadros wrote: > Hi, > > On 08/07/15 12:41, Chunfeng Yun wrote: > > add a DT binding documentation of xHCI host controller for the > > MT8173 SoC from Mediatek. > > > > Signed-off-by: Chunfeng Yun > > --- > > .../devicetree/bindings/usb/mt8173-xhci.txt

Re: [PATCH v2 5/5] arm64: dts: mediatek: add xHCI & usb phy for mt8173

2015-07-20 Thread chunfeng yun
Hi, On Tue, 2015-07-14 at 18:12 +0800, Daniel Kurtz wrote: > Hi Chunfeng, > > On Wed, Jul 8, 2015 at 5:41 PM, Chunfeng Yun > wrote: > > Signed-off-by: Chunfeng Yun > > --- > > arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 15 +++ > > arch/arm64/boot/dts/mediatek/mt8173.dtsi| 2

Re: [PATCH v3 01/46] usb: gadget: encapsulate endpoint claiming mechanism

2015-07-20 Thread Krzysztof Opasiak
On 07/15/2015 08:31 AM, Robert Baldyga wrote: So far it was necessary for usb functions to set ep->driver_data in endpoint obtained from autoconfig to non-null value, to indicate that endpoint is claimed by function (in autoconfig it was checked if endpoint has set this field to non-null value,

Re: [PATCH v2 5/5] arm64: dts: mediatek: add xHCI & usb phy for mt8173

2015-07-20 Thread Yingjoe Chen
Hi Daniel, Chunfeng, On Mon, 2015-07-20 at 22:39 +0800, chunfeng yun wrote: > Hi, > > On Tue, 2015-07-14 at 18:12 +0800, Daniel Kurtz wrote: <...> > > > + > > > + usb_p1_vbus: fixedregulator@0 { > > > > Why @0 ? > It is the first fixed regulator, so set it to 0 as a index. > I will remov

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

2015-07-20 Thread chunfeng yun
On Mon, 2015-07-13 at 16:52 +0200, John Crispin wrote: > Hi > > there is a cross dependency between the modules. xhci-mtk.ko requires > xhci.ko to be loaded. however this will look for xhci_mtk_add_ep_quirk() > which is part of xhci-mtk. the modules will build but are not run time > loadable. > >

Re: [PATCH v2 5/5] arm64: dts: mediatek: add xHCI & usb phy for mt8173

2015-07-20 Thread Daniel Kurtz
On Mon, Jul 20, 2015 at 10:51 PM, Yingjoe Chen wrote: > > Hi Daniel, Chunfeng, > > > On Mon, 2015-07-20 at 22:39 +0800, chunfeng yun wrote: >> Hi, >> >> On Tue, 2015-07-14 at 18:12 +0800, Daniel Kurtz wrote: > <...> >> > > + >> > > + usb_p1_vbus: fixedregulator@0 { >> > >> > Why @0 ? >> It i

Re: [PATCH] xhci: Calculate old endpoints correctly on device reset

2015-07-20 Thread Brian Campbell
On Mon, Jul 20, 2015 at 04:38:16PM +0300, Mathias Nyman wrote: > > I remember there was an issue about something eating bandwidth as > suspend/resume > that never got fixed? > > here it is: > http://marc.info/?l=linux-usb&m=141561758015676&w=2 That sounds like the same problem (s/w bandwidth ch

Re: [PATCH v3 26/46] usb: gadget: pch_ud: add ep capabilities support

2015-07-20 Thread Krzysztof Opasiak
On 07/15/2015 08:32 AM, Robert Baldyga wrote: Convert endpoint configuration to new capabilities model. Signed-off-by: Robert Baldyga --- drivers/usb/gadget/udc/pch_udc.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/drivers/usb/gadget/udc/pch_udc.c b/

Re: [PATCH v3 4/5] usb: gadget: mass_storage: Use static array for luns

2015-07-20 Thread Michal Nazarewicz
On Mon, Jul 20 2015, Krzysztof Opasiak wrote: > Yeah I see it now. What I wrote was a problem before your patch but now > it seams to be ok, just nluns name is very misleading. Summing up, just > like you wrote this patch should not go to stable. For me nluns has always been *size* of the luns a

Re: [PATCH 1/1] usb: chipidea: host: add own hc_driver for each hcd

2015-07-20 Thread Alan Stern
On Mon, 20 Jul 2015, Peter Chen wrote: > > I don't think a new use case is needed. All you have to do is add > > something like this to host.c: > > > > static void ci_hdrc_host_driver_init(void) > > { > > ehci_init_driver(&ci_ehci_hc_driver, &ehci_ci_overrides); By the way, right here I for

Re: [PATCH v2 3/5] usb: phy: add usb3.0 phy driver for mt65xx SoCs

2015-07-20 Thread chunfeng yun
On Fri, 2015-07-10 at 08:42 +0200, Sascha Hauer wrote: > On Wed, Jul 08, 2015 at 05:41:05PM +0800, Chunfeng Yun wrote: > > Signed-off-by: Chunfeng Yun > > --- > > drivers/usb/phy/Kconfig | 10 + > > drivers/usb/phy/Makefile | 1 + > > drivers/usb/phy/phy-mt65xx-usb3.c | 856

Re: [PATCH 1/4] usb: gadget: udc: bdc: Fix a driver crash on disconnect

2015-07-20 Thread Felipe Balbi
On Thu, Jul 16, 2015 at 05:47:35PM -0400, Al Cooper wrote: > ep_dequeue() in bdc_ep.c was capturing the hw dequeue pointer > incorrectly by reading the wrong register for the upper 32 bits. > The header file defining the registers was incorrect. > > Signed-off-by: Al Cooper so you fix a driver c

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

2015-07-20 Thread Johan Hovold
[ +CC: Alan, Oliver, Peter, Toby, linux-serial, linux-usb ] On Sun, Jul 19, 2015 at 11:37:07PM +0200, Sven Brauch wrote: > Since acm_process_read_urb does not check the return value > of tty_insert_flip_string, it can happen that not all data > is copied from the urb to the tty if the tty buffer >

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

2015-07-20 Thread Felipe Balbi
Hi, On Wed, Jul 15, 2015 at 09:49:05AM +, Subbaraya Sundeep Bhatta wrote: > Hi John, > > > -Original Message- > > From: Felipe Balbi [mailto:ba...@ti.com] > > Sent: Tuesday, July 14, 2015 12:29 AM > > To: John Youn > > Cc: ba...@ti.com; Subbaraya Sundeep Bhatta; gre...@linuxfoundation

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

2015-07-20 Thread Sven Brauch
On 20/07/15 19:25, Johan Hovold wrote: > What kernel version are you using? I'm using linux 4.1.2. > The idea of adding another layer of buffering in the cdc-acm driver has > been suggested in the past but was rejected (or at least questioned). > See for example this thread: > > https://lkm

[PATCH v4 0/5] Mass storage fixes and improvements

2015-07-20 Thread Krzysztof Opasiak
Hello, This series fix a few bugs in mass storage function, adds a warning message when binding function with not contiguous LUN ids and replace dynamically allocated luns array with static one what simplifies the code. Best regards, -- Krzysztof Opasiak Samsung R&D Institute Poland Samsung Ele

[PATCH v4 1/5] usb: gadget: mass_storage: Free buffers if create lun fails

2015-07-20 Thread Krzysztof Opasiak
Creation of LUN 0 may fail (for example due to ENOMEM). As fsg_common_set_num_buffers() does some memory allocation we should free it before it becomes unavailable. Signed-off-by: Krzysztof Opasiak Acked-by: Michal Nazarewicz --- drivers/usb/gadget/function/f_mass_storage.c |5 + 1 file

[PATCH v4 2/5] usb: gadget: mass_storage: Place EXPORT_SYMBOL_GPL() after func definition

2015-07-20 Thread Krzysztof Opasiak
EXPORT_SYMBOL_GPL() is usually placed after function definition not before. Signed-off-by: Krzysztof Opasiak Acked-by: Michal Nazarewicz --- drivers/usb/gadget/function/f_mass_storage.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/gadget/function/f_mass_sto

[PATCH v4 3/5] usb: gadget: storage-common: Set FSG_MAX_LUNS to 16

2015-07-20 Thread Krzysztof Opasiak
Mass storage spec allows up to 16 LUNs, so let's not add some more restrictive limits. Signed-off-by: Krzysztof Opasiak Acked-by: Michal Nazarewicz --- drivers/usb/gadget/function/f_mass_storage.c |2 +- drivers/usb/gadget/function/storage_common.h |2 +- 2 files changed, 2 insertions(+

[PATCH v4 5/5] usb: gadget: mass_storage: Warn if LUNs ids are not contiguous

2015-07-20 Thread Krzysztof Opasiak
According to mass storage specification: "Logical Unit Numbers on the device shall be numbered contiguously starting from LUN 0 to a maximum LUN of 15 (Fh)" So let's at least print a warning message that LUNs ids should be contiguous. Signed-off-by: Krzysztof Opasiak Acked-by: Michal Nazarewic

[PATCH v4 4/5] usb: gadget: mass_storage: Use static array for luns

2015-07-20 Thread Krzysztof Opasiak
This patch replace dynamicly allocated luns array with static one. This simplifies the code of mass storage function and modules. Signed-off-by: Krzysztof Opasiak Acked-by: Michal Nazarewicz --- drivers/usb/gadget/function/f_mass_storage.c | 127 +- drivers/usb/gadget/f

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

2015-07-20 Thread John Youn
On 7/20/2015 10:51 AM, Felipe Balbi wrote: > Hi, > > On Wed, Jul 15, 2015 at 09:49:05AM +, Subbaraya Sundeep Bhatta wrote: >> Hi John, >> >>> -Original Message- >>> From: Felipe Balbi [mailto:ba...@ti.com] >>> Sent: Tuesday, July 14, 2015 12:29 AM >>> To: John Youn >>> Cc: ba...@ti.com

Several races in "usbnet" module (kernel 4.1.x)

2015-07-20 Thread Eugene Shatokhin
Hi, I have recently found several data races in "usbnet" module, checked on vanilla kernel 4.1.0 on x86_64. The races do actually happen, I have confirmed it by adding delays and using hardware breakpoints to detect the conflicting memory accesses (with RaceHound tool, https://github.com/winn

[PATCH] usb: dwc3: Reset the transfer resource index on SET_INTERFACE

2015-07-20 Thread John Youn
This fixes an issue introduced in a previous patch that added checking of the return status of device endpoint commands. It caught the Set Endpoint Transfer Resource command failing occasionally. This is because the Transfer Resource Index was not getting reset during a SET_INTERFACE request. Sign

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

2015-07-20 Thread Felipe Balbi
On Mon, Jul 20, 2015 at 06:16:46PM +, John Youn wrote: > On 7/20/2015 10:51 AM, Felipe Balbi wrote: > > Hi, > > > > On Wed, Jul 15, 2015 at 09:49:05AM +, Subbaraya Sundeep Bhatta wrote: > >> Hi John, > >> > >>> -Original Message- > >>> From: Felipe Balbi [mailto:ba...@ti.com] > >>>

Re: [PATCH v4 5/5] usb: gadget: mass_storage: Warn if LUNs ids are not contiguous

2015-07-20 Thread Sergei Shtylyov
Hello. On 07/20/2015 09:15 PM, Krzysztof Opasiak wrote: According to mass storage specification: "Logical Unit Numbers on the device shall be numbered contiguously starting from LUN 0 to a maximum LUN of 15 (Fh)" So let's at least print a warning message that LUNs ids should be contiguo

Remote wakeup in USB gadget drivers

2015-07-20 Thread Bin Liu
Hi, I was trying to understand how remote wakeup works in the gadget framework. It seems most UDC drivers define .wakeup() call, which is invoked by usb_gadget_wakeup() wrapper. But none of the gadget drivers (except g_zero) calls it. Does it mean the gadget framework does not support remote wake

Re: Remote wakeup in USB gadget drivers

2015-07-20 Thread Felipe Balbi
On Mon, Jul 20, 2015 at 02:02:30PM -0500, Bin Liu wrote: > Hi, > > I was trying to understand how remote wakeup works in the gadget framework. > > It seems most UDC drivers define .wakeup() call, which is invoked by > usb_gadget_wakeup() wrapper. But none of the gadget drivers (except > g_zero) c

Re: Remote wakeup in USB gadget drivers

2015-07-20 Thread Bin Liu
On Mon, Jul 20, 2015 at 2:10 PM, Felipe Balbi wrote: > On Mon, Jul 20, 2015 at 02:02:30PM -0500, Bin Liu wrote: >> Hi, >> >> I was trying to understand how remote wakeup works in the gadget framework. >> >> It seems most UDC drivers define .wakeup() call, which is invoked by >> usb_gadget_wakeup()

Re: Remote wakeup in USB gadget drivers

2015-07-20 Thread Felipe Balbi
On Mon, Jul 20, 2015 at 02:39:58PM -0500, Bin Liu wrote: > On Mon, Jul 20, 2015 at 2:10 PM, Felipe Balbi wrote: > > On Mon, Jul 20, 2015 at 02:02:30PM -0500, Bin Liu wrote: > >> Hi, > >> > >> I was trying to understand how remote wakeup works in the gadget framework. > >> > >> It seems most UDC dr

Re: Remote wakeup in USB gadget drivers

2015-07-20 Thread Bin Liu
On Mon, Jul 20, 2015 at 2:53 PM, Felipe Balbi wrote: > On Mon, Jul 20, 2015 at 02:39:58PM -0500, Bin Liu wrote: >> On Mon, Jul 20, 2015 at 2:10 PM, Felipe Balbi wrote: >> > On Mon, Jul 20, 2015 at 02:02:30PM -0500, Bin Liu wrote: >> >> Hi, >> >> >> >> I was trying to understand how remote wakeup

[PATCH 1/2] usb: dwc3: gadget: add a trace when disabling EPs

2015-07-20 Thread Felipe Balbi
We have a "Enabling %s" trace when enabling an endpoint but that message felt lonely without a matching "Disabling %s". Add it. Signed-off-by: Felipe Balbi --- drivers/usb/dwc3/gadget.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c in

[PATCH 2/2] usb: dwc3: gadget: defer endpoint name change

2015-07-20 Thread Felipe Balbi
We should only change endpoint names when we actually manage to enable/disable it. Signed-off-by: Felipe Balbi --- drivers/usb/dwc3/gadget.c | 42 +- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dw

Re: xhci_hcd 0000:00:14.0: ERROR Transfer event TRB DMA ptr not part of current TD ep_index 2 comp_code 1

2015-07-20 Thread Arkadiusz Miskiewicz
On Saturday 18 of July 2015, Arkadiusz Miskiewicz wrote: > Hi. > > I'm on 4.2.0-rc2-00077-gf760b87 kernel and while trying to copy some file > from usb storage (sata disk behind sata-usb bridge or pendrive; hapens in > both cases) copying process hangs just early after start with: Looks like susp

Re: [PATCH 1/2] usb: dwc3: gadget: add a trace when disabling EPs

2015-07-20 Thread Fabio Estevam
On Mon, Jul 20, 2015 at 5:09 PM, Felipe Balbi wrote: > We have a "Enabling %s" trace when enabling an endpoint > but that message felt lonely without a matching > "Disabling %s". Add it. > > Signed-off-by: Felipe Balbi > --- > drivers/usb/dwc3/gadget.c | 2 ++ > 1 file changed, 2 insertions(+) >

Re: [PATCH 1/2] usb: dwc3: gadget: add a trace when disabling EPs

2015-07-20 Thread Felipe Balbi
On Mon, Jul 20, 2015 at 05:36:34PM -0300, Fabio Estevam wrote: > On Mon, Jul 20, 2015 at 5:09 PM, Felipe Balbi wrote: > > We have a "Enabling %s" trace when enabling an endpoint > > but that message felt lonely without a matching > > "Disabling %s". Add it. > > > > Signed-off-by: Felipe Balbi > >

Re: [PATCH 1/2] usb: dwc3: gadget: add a trace when disabling EPs

2015-07-20 Thread Felipe Balbi
On Mon, Jul 20, 2015 at 03:38:17PM -0500, Felipe Balbi wrote: > On Mon, Jul 20, 2015 at 05:36:34PM -0300, Fabio Estevam wrote: > > On Mon, Jul 20, 2015 at 5:09 PM, Felipe Balbi wrote: > > > We have a "Enabling %s" trace when enabling an endpoint > > > but that message felt lonely without a matchin

[RFC ebeam PATCH 1/2] hid: Blacklist eBeam devices

2015-07-20 Thread Yann Cantin
Signed-off-by: Yann Cantin --- drivers/hid/hid-core.c | 6 ++ drivers/hid/hid-ids.h | 6 ++ 2 files changed, 12 insertions(+) diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c index 157c627..db4c7ee 100644 --- a/drivers/hid/hid-core.c +++ b/drivers/hid/hid-core.c @@ -2267,6 +

[RFC ebeam PATCH 2/2] input: misc: New USB eBeam input driver

2015-07-20 Thread Yann Cantin
Signed-off-by: Yann Cantin --- Documentation/ABI/testing/sysfs-driver-ebeam | 53 ++ drivers/input/misc/Kconfig | 22 + drivers/input/misc/Makefile | 1 + drivers/input/misc/ebeam.c | 777 +++ 4 files changed, 853 in

[RFC ebeam PATCH 0/3] new USB eBeam input driver

2015-07-20 Thread Yann Cantin
Hi, New USB input driver for eBeam devices. Currently supported (tested) : - Luidia eBeam classic projection and edge projection models - Nec "interactive solution" NP01Wi1 & NP01Wi2 accessories. Patch 1 to blacklist the devices for hid generic-usb. Patch 2 is the actual driver. Notable stuff

Re: [RFC ebeam PATCH 2/2] input: misc: New USB eBeam input driver

2015-07-20 Thread Greg KH
On Mon, Jul 20, 2015 at 11:03:19PM +0200, Yann Cantin wrote: > Signed-off-by: Yann Cantin You need some kind of text in the changelog here... > --- > Documentation/ABI/testing/sysfs-driver-ebeam | 53 ++ > drivers/input/misc/Kconfig | 22 + > drivers/input/misc/Makefile

Re: [RFC ebeam PATCH 2/2] input: misc: New USB eBeam input driver

2015-07-20 Thread Dmitry Torokhov
On Mon, Jul 20, 2015 at 02:59:56PM -0700, Greg KH wrote: > On Mon, Jul 20, 2015 at 11:03:19PM +0200, Yann Cantin wrote: > > Signed-off-by: Yann Cantin > > > + > > + /* sysfs setup */ > > + err = sysfs_create_group(&intf->dev.kobj, &ebeam_attr_group); > > Ick, you just added the sysfs files t

Re: [PATCH 1/4] usb: gadget: udc: bdc: Fix a driver crash on disconnect

2015-07-20 Thread Alan Cooper
I did do a build for ARM dove and x86_64 and didn't see any build errors. Can you send me the build error? Thanks Al On Mon, Jul 20, 2015 at 1:15 PM, Felipe Balbi wrote: > On Thu, Jul 16, 2015 at 05:47:35PM -0400, Al Cooper wrote: >> ep_dequeue() in bdc_ep.c was capturing the hw dequeue pointer

Re: [RFC ebeam PATCH 2/2] input: misc: New USB eBeam input driver

2015-07-20 Thread Greg KH
On Mon, Jul 20, 2015 at 03:26:40PM -0700, Dmitry Torokhov wrote: > On Mon, Jul 20, 2015 at 02:59:56PM -0700, Greg KH wrote: > > On Mon, Jul 20, 2015 at 11:03:19PM +0200, Yann Cantin wrote: > > > Signed-off-by: Yann Cantin > > > > > + > > > + /* sysfs setup */ > > > + err = sysfs_create_group(&int

Re: [RFC ebeam PATCH 2/2] input: misc: New USB eBeam input driver

2015-07-20 Thread Dmitry Torokhov
On Mon, Jul 20, 2015 at 03:40:33PM -0700, Greg KH wrote: > On Mon, Jul 20, 2015 at 03:26:40PM -0700, Dmitry Torokhov wrote: > > On Mon, Jul 20, 2015 at 02:59:56PM -0700, Greg KH wrote: > > > On Mon, Jul 20, 2015 at 11:03:19PM +0200, Yann Cantin wrote: > > > > Signed-off-by: Yann Cantin > > > > >

Re: [PATCH 1/4] usb: gadget: udc: bdc: Fix a driver crash on disconnect

2015-07-20 Thread Felipe Balbi
Hi, (no top-posting, please) On Mon, Jul 20, 2015 at 06:35:49PM -0400, Alan Cooper wrote: > I did do a build for ARM dove and x86_64 and didn't see any build > errors. Can you send me the build error? do I really have to send the error ? Just look at the code: - temp = bdc_readl(bdc->regs

Re: [PATCH 1/1] usb: chipidea: host: add own hc_driver for each hcd

2015-07-20 Thread Peter Chen
On Mon, Jul 20, 2015 at 11:37:11AM -0400, Alan Stern wrote: > On Mon, 20 Jul 2015, Peter Chen wrote: > > > > I don't think a new use case is needed. All you have to do is add > > > something like this to host.c: > > > > > > static void ci_hdrc_host_driver_init(void) > > > { > > > ehci_init_dri

[PATCH V5 1/1] usb:serial add Fintek F81532/534 driver

2015-07-20 Thread Peter Hung
This driver is for Fintek F81532/F81534 USB to Serial Ports IC. Features: 1. F81534 is 1-to-4 & F81532 is 1-to-2 serial ports IC 2. Support Baudrate from B50 to B150 (excluding B100). 3. The RTS signal can be transformed their behavior with configuration by default ioctl TIOCGRS485/TIOC

[PATCH 1/1] usb: chipidea: ehci_init_driver is intended to call one time

2015-07-20 Thread Peter Chen
The ehci_init_driver is used to initialize hcd APIs for each ehci controller driver, it is designed to be called only one time and before driver register is called. The current design will cause ehci_init_driver is called multiple times at probe process, it will cause hc_driver's initialization aff

RE: [PATCH] usb: dwc3: Reset the transfer resource index on SET_INTERFACE

2015-07-20 Thread Subbaraya Sundeep Bhatta
Hi, > -Original Message- > From: John Youn [mailto:john.y...@synopsys.com] > Sent: Monday, September 17, 2001 12:30 PM > To: ba...@ti.com; Subbaraya Sundeep Bhatta > Cc: linux-usb@vger.kernel.org > Subject: [PATCH] usb: dwc3: Reset the transfer resource index on > SET_INTERFACE > > This f

Re: [PATCH] Bluetooth: btusb: match generic class code in interface descriptor

2015-07-20 Thread Sjoerd Simons
On Fri, 2015-07-17 at 11:12 -0600, Daniel Drake wrote: > btusb currently has a generic match on USB device descriptors: > { USB_DEVICE_INFO(0xe0, 0x01, 0x01) }, > > However, http://www.usb.org/developers/defined_class states: > > Base Class E0h (Wireless Controller) > This base class