Re: [PATCHv2 3/3] usb: gadget: uvc: configfs support in uvc function

2014-12-09 Thread Andrzej Pietrasiewicz
W dniu 07.12.2014 o 23:27, Laurent Pinchart pisze: Hi Andrzej, Thank you for the patch. I think we're nearly there :-) Please see inline @@ -787,25 +789,104 @@ static void uvc_free_inst(struct usb_function_instance *f) [snip] static void uvc_free(struct usb_function *f) {

chipidea hangs on mx51 running linux-next

2014-12-09 Thread Fabio Estevam
Hi, I am running linux-next 20141208 on a imx51-babbage board and the following hang is observed: [1.243660] usbcore: registered new interface driver usb-storage [1.261619] ci_hdrc ci_hdrc.1: doesn't support gadget [1.266730] ci_hdrc ci_hdrc.1: EHCI Host Controller [1.272058] ci_h

[PATCH] usb: dwc2: gadget: kill requests with 'force' in s3c_hsotg_udc_stop()

2014-12-09 Thread Robert Baldyga
This makes us sure that all requests are completed before we unbind gadget. There are assumptions in gadget API that all requests have to be completed and leak of complete can break some usb function drivers. For example unbind of ECM function can cause NULL pointer dereference: [ 26.396595] co

Re: chipidea hangs on mx51 running linux-next

2014-12-09 Thread Matthieu CASTET
Le Tue, 9 Dec 2014 11:09:55 -0200, Fabio Estevam a écrit : > Hi, > > I am running linux-next 20141208 on a imx51-babbage board and the > following hang is observed: > > [1.243660] usbcore: registered new interface driver usb-storage > [1.261619] ci_hdrc ci_hdrc.1: doesn't support gadget

PROBLEM: [ums-realtek] media change events incorrectly implemented?

