[PATCH net v2] net: qmi_wwan: call subdriver with control intf only

2012-09-12 Thread Bjørn Mork
This fixes a hang on suspend due to calling wdm_suspend on the unregistered data interface. The hang should have been a NULL pointer reference had it not been for a logic error in the cdc_wdm code. commit 230718bd net: qmi_wwan: bind to both control and data interface changed qmi_wwan to use cd

Re: Samsung SCX-3205 scanner does not work with Toshiba Satellite L855

2012-09-12 Thread Michal Nowak
On 09/12/2012 10:51 PM, Alan Stern wrote: > On Wed, 12 Sep 2012, Michal Nowak wrote: > >> From https://bugzilla.kernel.org/show_bug.cgi?id=47421: >> >> I have problems with Samsung SCX-3205 scanner on Toshiba Satellite L855. >> When is scanner connected to the notebook (via USB) I can see it via >

Re: [PATCH 00/12] usb: chipidea updates for v3.7

2012-09-12 Thread Alexander Shishkin
Greg Kroah-Hartman writes: > On Wed, Sep 12, 2012 at 02:57:59PM +0300, Alexander Shishkin wrote: >> Hi, >> >> This bunch of patches contains fixes in the core driver and updates >> for chipidea integrated in imx chips. > > As your first few patches were marked for the stable tree, I also pulled

Re: [PATCH] usb: host: tegra: code clean up

2012-09-12 Thread Stephen Warren
On 09/12/2012 09:42 PM, Venu Byravarasu wrote: > Stephen Warren wrote at Wednesday, September 12, 2012 11:41 PM: >> On 09/12/2012 01:02 AM, Venu Byravarasu wrote: >>> As part of code clean up, used devm counterparts for the APIs >>> possible. >> >> Almost all of this patch has already been applied

Re: [PATCH] USB: phy: re-organize tegra phy driver

2012-09-12 Thread Stephen Warren
On 09/12/2012 09:49 PM, Venu Byravarasu wrote: > Stephen Warren wrote at Thursday, September 13, 2012 12:06 AM: >> On 09/12/2012 04:58 AM, Venu Byravarasu wrote: >>> Nvidia produces several Tegra SOCs viz Tegra2, Tegra3 etc. >>> In order to support USB phy drivers on these SOCs, existing >>> phy dr

Re: [PATCH] USB: phy: re-organize tegra phy driver

2012-09-12 Thread Stephen Warren
On 09/12/2012 10:16 PM, Venu Byravarasu wrote: > Forgot to address some of the comments made by stephen, in my previous update. > Hence addressing them now. > Thanks a lot Stephen, for detailed review. OK, so since this patch is basically just splitting the file into multiple parts, you can ignore

RE: [PATCH] USB: phy: re-organize tegra phy driver

2012-09-12 Thread Venu Byravarasu
Forgot to address some of the comments made by stephen, in my previous update. Hence addressing them now. Thanks a lot Stephen, for detailed review. > -Original Message- > From: Stephen Warren [mailto:swar...@wwwdotorg.org] > Sent: Thursday, September 13, 2012 12:06 AM > To: Venu Byravaras

RE: [PATCH] USB: phy: re-organize tegra phy driver

2012-09-12 Thread Venu Byravarasu
> -Original Message- > From: Stephen Warren [mailto:swar...@wwwdotorg.org] > Sent: Thursday, September 13, 2012 12:06 AM > To: Venu Byravarasu > Cc: ba...@ti.com; linux-ker...@vger.kernel.org; linux-usb@vger.kernel.org; > linux-te...@vger.kernel.org > Subject: Re: [PATCH] USB: phy: re-organ

RE: [PATCH] usb: host: tegra: code clean up

2012-09-12 Thread Venu Byravarasu
> -Original Message- > From: Stephen Warren [mailto:swar...@wwwdotorg.org] > Sent: Wednesday, September 12, 2012 11:41 PM > To: Venu Byravarasu > Cc: ba...@ti.com; linux-ker...@vger.kernel.org; linux-usb@vger.kernel.org > Subject: Re: [PATCH] usb: host: tegra: code clean up > > On 09/12/20

Re: Samsung SCX-3205 scanner does not work with Toshiba Satellite L855

