Re: [PATCH] usb: gadget: function: Remove redundant usb_free_all_descriptors

2014-10-13 Thread Robert Baldyga
Hi, On 10/10/2014 03:09 PM, pavi1729 ivap wrote: > From: Pavitra > Date: Fri, 10 Oct 2014 16:05:30 +0530 > Subject: [PATCH] usb: gadget: function: Remove redundant > usb_free_all_descriptors > > Removed the usb_free_all_descriptors call in *_bind functions > as this call is already present in u

Re: [PATCH v6 07/12] usb: chipidea: add a usb2 driver for ci13xxx

2014-10-13 Thread Peter Chen
On Thu, Sep 25, 2014 at 07:37:50PM -0500, Felipe Balbi wrote: > HI, > > On Fri, Sep 26, 2014 at 07:39:34AM +0800, Peter Chen wrote: > > On Thu, Sep 25, 2014 at 09:15:53AM -0500, Felipe Balbi wrote: > > > On Wed, Sep 24, 2014 at 02:23:38PM +0200, Arnd Bergmann wrote: > > > > On Wednesday 24 Septemb

Re: [PATCH] usb: gadget: function: Remove redundant usb_free_all_descriptors

2014-10-13 Thread pavi1729 Sid
On Mon, Oct 13, 2014 at 2:02 PM, Robert Baldyga wrote: > Hi, > > On 10/10/2014 03:09 PM, pavi1729 ivap wrote: >> From: Pavitra >> Date: Fri, 10 Oct 2014 16:05:30 +0530 >> Subject: [PATCH] usb: gadget: function: Remove redundant >> usb_free_all_descriptors >> >> Removed the usb_free_all_descripto

[PATCH v2] usb: musb: musb_dsps: fix NULL pointer in suspend

2014-10-13 Thread Sebastian Andrzej Siewior
So testing managed to configure musb in DMA mode but not load the matching cppi41 driver for DMA. This results in |musb-hdrc musb-hdrc.0.auto: Failed to request rx1. |musb-hdrc musb-hdrc.0.auto: musb_init_controller failed with status -517 |platform musb-hdrc.0.auto: Driver musb-hdrc requests prob

Re: [PATCH] usb: musb: dsps: kill OTG timer on suspend

2014-10-13 Thread Sebastian Andrzej Siewior
On 2014-09-15 09:39:09 [-0500], Felipe Balbi wrote: > if we don't make sure to kill the timer, it could > expire after we have already gated our clocks. > > That will trigger a Data Abort exception because > we would try to access register while clock is gated. That timer deserves to be killed be

RE: RCU bug with v3.17-rc3 ?

2014-10-13 Thread David Laight
From: Nathan Lynch > On 10/10/2014 11:25 AM, Russell King - ARM Linux wrote: > > > > Right, so GCC 4.8.{1,2} are totally unsuitable for kernel building (and > > it seems that this has been known about for some time.) > > Looking at http://gcc.gnu.org/PR58854 it seems that all 4.8.x for x < 3 > are

Subject: [PATCH] usb: gadget: function: Remove redundant usb_free_all_descriptors

2014-10-13 Thread pavi1729 Sid
Removed the usb_free_all_descriptors call in *_bind functions as this call is already present in usb_assign_descriptors. usb_assign_descriptors is the only call where usb descriptor allocation happens, also in case of error freeing of the allocated memory takes place in the same call. Hence the cal

[gadget] Testing midi function

2014-10-13 Thread Andrzej Pietrasiewicz
Hi all, I would like to try the midi function of usb gadget. Can anybody please tell me: 1) What userspace programs do I need at the gadget side? How to run them? 2) What do I need at the host side? How to tell whether my midi gadget is working correctly? AP -- To unsubscribe from this list:

[PATCH] usb: musb: dsps: start OTG timer on resume again

