Re: [PATCH 3/4] usbdevfs: Use scatter-gather lists for large bulk transfers

2012-07-07 Thread Hans de Goede
Hi, On 07/06/2012 07:51 PM, Greg Kroah-Hartman wrote: On Wed, Jul 04, 2012 at 09:18:03AM +0200, Hans de Goede wrote: +static void snoop_urb_data(struct urb *urb, unsigned len) +{ + int i, size; + + if (!usbfs_snoop) + return; + + if (urb->num_sgs == 0) { +

Re: [PATCH 4/4] usb/hcd: Ensure scatter-gather is not used for isoc transfers

2012-07-07 Thread Hans de Goede
Hi, On 07/06/2012 07:52 PM, Greg Kroah-Hartman wrote: On Wed, Jul 04, 2012 at 09:18:04AM +0200, Hans de Goede wrote: We don't support sg for isoc transfers, enforce this. Signed-off-by: Hans de Goede Acked-by: Alan Stern --- drivers/usb/core/hcd.c |7 ++- 1 file changed, 6 inserti

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Oleksij Rempel
On 07.07.2012 11:20, Eric Ding wrote: > On 07/06/2012 11:47 PM, Alan Stern wrote: >> On Fri, 6 Jul 2012, Alan Cox wrote: >> Yes, but we still need to know the reason why. Neither Rafael nor I has been able to figure out why that commit messed things up. >>> >>> Was the driver doing any

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Eric Ding
On 07/07/2012 06:09 PM, Oleksij Rempel wrote: > On 07.07.2012 11:20, Eric Ding wrote: >> On 07/06/2012 11:47 PM, Alan Stern wrote: >>> On Fri, 6 Jul 2012, Alan Cox wrote: >>> > Yes, but we still need to know the reason why. Neither Rafael nor I > has been able to figure out why that commi

Re: [PATCH 4/5] usb: gadget/uvc: Port UVC webcam gadget to use videobuf2 framework

2012-07-07 Thread Laurent Pinchart
Hi Bhupesh, On Tuesday 03 July 2012 23:42:59 Bhupesh SHARMA wrote: > Hi Laurent, > > Thanks for your review and sorry for being late with my replies. > I have a lot on my plate these days :) No worries, I'm no less busy anyway :-) > On Tuesday, June 19, 2012 4:19 AM Laurent Pinchart wrote: > >

Re: [PATCH 5/5] usb: gadget/uvc: Add support for 'USB_GADGET_DELAYED_STATUS' response for a set_intf(alt-set 1) command

2012-07-07 Thread Laurent Pinchart
Hi Bhupesh, On Tuesday 03 July 2012 23:47:14 Bhupesh SHARMA wrote: > On Wednesday, June 20, 2012 3:19 AM Laurent Pinchart wrote: > > On Friday 01 June 2012 15:08:58 Bhupesh Sharma wrote: > > > This patch adds the support in UVC webcam gadget design for providing > > > USB_GADGET_DELAYED_STATUS in

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Oleksij Rempel
On 07.07.2012 13:38, Eric Ding wrote: > On 07/07/2012 06:09 PM, Oleksij Rempel wrote: >> On 07.07.2012 11:20, Eric Ding wrote: >>> On 07/06/2012 11:47 PM, Alan Stern wrote: On Fri, 6 Jul 2012, Alan Cox wrote: >> Yes, but we still need to know the reason why. Neither Rafael nor I >>>

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Eric Ding
On 07/07/2012 09:11 PM, Oleksij Rempel wrote: > On 07.07.2012 13:38, Eric Ding wrote: >> On 07/07/2012 06:09 PM, Oleksij Rempel wrote: >>> On 07.07.2012 11:20, Eric Ding wrote: On 07/06/2012 11:47 PM, Alan Stern wrote: > On Fri, 6 Jul 2012, Alan Cox wrote: > >>> Yes, but we still n

Re: [PATCH v8 06/14] USB: Chipidea: rename struct ci13xxx variables from udc to ci

2012-07-07 Thread Richard Zhao
On Fri, Jul 06, 2012 at 11:07:21AM -0700, Greg KH wrote: > On Fri, Jun 29, 2012 at 05:48:54PM +0800, Richard Zhao wrote: > > struct ci13xxx represent the controller, which may be device or host, > > so name its variables as ci. > > > > Signed-off-by: Richard Zhao > > Reviewed-by: Felipe Balbi >

[PATCH v9 REBASE 0/9] add imx usb driver based on Greg next tree

2012-07-07 Thread Richard Zhao
The work is based on ci13xxx rework done by Alexander Shishkin. To let Greg pick up, I also added patches Alex queued that I depends on. This patch series can also be found at: https://github.com/riczhao/kernel-imx/commits/topics/usb-driver Status: - this version only support host - usbotg

[PATCH v9 REBASE 2/9] USB: Chipidea: add unified ci13xxx_{add,remove}_device for platform drivers

2012-07-07 Thread Richard Zhao
Platform drivers do the similar things to add/remove ci13xxx device, so create a unified one. Signed-off-by: Richard Zhao Reviewed-by: Felipe Balbi Signed-off-by: Alexander Shishkin --- drivers/usb/chipidea/ci13xxx_msm.c | 34 ++-- drivers/usb/chipidea/ci13xxx_pci.c |

[PATCH v9 REBASE 3/9] USB: Chipidea: add ci13xxx device id management

2012-07-07 Thread Richard Zhao
We use ida_simple_get and ida_simple_remove to manage the ids. Signed-off-by: Richard Zhao Reviewed-by: Felipe Balbi Signed-off-by: Alexander Shishkin --- drivers/usb/chipidea/core.c | 21 - 1 file changed, 16 insertions(+), 5 deletions(-) diff --git a/drivers/usb/chipid

[PATCH v9 REBASE 4/9] usb: chipidea: select USB_EHCI_ROOT_HUB_TT in USB_CHIPIDEA_HOST of Kconfig

2012-07-07 Thread Richard Zhao
ci13xxx host needs Root Hub Transaction Translators. Reported-by: Shawn Guo Signed-off-by: Richard Zhao Acked-by: Felipe Balbi Signed-off-by: Alexander Shishkin Acked-by: Marek Vasut --- drivers/usb/chipidea/Kconfig |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/chipidea/

[PATCH v9 REBASE 5/9] usb: otg: add notify_connect/notify_disconnect callback

2012-07-07 Thread Richard Zhao
This let usb phy driver has a chance to change hw settings when connect status change. Signed-off-by: Richard Zhao Acked-by: Felipe Balbi Tested-by: Subodh Nijsure --- include/linux/usb/otg.h | 21 + 1 file changed, 21 insertions(+) diff --git a/include/linux/usb/otg.h b

[PATCH v9 REBASE 7/9] usb: chipidea: permit driver bindings pass phy pointer

2012-07-07 Thread Richard Zhao
Sometimes, the driver bindings may know what phy they use. For example, when using device tree, the usb controller may have a phandler pointing to usb phy. Signed-off-by: Richard Zhao Reviewed-by: Marek Vasut Acked-by: Felipe Balbi Tested-by: Subodh Nijsure --- drivers/usb/chipidea/ci.h|

[PATCH v9 REBASE 6/9] USB: notify phy when root hub port connect change

2012-07-07 Thread Richard Zhao
Phy may need to change settings when port connect change. Signed-off-by: Richard Zhao Tested-by: Subodh Nijsure --- drivers/usb/core/hub.c |8 1 file changed, 8 insertions(+) diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c index 4cc8dc9..2ba9d84 100644 --- a/drivers/u

[PATCH v9 REBASE 8/9] usb: otg: add basic mxs phy driver support

2012-07-07 Thread Richard Zhao
mxs phy is used in Freescale i.MX SoCs, for example imx23, imx28, imx6Q. This patch adds the basic host support. Signed-off-by: Richard Zhao Signed-off-by: Marek Vasut Cc: Peter Chen Acked-by: Felipe Balbi Tested-by: Subodh Nijsure --- Documentation/devicetree/bindings/usb/mxs-phy.txt | 13

[PATCH v9 REBASE 9/9] usb: chipidea: add imx platform driver

2012-07-07 Thread Richard Zhao
This patch supports only the host-mode functionality so far. Signed-off-by: Richard Zhao Signed-off-by: Marek Vasut Cc: Peter Chen Cc: Alexander Shishkin Cc: Felipe Balbi Cc: Greg Kroah-Hartman Tested-by: Subodh Nijsure --- .../devicetree/bindings/usb/ci13xxx-imx.txt| 18 ++ driv

Re: [PATCH V7 2/9] usb: make usb port a real device

2012-07-07 Thread Lan Tianyu
On 2012/7/7 2:12, Greg KH wrote: On Fri, Jul 06, 2012 at 02:13:53PM +0800, Lan Tianyu wrote: Change since v4: remove struct usb_port->udev member since it is not used. This patch is to make usb port a real device under usb hub interface. Move port_owner to struct usb_port. As this adds sysfs

Re: [PATCH V7 9/9] usb : Add usb port's power control attributes

2012-07-07 Thread Lan Tianyu
On 2012/7/7 2:14, Greg KH wrote: On Fri, Jul 06, 2012 at 02:14:00PM +0800, Lan Tianyu wrote: Change since v5: Add usb_autopm_get/put_interface() around the port power operations in the control attribute's callback to deal with situation that control attribute is changed when the usb hub is suspe

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Oleksij Rempel
On 07.07.2012 15:55, Eric Ding wrote: > On 07/07/2012 09:11 PM, Oleksij Rempel wrote: >> On 07.07.2012 13:38, Eric Ding wrote: >>> On 07/07/2012 06:09 PM, Oleksij Rempel wrote: On 07.07.2012 11:20, Eric Ding wrote: > On 07/06/2012 11:47 PM, Alan Stern wrote: >> On Fri, 6 Jul 2012, Alan

Re: infos about device ZTE MF821D 2G,3G,4G/LTE usb-modem/networkcard

2012-07-07 Thread Thomas Schäfer
The modem works at /dev/USB2. In my point of view "19d2 0326" should/could be added to the option driver. Thomas -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-i

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Alan Stern
On Sat, 7 Jul 2012, Oleksij Rempel wrote: > >> Ok, i guess i know your problem but i doubt it will be completely fixed > >> by changing powermanagement behavior. Two logitech cams i tested is > >> really easy to confuse/brake/freeze. Just turn off the stream before it > >> will send first frame.

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Rafael J. Wysocki
On Saturday, July 07, 2012, Alan Stern wrote: > On Sat, 7 Jul 2012, Oleksij Rempel wrote: > > > >> Ok, i guess i know your problem but i doubt it will be completely fixed > > >> by changing powermanagement behavior. Two logitech cams i tested is > > >> really easy to confuse/brake/freeze. Just tu

Re: Bug#677472: [3.1->3.2 regression] Immediate wake on suspend, associated with OHCI on MCP51

2012-07-07 Thread Octavio Alvarez
Hi, Alan! So, after about more than a week of bisecting, and thanks to Jonathan Nieder's more-than-precise instructions, the results are in. On Mon, 25 Jun 2012 11:41:31 -0700, Alan Stern wrote: On Mon, 25 Jun 2012, Octavio Alvarez wrote: On Mon, 25 Jun 2012 07:33:11 -0700, Alan Stern

Re: [PATCH V7 2/9] usb: make usb port a real device

2012-07-07 Thread Greg KH
On Sat, Jul 07, 2012 at 11:01:53PM +0800, Lan Tianyu wrote: > On 2012/7/7 2:12, Greg KH wrote: > >On Fri, Jul 06, 2012 at 02:13:53PM +0800, Lan Tianyu wrote: > >>Change since v4: remove struct usb_port->udev member since it is not used. > >> > >>This patch is to make usb port a real device under us

"usb: dwc3: resume phy during gadget initialization on recent cores" breaks phy suspend

2012-07-07 Thread Paul Zimmerman
Commit "usb: dwc3: resume phy during gadget initialization on recent cores" breaks phy suspend on cores newer than 1.94a. The core will still operate, but the power savings provided by phy suspend are lost. Pratyush, did you see an actual problem caused by the original code? If so, then you need t

Re: [PATCH 3/4] usbdevfs: Use scatter-gather lists for large bulk transfers

2012-07-07 Thread Greg Kroah-Hartman
On Sat, Jul 07, 2012 at 09:11:27AM +0200, Hans de Goede wrote: > Hi, > > On 07/06/2012 07:51 PM, Greg Kroah-Hartman wrote: > >On Wed, Jul 04, 2012 at 09:18:03AM +0200, Hans de Goede wrote: > >>+static void snoop_urb_data(struct urb *urb, unsigned len) > >>+{ > >>+ int i, size; > >>+ > >>+ if (

Re: [PATCH V7 2/9] usb: make usb port a real device

2012-07-07 Thread Sarah Sharp
On Fri, Jul 06, 2012 at 11:12:32AM -0700, Greg KH wrote: > On Fri, Jul 06, 2012 at 02:13:53PM +0800, Lan Tianyu wrote: > > Change since v4: remove struct usb_port->udev member since it is not used. > > > > This patch is to make usb port a real device under usb hub interface. > > Move port_owner to

Re: [PATCH 3/4] usbdevfs: Use scatter-gather lists for large bulk transfers

2012-07-07 Thread Alan Stern
On Sat, 7 Jul 2012, Greg Kroah-Hartman wrote: > > >Minor cleanup in the future, can't this be merged with snoop_urb() that > > >way you don't have to do the logic checking in the places you call this > > >function instead of snoop_urb()? That would make it a bit simpler for > > >the "normal" code

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Alan Stern
On Sat, 7 Jul 2012, Rafael J. Wysocki wrote: > > Well, the quirk does make sense. What doesn't make sense is why moving > > the runtime PM operation pointers from usb_bus_type to usb_device_type > > should cause any change in the autosuspend behavior. That's what we > > would like to know. >

Re: Bug#677472: [3.1->3.2 regression] Immediate wake on suspend, associated with OHCI on MCP51

2012-07-07 Thread Alan Stern
On Sat, 7 Jul 2012, Octavio Alvarez wrote: > > If you build a kernel with CONFIG_USB_DEBUG enabled, what > > shows up in /sys/kernel/debug/usb/ohci/*/registers? > > [Sat Jul 07 12:49:27 -0700 -- alvarezp@octavio:/sys/kernel/debug/usb] > $ grep . ohci/*/registers > bus pci, device :00:0b.0 > O

Re: [3.1->3.2 regression] Immediate wake on suspend, associated with OHCI on MCP51

2012-07-07 Thread Jonathan Nieder
Hi, Quick administrivia. Alan Stern wrote: > Yes, that commit enables wakeup for USB host controllers by default. > Before that, you had to enable wakeup by hand. The question is: Why > does the controller think it needs to wake up the system? Yotam Benshalom from https://bugzilla.kernel.org

[PATCH RESEND 2/3] usb: storage: add support for write cache quirk

2012-07-07 Thread Namjae Jeon
From: Namjae Jeon Add support for write cache quirk on usb hdd. scsi driver will be set to wce by detecting write cache quirk in quirk list when plugging usb hdd. Signed-off-by: Namjae Jeon Signed-off-by: Pankaj Kumar Signed-off-by: Amit Sahrawat Acked-by: Alan Stern --- Documentation/kerne

[PATCH RESEND 1/3] scsi: set to WCE if usb cache quirk is present.

2012-07-07 Thread Namjae Jeon
From: Namjae Jeon Make use of USB quirk method to identify such HDD while reading the cache status in sd_probe(). If cache quirk is present for the HDD, lets assume that cache is enabled and make WCE bit equal to 1. Signed-off-by: Namjae Jeon Signed-off-by: Pankaj Kumar Signed-off-by: Amit Sah

[PATCH RESEND 0/3] scsi: fix internal write cache issue on usb hdd.

2012-07-07 Thread Namjae Jeon
From: Namjae Jeon The numbers of USB HDDs(All USB HDD I checked) does not respond correctly to scsi mode sense command for retrieving the write cache page status. Even though write cache is enabled by default, due to scsi driver assume that cache is not enabled which in turn might lead to loss of

[PATCH RESEND 3/3] usb: storage: update usb devices for write cache quirk in quirk list.

2012-07-07 Thread Namjae Jeon
From: Namjae Jeon Update information of Seagate Portable HDD and WD My Passport HDD in quirk list. Signed-off-by: Namjae Jeon Signed-off-by: Pankaj Kumar Signed-off-by: Amit Sahrawat Acked-by: Alan Stern --- drivers/usb/storage/unusual_devs.h | 12 1 file changed, 12 insertio

Re: Bug#677472: [3.1->3.2 regression] Immediate wake on suspend, associated with OHCI on MCP51

2012-07-07 Thread Octavio Alvarez
On Sat, 07 Jul 2012 19:23:08 -0700, Alan Stern wrote: > Also, what does the "lspci -vv" output show for the controller if you > run it with superuser permissions? [Sat Jul 07 12:50:10 -0700 -- alvarezp@octavio:/sys/kernel/debug/usb] $ sudo lspci -vv -s :00:0b.1 0b.1 is the EHCI control

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Eric Ding
On 07/08/2012 09:52 AM, Alan Stern wrote: > On Sat, 7 Jul 2012, Rafael J. Wysocki wrote: > >>> Well, the quirk does make sense. What doesn't make sense is why moving >>> the runtime PM operation pointers from usb_bus_type to usb_device_type >>> should cause any change in the autosuspend behavior

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Eric Ding
On 07/08/2012 03:18 AM, Alan Stern wrote: > Apparently the behavior before commit e1620d5 was that the webcam > didn't get suspended, and after the commit it did. Unfortunately > the usbmon traces do not explain this difference; all they show is > when/whether a suspend took place. > > For examp

[PATCH v2 1/1] Input: xpad - Handle all variations of Mad Catz Beat Pad

2012-07-07 Thread Yuri Khan
* Add a quirk to usbhid to ignore this device --- drivers/hid/hid-ids.h |3 +++ drivers/hid/usbhid/hid-quirks.c |2 ++ drivers/input/joystick/xpad.c |1 + 3 files changed, 6 insertions(+) diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index d1cdd2d..43c3d75 100

Re: [PATCH] From 2.6.39-rc1 onward, the Logitech Quickcam Fusion webcam (046d:08c1) stops

2012-07-07 Thread Oleksij Rempel (fishor)
On 08.07.2012 07:37, Eric Ding wrote: On 07/08/2012 09:52 AM, Alan Stern wrote: On Sat, 7 Jul 2012, Rafael J. Wysocki wrote: Well, the quirk does make sense. What doesn't make sense is why moving the runtime PM operation pointers from usb_bus_type to usb_device_type should cause any change in