2012-09-12 Thread Sarah Sharp
On Wed, Sep 12, 2012 at 04:51:40PM -0400, Alan Stern wrote: > On Wed, 12 Sep 2012, Michal Nowak wrote: > > > From https://bugzilla.kernel.org/show_bug.cgi?id=47421: > > > > I have problems with Samsung SCX-3205 scanner on Toshiba Satellite L855. > > When is scanner connected to the notebook (via

Re: XHCI: URB not cancelled during disconnect of a MSC device

2012-09-12 Thread Sarah Sharp
On Wed, Sep 12, 2012 at 03:36:39PM +0530, Ajay Gupta wrote: > Hi, > > I am using v3.5 kernel and running a test where I disconnect a SS MSC > device while a big file is being written to it. I am also watching the > lsusb output in parallel. Expectation is that SS MSC device should > immediately di

Re: Samsung SCX-3205 scanner does not work with Toshiba Satellite L855

2012-09-12 Thread Alan Stern
On Wed, 12 Sep 2012, Michal Nowak wrote: > From https://bugzilla.kernel.org/show_bug.cgi?id=47421: > > I have problems with Samsung SCX-3205 scanner on Toshiba Satellite L855. > When is scanner connected to the notebook (via USB) I can see it via > `scanimage -L` on first attempt but on any other

Re: EHCI: iaa_watchdog_start() warning followed by NULL ptr dereference in start_unlink_async()

2012-09-12 Thread Alan Stern
On Wed, 12 Sep 2012, Hemant Kumar wrote: > Hi Alen, > > if we add check for ehci->reclaim->qh_state == QH_STATE_UNLINK in > ehci_irq() and ehci_iaa_watchdog() i still see a race : > what if end_unlink_async for QH1 called start_unlink_async(QH2) and IAAD > interrupt fired after QH2->qh_state is s

[PATCH] usb: gadget: lpc32xx_udc: Disable setup request error message

2012-09-12 Thread Alexandre Pereira da Silva
This message is an debugging message. It's useful for finding protocol details but it's not necessarily an error. Signed-off-by: Alexandre Pereira da Silva --- drivers/usb/gadget/lpc32xx_udc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/gadget/lpc32xx_udc.c

[PATCH] usb: gadget: lpc32xx_udc: Fix compatibility with STOTG04

2012-09-12 Thread Alexandre Pereira da Silva
The STOTG04 is an replacement for ISP1301. Most of the registers on STOTG04 are the same as on ISP1301, but the register ISP1301_I2C_OTG_CONTROL_2 (address 0x10) doesn't exist on the ST part. This is a work around for this by using the interrupt source register that should behave the same on both

Re: EHCI: iaa_watchdog_start() warning followed by NULL ptr dereference in start_unlink_async()

2012-09-12 Thread Hemant Kumar
> On Wed, 12 Sep 2012, Pavan Kondeti wrote: > >> Thanks for your response. I and Hemant work together. We are thinking a >> scenario where this NULL dereference could happen. Please let us know if >> our understanding is correct or not. we are assuming that async doorbell >> interrupt can not be st

Re: [PATCH] USB: phy: re-organize tegra phy driver

2012-09-12 Thread Stephen Warren
On 09/12/2012 04:58 AM, Venu Byravarasu wrote: > Nvidia produces several Tegra SOCs viz Tegra2, Tegra3 etc. > In order to support USB phy drivers on these SOCs, existing > phy driver is split into SOC agnostic common USB phy driver and > tegra2 specific USB phy driver. > This will facilitate easy a

Re: xhci: Fix bug after deq ptr set to link TRB.

2012-09-12 Thread Greg KH
On Wed, Sep 12, 2012 at 11:19:42AM -0700, Oleg Rombakh wrote: > Hi, > > I am working on the project that involves USB3, 3.0 kernel, and WebCams. > > One of our h/w platforms utilizes NEC uPD720200 USB3 chipset. > > The image we run is based on SUSE 11SP2 (kernel 3.0.38) and on this > kernel we e

Re: [PATCH 00/12] usb: chipidea updates for v3.7

2012-09-12 Thread Greg Kroah-Hartman
On Wed, Sep 12, 2012 at 02:57:59PM +0300, Alexander Shishkin wrote: > Hi, > > This bunch of patches contains fixes in the core driver and updates > for chipidea integrated in imx chips. As your first few patches were marked for the stable tree, I also pulled them into the usb-linus branch. Next