2014-12-09 Thread Rubén Herrero
With systemd 217, systemd-udevd complains every ~10 seconds (see below). No error shows up on 216. The systemd people claim this is a drivers issue. (See: https://bugs.freedesktop.org/show_bug.cgi?id=86414#c5) >Lennart Poettering: >Seems to be your card reader. The "ums-realtek" kernel USB driver

Re: ptp-gadget (by denx) on beaglebone black

2014-12-09 Thread Felipe Balbi
Hi, On Tue, Dec 09, 2014 at 03:47:24AM +, ivo welch wrote: > > dear usb experts---per request in the readme file, I am asking the question > here. > > I am trying to get a ptp-gadget device-side implementation of the Picture > Transfer Protocol to work on a beaglebone black (AM335x), runnin

Re: [PATCH] usb: hcd: get/put device and hcd for hcd_buffers()

2014-12-09 Thread 'Greg Kroah-Hartman'
On Mon, Dec 08, 2014 at 09:44:05AM +, David Laight wrote: > From: Greg Kroah-Hartman > > On Fri, Dec 05, 2014 at 09:03:57PM +0100, Sebastian Andrzej Siewior wrote: > > > Consider the following scenario: > > > - plugin a webcam > > > - play the stream via gst-launch-0.10 v4l2src device=/dev/vide

Re: [PATCH] usb: hcd: get/put device and hcd for hcd_buffers()

2014-12-09 Thread Sebastian Andrzej Siewior
On 12/09/2014 04:24 PM, 'Greg Kroah-Hartman' wrote: > On Mon, Dec 08, 2014 at 09:44:05AM +, David Laight wrote: >> From: Greg Kroah-Hartman >>> On Fri, Dec 05, 2014 at 09:03:57PM +0100, Sebastian Andrzej Siewior wrote: Consider the following scenario: - plugin a webcam - play the

Re: Modifying USB Audio gadget driver

2014-12-09 Thread Priya S
On Wed, Dec 3, 2014 at 6:06 PM, Felipe Balbi wrote: > Hi, > > (no top-posting, please) > > On Wed, Dec 03, 2014 at 05:24:22PM -0600, Priya S wrote: >> Great! That fixes it. Thanks. >> >> Windows is not able to install the driver though. These are the USB >> descriptors that I see from the host-sid

[PATCH 1/2] usb: chipidea: usbmisc: skip clocks on i.MX6

2014-12-09 Thread Philipp Zabel
On i.MX6Q, the USBMISC registers are clocked by the IPG clock, so there is no need to enable USBOH3 for the USBMISC driver. Signed-off-by: Philipp Zabel --- drivers/usb/chipidea/usbmisc_imx.c | 32 ++-- 1 file changed, 18 insertions(+), 14 deletions(-) diff --git a/d

[PATCH 2/2] ARM: dts: imx6qdl: remove USBOH3 clock from usbmisc, it is clocked by IPG

2014-12-09 Thread Philipp Zabel
On i.MX6Q, the USBMISC registers are clocked by the IPG clock, so there is no need to enable USBOH3 for the USBMISC driver. Signed-off-by: Philipp Zabel --- arch/arm/boot/dts/imx6qdl.dtsi | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qd

Re: [PATCH] usb: hcd: get/put device and hcd for hcd_buffers()

2014-12-09 Thread 'Greg Kroah-Hartman'
On Tue, Dec 09, 2014 at 05:01:35PM +0100, Sebastian Andrzej Siewior wrote: > On 12/09/2014 04:24 PM, 'Greg Kroah-Hartman' wrote: > > On Mon, Dec 08, 2014 at 09:44:05AM +, David Laight wrote: > >> From: Greg Kroah-Hartman > >>> On Fri, Dec 05, 2014 at 09:03:57PM +0100, Sebastian Andrzej Siewior

Re: [PATCH net-next] r8152: redefine REALTEK_USB_DEVICE

2014-12-09 Thread David Miller
From: Hayes Wang Date: Thu, 4 Dec 2014 10:43:11 +0800 > Redefine REALTEK_USB_DEVICE for the desired USB interface for probe(). > There are three USB interfaces for the device. USB_CLASS_COMM and > USB_CLASS_CDC_DATA are for ECM mode (config #2). USB_CLASS_VENDOR_SPEC > is for the vendor mode (con

UAS crash with Apricorn USB3 SATA bridge

2014-12-09 Thread Darrick J. Wong
Hi, I have an Apricorn USB 3 disk dongle thing that claims to support UAS. However, the kernel crashes when I plug it in[1]. I'm not sure what this is caused by, but I also have an ASMedia 2105 SATA bridge that works with UAS just fine. Not sure if the Apricorn thing is simply broken, or if this

Re: [PATCH 0/20] fix misspelling of current function in string

2014-12-09 Thread Julian Calaby
Hi Julia, On Mon, Dec 8, 2014 at 5:43 PM, Julia Lawall wrote: > On Mon, 8 Dec 2014, Julian Calaby wrote: > >> Hi Julia, >> >> On Mon, Dec 8, 2014 at 6:20 AM, Julia Lawall wrote: >> > These patches replace what appears to be a reference to the name of the >> > current function but is misspelled i

Re: ptp-gadget (by denx) on beaglebone black

2014-12-09 Thread Greg KH
On Tue, Dec 09, 2014 at 03:47:24AM +, ivo welch wrote: > > dear usb experts---per request in the readme file, I am asking the question > here. > > I am trying to get a ptp-gadget device-side implementation of the Picture > Transfer Protocol to work on a beaglebone black (AM335x), running deb

Re: VFS and LSM issues

2014-12-09 Thread Greg KH
On Mon, Dec 08, 2014 at 06:04:30PM +0100, Krzysztof Opasiak wrote: > Dear All, > > I'm Krzysztof Opasiak from SRPOL (Samsung). I'm working on USB support > in Tizen and mainline. In those works we use two Virtual File Systems > - ConfigFS and FunctionFS. Recently I have tried to use them with > SM

Re: ptp-gadget (by denx) on beaglebone black

2014-12-09 Thread ivo welch
yes, please ignore my ptp-gadget question. it was naive of me. the developer (denx) had stated this posting request in his readme file, and I simply followed it without thinking. it's probably abandoned code by now. is there a common ptp gadget client (for common embedded linux gadget devices)

Re: [PATCH] usb: chipidea: replace udelay in loops with usleep_range

2014-12-09 Thread Peter Chen
On Mon, Dec 08, 2014 at 06:15:38PM +0100, Heiner Kallweit wrote: > Both delays are at the lower end of where the use of usleep_range > is recommended. However as both udelay's occur in loops I think it > makes sense to replace them with sleeping equivalents to avoid > longer busy-waits. > > Signed

Re: [PATCH 1/2] usb: chipidea: usbmisc: skip clocks on i.MX6

2014-12-09 Thread Peter Chen
On Tue, Dec 09, 2014 at 05:13:35PM +0100, Philipp Zabel wrote: > On i.MX6Q, the USBMISC registers are clocked by the IPG clock, > so there is no need to enable USBOH3 for the USBMISC driver. > Access the registers at usbmisc needs the same clock with chipidea core, both are usboh3. For i.mx6 ser

Re: [PATCH 1/2] usb: chipidea: usbmisc: skip clocks on i.MX6

2014-12-09 Thread Uwe Kleine-König
Hello Philipp, On Tue, Dec 09, 2014 at 05:13:35PM +0100, Philipp Zabel wrote: > + /* on i.MX6Q, the USBMISC register space is clocked by the IPG clock */ > + if (data->ops != &imx6q_usbmisc_ops) { > + data->clk = devm_clk_get(&pdev->dev, NULL); I wonder if it was better/simpler