2014-10-13 Thread Sebastian Andrzej Siewior
Commit 468bcc2a2ca ("usb: musb: dsps: kill OTG timer on suspend") stopped the timer in suspend path but forgot the re-enable it in the resume path. This patch fixes the behaviour. Cc: # v3.14+ Fixes 468bcc2a2ca "usb: musb: dsps: kill OTG timer on suspend" Signed-off-by: Sebastian Andrzej Siewior

Re: [gadget] Testing midi function

2014-10-13 Thread Clemens Ladisch
Andrzej Pietrasiewicz wrote: > I would like to try the midi function of usb gadget. > > Can anybody please tell me: > > 1) What userspace programs do I need at the gadget side? I've never used it, but as far as I can see, the MIDI function ends up as a normal sound card with MIDI port(s). Your de

Re: [gadget] Testing midi function

2014-10-13 Thread Andrzej Pietrasiewicz
Hi Clemens, W dniu 13.10.2014 o 12:44, Clemens Ladisch pisze: Andrzej Pietrasiewicz wrote: I've never used it, but as far as I can see, the MIDI function ends up as a normal sound card with MIDI port(s). Your device firmware would access them through the normal ALSA rawmidi or sequencer A

Re: RCU bug with v3.17-rc3 ?

2014-10-13 Thread Russell King - ARM Linux
On Mon, Oct 13, 2014 at 09:11:34AM +, David Laight wrote: > From: Nathan Lynch > > On 10/10/2014 11:25 AM, Russell King - ARM Linux wrote: > > > > > > Right, so GCC 4.8.{1,2} are totally unsuitable for kernel building (and > > > it seems that this has been known about for some time.) > > > > L

Re: [PATCH 1/2] chipidea: core: Remove unneeded OOM message

2014-10-13 Thread Fabio Estevam
On Mon, Oct 13, 2014 at 2:53 AM, Peter Chen wrote: > On Sun, Oct 12, 2014 at 11:26:33PM -0300, Fabio Estevam wrote: >> From: Fabio Estevam >> >> MM core code already complains when devm_kzalloc() fails, so no need to print >> the error locally. >> > > For the title: you may need to add "usb" firs

[PATCH v2 2/2] usb: chipidea: ci_hdrc_imx.c: Remove unneeded OOM message

2014-10-13 Thread Fabio Estevam
MM core code already complains when devm_kzalloc() fails, so no need to print the error locally. Signed-off-by: Fabio Estevam --- Changes since v1: - Add usb in the Subject drivers/usb/chipidea/ci_hdrc_imx.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/usb/chip

Re: xHCI bug