xhci: Fix bug after deq ptr set to link TRB.

2012-09-12 Thread Oleg Rombakh
Hi, I am working on the project that involves USB3, 3.0 kernel, and WebCams. One of our h/w platforms utilizes NEC uPD720200 USB3 chipset. The image we run is based on SUSE 11SP2 (kernel 3.0.38) and on this kernel we experience problem with web cameras attached to USB3. The same cameras work on

Re: [PATCH] usb: host: tegra: code clean up

2012-09-12 Thread Stephen Warren
On 09/12/2012 01:02 AM, Venu Byravarasu wrote: > As part of code clean up, used devm counterparts for the APIs > possible. Almost all of this patch has already been applied as: bc2ff98 drivers/usb/host/ehci-tegra.c: use devm_ functions (btw, that patch has a much better patch subject than this o

Samsung SCX-3205 scanner does not work with Toshiba Satellite L855

2012-09-12 Thread Michal Nowak
>From https://bugzilla.kernel.org/show_bug.cgi?id=47421: I have problems with Samsung SCX-3205 scanner on Toshiba Satellite L855. When is scanner connected to the notebook (via USB) I can see it via `scanimage -L` on first attempt but on any other attempt I can't see it any more, unless I turn the

[PATCH 10/15] drivers/usb/host/xhci-ring.c: removes unnecessary semicolon

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin --- drivers/usb/host/xhci-ring.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/host/xhci-ring.c b/drivers/usb/host

[PATCH 13/15] drivers/usb/misc/rio500.c: removes unnecessary semicolon

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin --- drivers/usb/misc/rio500.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/misc/rio500.c b/drivers/usb/misc/rio50

[PATCH 15/15] drivers/usb/host/sl811-hcd.c: removes unnecessary semicolon

2012-09-12 Thread Peter Senna Tschudin
From: Peter Senna Tschudin removes unnecessary semicolon Found by Coccinelle: http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin --- drivers/usb/host/sl811-hcd.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/host/sl811-hcd.c b/drivers/usb/host

Re: [PATCH v5 1/5] usb: phy: samsung: Introducing usb phy driver for hsotg

2012-09-12 Thread Tomasz Figa
Hi Praveen, Marc, On Wednesday 12 of September 2012 13:08:26 Marc Kleine-Budde wrote: > > +static const struct of_device_id samsung_usbphy_dt_match[]; > > + > > +static inline int samsung_usbphy_get_driver_data(struct > > platform_device *pdev) +{ > > +#ifdef CONFIG_OF > > if (IS_ENABLED(CONFIG_O

Re: [balbi-usb:merge-result-for-greg 66/99] drivers/usb/core/hub.c:2654 usb_get_hub_port_power_state() error: doing dma on the stack ((null))

2012-09-12 Thread Lan Tianyu
On 2012/9/12 21:11, Alan Stern wrote: On Wed, 12 Sep 2012, Fengguang Wu wrote: Hi Felipe, FYI, there are new smatch warnings show up in tree: git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git merge-result-for-greg head: 23953bde3e4d6aa8780dc054f6ad9882ac63f4f4 commit: e918fa161

[no subject]

2012-09-12 Thread Christoph Jung
Hello, I am from the company "Code Mercenaries GmbH" from Germany. We have some USB HID devices wich work with Linux. Since kernel version 2.6 our default products will be included in the kernel. Devicedetails VendorID: 0x07C0 ProductIDs: 0x1500, 0x1501, 0x1511, 0x1512, 0x1503 Devicename: iowar

Re: [PATCH] usb: host: tegra: code clean up

2012-09-12 Thread Felipe Balbi
Hi, On Wed, Sep 12, 2012 at 12:32:42PM +0530, Venu Byravarasu wrote: > As part of code clean up, used devm counterparts for the APIs > possible. > > Signed-off-by: Venu Byravarasu EHCI is Alan's domain. > --- > drivers/usb/host/ehci-tegra.c | 46 ++-- > 1

Re: EHCI: iaa_watchdog_start() warning followed by NULL ptr dereference in start_unlink_async()

2012-09-12 Thread Pavan Kondeti
On 9/12/2012 6:29 PM, Alan Stern wrote: > On Wed, 12 Sep 2012, Pavan Kondeti wrote: > >> Thanks for your response. I and Hemant work together. We are thinking a >> scenario where this NULL dereference could happen. Please let us know if >> our understanding is correct or not. we are assuming that

Re: [PATCH 2/3] usb: otg: add device tree support to otg library

2012-09-12 Thread Marc Kleine-Budde
On 09/06/2012 05:27 PM, Marc Kleine-Budde wrote: >> We can use something like "-phy" as the phandle name. And the >> users can get the phy by using >> devm_usb_get_phy_by_phandle(dev, ""). >> (So the frwrk appends *-phy* to the name and searches). Or we don't >> have any

Re: [PATCH] usb: move children deallocation after quiescing the hub

2012-09-12 Thread Alan Stern
On Wed, 12 Sep 2012, Alexander Shishkin wrote: > Commit ff823c79a5c33194c2e5594f7c4686ea3547910c ("usb: move children > to struct usb_port") forgot to consider the hub_disconnect sequence, > which releases ports before quiescing the hub, which will lead to a > use-after-free, since hub_quiesce() w

Re: [balbi-usb:merge-result-for-greg 66/99] drivers/usb/core/hub.c:2654 usb_get_hub_port_power_state() error: doing dma on the stack ((null))

2012-09-12 Thread Alan Stern
On Wed, 12 Sep 2012, Fengguang Wu wrote: > Hi Felipe, > > FYI, there are new smatch warnings show up in > > tree: git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git > merge-result-for-greg > head: 23953bde3e4d6aa8780dc054f6ad9882ac63f4f4 > commit: e918fa161f510136fce45a524e934fe20e

Re: EHCI: iaa_watchdog_start() warning followed by NULL ptr dereference in start_unlink_async()

2012-09-12 Thread Alan Stern
On Wed, 12 Sep 2012, Pavan Kondeti wrote: > Thanks for your response. I and Hemant work together. We are thinking a > scenario where this NULL dereference could happen. Please let us know if > our understanding is correct or not. we are assuming that async doorbell > interrupt can not be stopped b

Re: [PATCH v4] usb: phy: add R-Car USB phy driver

2012-09-12 Thread Marc Kleine-Budde
On 09/11/2012 04:06 AM, Kuninori Morimoto wrote: > This patch adds Renesas R-Car USB phy driver. > It supports R8A7779 chip at this point. > > R-Car has some USB controllers, but has only one phy-initializer. > So, this driver is counting users. > > Signed-off-by: Kuninori Morimoto > --- > v3 ->

Re: [PATCH 2/2] usb: chipidea: usbmisc: prepare driver to handle more than one soc

2012-09-12 Thread Marc Kleine-Budde
On 09/07/2012 10:20 AM, Richard Zhao wrote: > On Thu, Sep 06, 2012 at 05:18:29PM +0200, Marc Kleine-Budde wrote: >> On 09/06/2012 05:15 PM, Richard Zhao wrote: >>> Hi Marc, >>> >>> usbmisc_imx6q.c is only for imx6x. And for a certain running kernel, >>> there will be always one driver instance. >>

Re: [balbi-usb:merge-result-for-greg 66/99] drivers/usb/core/hub.c:2654 usb_get_hub_port_power_state() error: doing dma on the stack ((null))

2012-09-12 Thread Fengguang Wu
On Wed, Sep 12, 2012 at 03:02:13PM +0300, Felipe Balbi wrote: > Hi, > > On Wed, Sep 12, 2012 at 11:50:57AM +0300, Dan Carpenter wrote: > > On Wed, Sep 12, 2012 at 04:26:22PM +0800, Fengguang Wu wrote: > > > Hi Felipe, > > > > > > FYI, there are new smatch warnings show up in > > > > > > tree:

Re: [balbi-usb:merge-result-for-greg 66/99] drivers/usb/core/hub.c:2654 usb_get_hub_port_power_state() error: doing dma on the stack ((null))

2012-09-12 Thread Felipe Balbi
Hi, On Wed, Sep 12, 2012 at 11:50:57AM +0300, Dan Carpenter wrote: > On Wed, Sep 12, 2012 at 04:26:22PM +0800, Fengguang Wu wrote: > > Hi Felipe, > > > > FYI, there are new smatch warnings show up in > > > > tree: git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git > > merge-result-fo

[PATCH 12/12] USB: chipidea: re-order irq handling to avoid unhandled irqs

2012-09-12 Thread Alexander Shishkin
From: Richard Zhao - let role driver handle irq before ID change check; this gives the role driver a chance to handle disconnect; - disable irq during switch role; no role driver to handle irq in the period. Tested-by: Michael Grzeschik Tested-by: Marc Kleine-Budde Signed-off-by: Richard Z

[PATCH 11/12] USB: chipidea: add -DDEBUG if debugging is enabled

2012-09-12 Thread Alexander Shishkin
From: Richard Zhao This patch turns on debugging output if CONFIG_USB_CHIPIDEA_DEBUG is enabled. Signed-off-by: Richard Zhao Tested-by: Michael Grzeschik Tested-by: Marc Kleine-Budde Signed-off-by: Alexander Shishkin --- drivers/usb/chipidea/Makefile |2 ++ 1 file changed, 2 insertions(

[PATCH 10/12] USB: chipidea: add vbus detect for udc

2012-09-12 Thread Alexander Shishkin
From: Richard Zhao Using vbus valid interrupt to detect vbus. Tested-by: Michael Grzeschik Tested-by: Marc Kleine-Budde Signed-off-by: Richard Zhao Signed-off-by: Alexander Shishkin --- drivers/usb/chipidea/ci.h |1 + drivers/usb/chipidea/udc.c | 39 ++

[PATCH 09/12] USB: chipidea: acknowledge ID change interrupt in irq handler

2012-09-12 Thread Alexander Shishkin
From: Richard Zhao In order to avoid re-queueing of the role changing work, we need to clear the ID change interrupt bit right in the irq handler. Tested-by: Michael Grzeschik Tested-by: Marc Kleine-Budde Signed-off-by: Richard Zhao Signed-off-by: Alexander Shishkin --- drivers/usb/chipidea

[PATCH 06/12] USB: chipidea: add imx usbmisc support

2012-09-12 Thread Alexander Shishkin
From: Richard Zhao i.MX usb controllers share non-core registers, which may include SoC specific controls. We turn it into a usbmisc device and usbmisc driver set operations needed by ci13xxx_imx driver. For example, Sabrelite board has bad over-current design, we can usbmisc to disable over-cur

[PATCH 08/12] USB: chipidea: delay 2ms before read ID status at probe time

2012-09-12 Thread Alexander Shishkin
From: Richard Zhao The ID pin needs 1ms debounce time, even at probe time. We delay 2ms to be on the safe side. Tested-by: Michael Grzeschik Tested-by: Marc Kleine-Budde Signed-off-by: Richard Zhao Signed-off-by: Alexander Shishkin --- drivers/usb/chipidea/core.c |2 ++ 1 file changed,

[PATCH 04/12] usb: chipidea: cleanup dma_pool if udc_start() fails

2012-09-12 Thread Alexander Shishkin
From: Marc Kleine-Budde If udc_start() fails the qh_pool dma-pool cannot be closed because it's still in use. This patch factors out the dma_pool_free() loop into destroy_eps() and calls it in the error path of udc_start(), too. Cc: Reviewed-by: Richard Zhao Signed-off-by: Marc Kleine-Budde S

[PATCH 07/12] USB: chipidea: imx: add pinctrl support

2012-09-12 Thread Alexander Shishkin
From: Richard Zhao Some controllers may not need to setup pinctrl, so we don't fail the probe if pinctrl get/select failed. Tested-by: Michael Grzeschik Tested-by: Marc Kleine-Budde Signed-off-by: Richard Zhao Signed-off-by: Alexander Shishkin --- drivers/usb/chipidea/ci13xxx_imx.c |7 +

[PATCH 05/12] usb: chipidea: udc: don't stall endpoint if request list is empty in isr_tr_complete_low

2012-09-12 Thread Alexander Shishkin
From: Michael Grzeschik When attaching an imx28 or imx53 in USB gadget mode to a Windows host and starting a rndis connection we see this message every 4-10 seconds: g_ether gadget: high speed config #2: RNDIS Analysis shows that each time this message is printed, the rndis connection is re

[PATCH 03/12] usb: chipidea: udc: fix error path in udc_start()

2012-09-12 Thread Alexander Shishkin
From: Marc Kleine-Budde This patch fixes the error path of udc_start(). Now NULL is used to unset the peripheral with otg_set_peripheral(). Cc: Reviewed-by: Richard Zhao Signed-off-by: Marc Kleine-Budde Signed-off-by: Alexander Shishkin --- drivers/usb/chipidea/udc.c |2 +- 1 file chang

Re: XHCI: URB not cancelled during disconnect of a MSC device

2012-09-12 Thread Ajay Gupta
Hi, >> I am using v3.5 kernel and running a test where I disconnect >> a SS MSC device while a big file is being written to it. I am >> also watching the lsusb output in parallel. Expectation is >> that SS MSC device should immediately disappear from lsusb >> output but sometime I see that SS MSC i

[PATCH 02/12] usb: chipidea: udc: add pullup fuction, needed by the uvc gadget

2012-09-12 Thread Alexander Shishkin
From: Michael Grzeschik Add function to physicaly enable or disable of pullup connection on the USB-D+ line. The uvc gaget will fail, if this function is not implemented. Cc: Signed-off-by: Michael Grzeschik Acked-by: Felipe Balbi Signed-off-by: Marc Kleine-Budde Signed-off-by: Alexander Shi

[PATCH 01/12] usb: chipidea: udc: fix setup of endpoint maxpacket size

2012-09-12 Thread Alexander Shishkin
From: Michael Grzeschik This patch changes the setup of the endpoint maxpacket size. All non control endpoints are initialized with an undefined ((unsigned short)~0) maxpacket size. The maxpacket size of Endpoint 0 will be kept at CTRL_PAYLOAD_MAX. Some gadget drivers check for the maxpacket siz

[PATCH 00/12] usb: chipidea updates for v3.7

2012-09-12 Thread Alexander Shishkin
Hi, This bunch of patches contains fixes in the core driver and updates for chipidea integrated in imx chips. Marc Kleine-Budde (2): usb: chipidea: udc: fix error path in udc_start() usb: chipidea: cleanup dma_pool if udc_start() fails Michael Grzeschik (3): usb: chipidea: udc: fix setup o

RE: XHCI: URB not cancelled during disconnect of a MSC device

2012-09-12 Thread B, Ravi
> > I am using v3.5 kernel and running a test where I disconnect > a SS MSC device while a big file is being written to it. I am > also watching the lsusb output in parallel. Expectation is > that SS MSC device should immediately disappear from lsusb > output but sometime I see that SS MSC is

[PATCH] usb: move children deallocation after quiescing the hub

2012-09-12 Thread Alexander Shishkin
Commit ff823c79a5c33194c2e5594f7c4686ea3547910c ("usb: move children to struct usb_port") forgot to consider the hub_disconnect sequence, which releases ports before quiescing the hub, which will lead to a use-after-free, since hub_quiesce() will try to disconnect ports' children, which are already

RE: [PATCH] usb: gadget: fsl_udc_core: do not immediatly prime STATUS for IN xfer

2012-09-12 Thread Li Yang-R58472
> -Original Message- > From: Felipe Balbi [mailto:ba...@ti.com] > Sent: Thursday, September 06, 2012 10:28 PM > To: Enrico Scholz > Cc: ba...@ti.com; Chen Peter-B29397; linux-usb@vger.kernel.org; linuxppc- > d...@lists.ozlabs.org; Li Yang-R58472; gre...@linuxfoundation.org > Subject: Re:

[PATCH] USB: phy: re-organize tegra phy driver

2012-09-12 Thread Venu Byravarasu
Nvidia produces several Tegra SOCs viz Tegra2, Tegra3 etc. In order to support USB phy drivers on these SOCs, existing phy driver is split into SOC agnostic common USB phy driver and tegra2 specific USB phy driver. This will facilitate easy addition & deletion of phy drivers for Tegra SOCs. Signed

Re: [PATCH v5 1/5] usb: phy: samsung: Introducing usb phy driver for hsotg

2012-09-12 Thread Marc Kleine-Budde
On 09/12/2012 12:45 PM, Praveen Paneri wrote: > This driver uses usb_phy interface to interact with s3c-hsotg. Supports > phy_init and phy_shutdown functions to enable/disable phy. Tested with > smdk6410 and smdkv310. More SoCs can be brought under later. > > Signed-off-by: Praveen Paneri > Acked

Re: usb3 writes fail with transcend rdf8 + ASMedia ASM1042 woes

2012-09-12 Thread Sami Farin
On Sat, Aug 25, 2012 at 12:31:46 +0300, Sami Farin wrote: >... Did you get the email (360 kB), it did not show up at marc.info but I did not get delivery status notifications..? I uploaded the xhci debug log here: https://docs.google.com/open?id=0B_EMXtsx7o56eUw5dC10RDhTcmc these with git/torval

Re: [RFC] How to handle delays in isochronous transfers?

2012-09-12 Thread Clemens Ladisch
Jassi Brar wrote: > On Tue, Sep 11, 2012 at 11:46 PM, Clemens Ladisch wrote: >> When URB submission fails, the stream should be stopped. > > Since ISOCH transfers are not expected to be lossless (i.e, not > retried), perhaps we should simply lump all kinds of failures together > and ignore them an

Re: [RFC] How to handle delays in isochronous transfers?

2012-09-12 Thread Jassi Brar
On Tue, Sep 11, 2012 at 11:46 PM, Clemens Ladisch wrote: > Alan Stern wrote: >> I have tried some initial testing of my updates, using data-OUT >> transfers with URB_ISO_ASAP turned off for the data URBs in >> sound/usb/endpoint.c. When interrupts are delayed so long that >> synchronization is lo

Re: [PATCH v2 05/11] USB: chipidea: don't let probe fail if otg controller start one role failed

2012-09-12 Thread Alexander Shishkin
Richard Zhao writes: > On Tue, Sep 11, 2012 at 10:23:55AM +0300, Alexander Shishkin wrote: >> Richard Zhao writes: >> >> > On Wed, Aug 29, 2012 at 12:48:15PM +0300, Alexander Shishkin wrote: >> >> Richard Zhao writes: >> >> >> >> > On Wed, Aug 29, 2012 at 11:10:33AM +0300, Alexander Shishkin

Re: [PATCH v2 06/11] USB: mxs-phy: add basic otg support

2012-09-12 Thread Heikki Krogerus
Hi, On Tue, Aug 28, 2012 at 03:03:12PM +0800, Richard Zhao wrote: > +static int mxs_phy_set_host(struct usb_otg *otg, struct usb_bus *host) > +{ Shouldn't you at least save the host pointer? otg->host = host; > + return 0; > +} > + > +static int mxs_phy_set_peripheral(struct usb_otg *otg, >

Re: [PATCH v4] usb: phy: add R-Car USB phy driver

2012-09-12 Thread Sergei Shtylyov
Hello. On 12-09-2012 10:55, Simon Horman wrote: below is an incremental patch to add some workarounds that were made available to me. I wonder if you have some documentation available to replace the hex values with #defines? From: Kouei Abe arm: mach-rcar: Add USB-PHY workaround Sign

[PATCH v5 5/5] ARM: Exynos4210: Enabling samsung-usbphy driver

2012-09-12 Thread Praveen Paneri
Adding usbphy node for Exynos4210 along with the platform data. Signed-off-by: Praveen Paneri --- arch/arm/boot/dts/exynos4210.dtsi |5 + arch/arm/mach-exynos/include/mach/map.h |1 + arch/arm/mach-exynos/mach-exynos4-dt.c |8 arch/arm/mach-exynos/setup-usb-phy.c

[PATCH v5 4/5] ARM: S3C64XX: Enabling samsung-usbphy driver

2012-09-12 Thread Praveen Paneri
Adding platform device for samsung-usbphy driver. Enabling it for s3c64xx based machines using s3c-hsotg. Signed-off-by: Praveen Paneri --- arch/arm/mach-s3c64xx/include/mach/map.h |2 + arch/arm/mach-s3c64xx/mach-crag6410.c|7 ++ arch/arm/mach-s3c64xx/mach-smartq.c

[PATCH v5 3/5] ARM: S3C64XX: Removing old phy setup code

2012-09-12 Thread Praveen Paneri
This patch removes old phy code from platform side. 'setup-usb-phy.c' will be used for providing transceiver platform data in next patch. Not all of the platform data code is removed as there are others making use of platform_data defined for hsotg. That can be removed once all the SoCs start using

[PATCH v5 2/5] usb: s3c-hsotg: Adding phy driver support

2012-09-12 Thread Praveen Paneri
Adding the transceiver to hsotg driver. Keeping the platform data for continuing the smooth operation for boards which still uses it Signed-off-by: Praveen Paneri --- drivers/usb/gadget/s3c-hsotg.c | 37 +++-- 1 files changed, 27 insertions(+), 10 deletions(-)

[PATCH v5 1/5] usb: phy: samsung: Introducing usb phy driver for hsotg

2012-09-12 Thread Praveen Paneri
This driver uses usb_phy interface to interact with s3c-hsotg. Supports phy_init and phy_shutdown functions to enable/disable phy. Tested with smdk6410 and smdkv310. More SoCs can be brought under later. Signed-off-by: Praveen Paneri Acked-by: Heiko Stuebner --- .../devicetree/bindings/usb/sams

[PATCH v5 0/5] usb: phy: samsung: Introducing usb phy driver for samsung SoCs

2012-09-12 Thread Praveen Paneri
Changes from v4: Moved header file contents to driver's source file Removed unnecessary print message from driver's probe function Dropped the Free Software Foundation address from the header Changed the platform data code to use __initdata Changes from v3: Replaced susbsys_initcall()/module_exit(

XHCI: URB not cancelled during disconnect of a MSC device

2012-09-12 Thread Ajay Gupta
Hi, I am using v3.5 kernel and running a test where I disconnect a SS MSC device while a big file is being written to it. I am also watching the lsusb output in parallel. Expectation is that SS MSC device should immediately disappear from lsusb output but sometime I see that SS MSC is listed for 3

Re: [balbi-usb:merge-result-for-greg 66/99] drivers/usb/core/hub.c:2654 usb_get_hub_port_power_state() error: doing dma on the stack ((null))

2012-09-12 Thread Dan Carpenter
On Wed, Sep 12, 2012 at 04:26:22PM +0800, Fengguang Wu wrote: > Hi Felipe, > > FYI, there are new smatch warnings show up in > > tree: git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git > merge-result-for-greg > head: 23953bde3e4d6aa8780dc054f6ad9882ac63f4f4 > commit: e918fa161f5101

[balbi-usb:merge-result-for-greg 66/99] drivers/usb/core/hub.c:2654 usb_get_hub_port_power_state() error: doing dma on the stack ((null))

2012-09-12 Thread Fengguang Wu
Hi Felipe, FYI, there are new smatch warnings show up in tree: git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git merge-result-for-greg head: 23953bde3e4d6aa8780dc054f6ad9882ac63f4f4 commit: e918fa161f510136fce45a524e934fe20e62c8b1 [66/99] Merge tag 'gadget-for-v3.7' into merge-res

Re: [PATCH v2] usb: gadget: at91_udc: dt: fix platform_data check

2012-09-12 Thread Nicolas Ferre
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 09/12/2012 08:14 AM, Felipe Balbi : > Hi, > > On Tue, Sep 11, 2012 at 02:20:19PM +0200, Nicolas Ferre wrote: >> On 09/11/2012 02:07 PM, Fabio Porcedda : >>> Don't fail the initialization check for the platform_data if >>> there is avaiable an assoc

[PATCH] usb: host: tegra: code clean up

2012-09-12 Thread Venu Byravarasu
As part of code clean up, used devm counterparts for the APIs possible. Signed-off-by: Venu Byravarasu --- drivers/usb/host/ehci-tegra.c | 46 ++-- 1 files changed, 16 insertions(+), 30 deletions(-) diff --git a/drivers/usb/host/ehci-tegra.c b/drivers/usb/h

Re: [PATCH v4] usb: phy: add R-Car USB phy driver

2012-09-12 Thread Kuninori Morimoto
Dear Simon > On Mon, Sep 10, 2012 at 07:06:48PM -0700, Kuninori Morimoto wrote: > > This patch adds Renesas R-Car USB phy driver. > > It supports R8A7779 chip at this point. > > > > R-Car has some USB controllers, but has only one phy-initializer. > > So, this driver is counting users. > > > >