2014-10-13 Thread Mathias Nyman
Hi On 10.10.2014 21:47, Felipe Balbi wrote: > Hi Mathias, > > I seem to be able to rather easily kill xHCI by just running > test.sh/testusb with a USB2 device attached to it (I used my AM437x > Starter Kit for the test). > > I noticed that after not too long, all tests started failing (see belo

Re: xHCI bug

2014-10-13 Thread Mathias Nyman
On 10.10.2014 22:01, Tony Lindgren wrote: > * Felipe Balbi [141010 11:51]: >> Hi again, >> >> On Fri, Oct 10, 2014 at 01:47:55PM -0500, Felipe Balbi wrote: >>> I seem to be able to rather easily kill xHCI by just running >>> test.sh/testusb with a USB2 device attached to it (I used my AM437x >>> S

Re: Subject: [PATCH] usb: gadget: function: Remove redundant usb_free_all_descriptors

2014-10-13 Thread Robert Baldyga
Hi, "Subject: " at the beginning of your email subject looks unnecessary. When you're sending next version of your patch, you should add subject prefix "[PATCH vN]", when N is number of version (eg. [PATCH v2]). See Documentation/SubmittingPatches. ('git format-patch --subject-prefix' is helpful)

[PATCH v2 1/2] usb: chipidea: core: Remove unneeded OOM message

2014-10-13 Thread Fabio Estevam
MM core code already complains when devm_kzalloc() fails, so no need to print the error locally. Signed-off-by: Fabio Estevam --- Changes since v1: - Add usb in the Subject drivers/usb/chipidea/core.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/usb/chipidea/co

Re: Subject: [PATCH] usb: gadget: function: Remove redundant usb_free_all_descriptors

2014-10-13 Thread pavi1729 Sid
On Mon, Oct 13, 2014 at 6:55 PM, Robert Baldyga wrote: > Hi, > > "Subject: " at the beginning of your email subject looks unnecessary. > > When you're sending next version of your patch, you should add subject > prefix "[PATCH vN]", when N is number of version (eg. [PATCH v2]). See > Documentation

Re: [PATCH][RFC] storage: Reject bogus max LUN values

2014-10-13 Thread Matthew Dharm
Is there a constant we can pull from a SCSI header, instead of having a "magic number" in the code? Matt On Sun, Oct 12, 2014 at 5:54 PM, Alan Stern wrote: > On Sun, 12 Oct 2014, Mark Knibbs wrote: > >> --- linux-3.17/drivers/usb/storage/transport.c.orig 2014-10-05 >> 20:12:36.0 +0100

Re: xHCI bug

2014-10-13 Thread Felipe Balbi
On Mon, Oct 13, 2014 at 03:38:24PM +0300, Mathias Nyman wrote: > Hi > > On 10.10.2014 21:47, Felipe Balbi wrote: > > Hi Mathias, > > > > I seem to be able to rather easily kill xHCI by just running > > test.sh/testusb with a USB2 device attached to it (I used my AM437x > > Starter Kit for the tes

[PATCH v2] usb: gadget: function: Remove redundant usb_free_all_descriptors

2014-10-13 Thread pavi1729 Sid
Removed the usb_free_all_descriptors call in *_bind functions as this call is already present in usb_assign_descriptors. usb_assign_descriptors is the only call where usb descriptor allocation happens and in case of error, freeing of the allocated memory takes place inside the same call. Hence the

Re: [RFC PATCH 3/4] usb: dwc3: add quirk to be compatible for AMD NL

2014-10-13 Thread Felipe Balbi
Hi, On Sat, Oct 11, 2014 at 01:22:58PM +0800, Huang Rui wrote: > On Sat, Oct 11, 2014 at 01:14:44PM +0800, Huang Rui wrote: > > On Fri, Oct 10, 2014 at 09:04:15AM -0500, Felipe Balbi wrote: > > > Hi, > > > > > > On Fri, Oct 10, 2014 at 05:25:34PM +0800, Huang Rui wrote: > > > > > > I enabled dwc3

Re: XHCI host dies with LPM + webcam gadget

2014-10-13 Thread Mathias Nyman
On 25.09.2014 14:08, Amit Virdi wrote: > Hello Mathias, > > I'm trying to connect a webcam gadget on USB3.0 interface with xHCI host when > LPM is enabled. My xHCI host is TI's TUSB7340EVM. xHCI host stops responding > as soon as the webcam gadget module is inserted on the device side (The > de

Re: [PATCH v2] usb: ffs: fix regression when quirk_ep_out_aligned_size flag is set

2014-10-13 Thread Felipe Balbi
On Wed, Oct 08, 2014 at 02:12:18PM -0700, David Cohen wrote: > The commit '2e4c7553cd usb: gadget: f_fs: add aio support' broke the > quirk implemented to align buffer size to maxpacketsize on out endpoint. > As result, functionfs does not work on Intel platforms using dwc3 driver > (i.e. Bay Trail

Re: [PATCH v2] usb: atm: fix style issues

2014-10-13 Thread Jonas Brunsgaard
I will try again. This is my first commit to the kernel and the motivation for me is to learn the flow regarding patch submissions. So I am glad for the feedback, just remember to comment/criticize the code so I know what to improve ;) A new patch will be in your mailbox soon. Good day -- To uns

Re: [PATCH v2] usb: ffs: fix regression when quirk_ep_out_aligned_size flag is set

2014-10-13 Thread David Cohen
On Mon, Oct 13, 2014 at 10:32:12AM -0500, Felipe Balbi wrote: > On Wed, Oct 08, 2014 at 02:12:18PM -0700, David Cohen wrote: > > The commit '2e4c7553cd usb: gadget: f_fs: add aio support' broke the > > quirk implemented to align buffer size to maxpacketsize on out endpoint. > > As result, functionf

Re: [PATCH v2] usb: ffs: fix regression when quirk_ep_out_aligned_size flag is set

2014-10-13 Thread Felipe Balbi
On Mon, Oct 13, 2014 at 09:55:56AM -0700, David Cohen wrote: > On Mon, Oct 13, 2014 at 10:32:12AM -0500, Felipe Balbi wrote: > > On Wed, Oct 08, 2014 at 02:12:18PM -0700, David Cohen wrote: > > > The commit '2e4c7553cd usb: gadget: f_fs: add aio support' broke the > > > quirk implemented to align b

[PATCH v3] usb: ffs: fix regression when quirk_ep_out_aligned_size flag is set

2014-10-13 Thread David Cohen
The commit '2e4c7553cd usb: gadget: f_fs: add aio support' broke the quirk implemented to align buffer size to maxpacketsize on out endpoint. As result, functionfs does not work on Intel platforms using dwc3 driver (i.e. Bay Trail and Merrifield). This patch fixes the issue. This code is based on

Re: [PATCH][RFC] storage: Reject bogus max LUN values

2014-10-13 Thread Mark Knibbs
On Mon, 13 Oct 2014 07:14:39 -0700 Matthew Dharm wrote: > Is there a constant we can pull from a SCSI header, instead of having > a "magic number" in the code? I don't know, but it would probably be in a USB-related header; isn't the 4-bits-for-LUN a USB mass storage bulk only thing? I guess yo

Re: [PATCH] usb: musb: dsps: kill OTG timer on suspend

2014-10-13 Thread Felipe Balbi
On Mon, Oct 13, 2014 at 11:13:40AM +0200, Sebastian Andrzej Siewior wrote: > On 2014-09-15 09:39:09 [-0500], Felipe Balbi wrote: > > if we don't make sure to kill the timer, it could > > expire after we have already gated our clocks. > > > > That will trigger a Data Abort exception because > > we

Re: [PATCH v2 1/4] dwc3: exynos: Add support for SCLK present on Exynos7

2014-10-13 Thread Felipe Balbi
Hi, On Mon, Oct 13, 2014 at 01:54:59PM +0900, Anton Tikhomirov wrote: > Hi Vivek, > > > Exynos7 also has a separate special gate clock going to the IP > > apart from the usual AHB clock. So add support for the same. > > As we discussed before, Exynos7 SoCs have 7 clocks to be controlled > by the

Re: [PATCH v2 1/4] dwc3: exynos: Add support for SCLK present on Exynos7

2014-10-13 Thread Tomasz Figa
Hi Anton, On 13.10.2014 06:54, Anton Tikhomirov wrote: > Hi Vivek, > >> Exynos7 also has a separate special gate clock going to the IP >> apart from the usual AHB clock. So add support for the same. > > As we discussed before, Exynos7 SoCs have 7 clocks to be controlled > by the driver. Adding o

Re: [PATCH v2 2/2] usb: chipidea: ci_hdrc_imx.c: Remove unneeded OOM message

2014-10-13 Thread Peter Chen
On Mon, Oct 13, 2014 at 08:58:30AM -0300, Fabio Estevam wrote: > MM core code already complains when devm_kzalloc() fails, so no need to print > the error locally. > > Signed-off-by: Fabio Estevam > --- > Changes since v1: > - Add usb in the Subject > > drivers/usb/chipidea/ci_hdrc_imx.c | 4 +-

RE: [PATCH v2 1/4] dwc3: exynos: Add support for SCLK present on Exynos7

2014-10-13 Thread Anton Tikhomirov
Hello, > Hi Anton, > > On 13.10.2014 06:54, Anton Tikhomirov wrote: > > Hi Vivek, > > > >> Exynos7 also has a separate special gate clock going to the IP > >> apart from the usual AHB clock. So add support for the same. > > > > As we discussed before, Exynos7 SoCs have 7 clocks to be controlled >

Re: RCU bug with v3.17-rc3 ?

2014-10-13 Thread Greg KH
On Mon, Oct 13, 2014 at 12:43:07PM +0100, Russell King - ARM Linux wrote: > On Mon, Oct 13, 2014 at 09:11:34AM +, David Laight wrote: > > From: Nathan Lynch > > > On 10/10/2014 11:25 AM, Russell King - ARM Linux wrote: > > > > > > > > Right, so GCC 4.8.{1,2} are totally unsuitable for kernel bu

Fwd: [Bug 86201] ch341-uart ttyUSB0: usb_serial_generic_write_bulk_callback - nonzero urb status: -71

2014-10-13 Thread Cristian
Hello! Bug tracking Regards, -- Cristian -- Forwarded message -- From: Date: 2014-10-13 23:20 GMT-03:00 Subject: [Bug 86201] ch341-uart ttyUSB0: usb_serial_generic_write_bulk_callback - nonzero urb status: -71 To: carav...@gmail.com https://bugzilla.kernel.org/show_bug.cgi

Re: Fwd: [Bug 86201] ch341-uart ttyUSB0: usb_serial_generic_write_bulk_callback - nonzero urb status: -71

2014-10-13 Thread Greg Kroah-Hartman
On Mon, Oct 13, 2014 at 11:32:22PM -0300, Cristian wrote: > Hello! > > Bug tracking What do you mean by this? Please send the information that was in the bug to this list so that people can see it and try to help. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe li

Re: Fwd: [Bug 86201] ch341-uart ttyUSB0: usb_serial_generic_write_bulk_callback - nonzero urb status: -71

2014-10-13 Thread Cristian
Hello! The message appears when I'm printing and disconnect USB-Serial cable Regards, -- Cristian 2014-10-13 23:38 GMT-03:00 Greg Kroah-Hartman : > On Mon, Oct 13, 2014 at 11:32:22PM -0300, Cristian wrote: >> Hello! >> >> Bug tracking > > What do you mean by this? > > Please send the informati

Re: [PATCH v2 1/4] dwc3: exynos: Add support for SCLK present on Exynos7

2014-10-13 Thread Vivek Gautam
Hi Tomasz, On Tue, Oct 14, 2014 at 6:56 AM, Anton Tikhomirov wrote: > Hello, > >> Hi Anton, >> >> On 13.10.2014 06:54, Anton Tikhomirov wrote: >> > Hi Vivek, >> > >> >> Exynos7 also has a separate special gate clock going to the IP >> >> apart from the usual AHB clock. So add support for the sam

Re: [PATCH v2 1/4] dwc3: exynos: Add support for SCLK present on Exynos7

2014-10-13 Thread Vivek Gautam
Hi Felipe, On Tue, Oct 14, 2014 at 4:14 AM, Felipe Balbi wrote: > Hi, > > On Mon, Oct 13, 2014 at 01:54:59PM +0900, Anton Tikhomirov wrote: >> Hi Vivek, >> >> > Exynos7 also has a separate special gate clock going to the IP >> > apart from the usual AHB clock. So add support for the same. >> >>

[PATCH 1/1] usb: option: add support for Telit LE910

2014-10-13 Thread Daniele Palmas
option driver, added VID/PID for Telit LE910 modem. Interfaces description is almost the same than LE920, except that the qmi interface is number 2 (instead than 5). Signed-off-by: Daniele Palmas --- drivers/usb/serial/option.c |8 1 files changed, 8 insertions(+), 0 deletions(-) d