Re: v3.11-rc1 USB regressions

2013-07-16 Thread Daniel Mack
Hi Aaro, On 16.07.2013 00:56, Aaro Koskinen wrote: Hi, I think USB tree introduced regressions in v3.11-rc1, at least for some OMAP devices using legacy boot. Thanks for checking the tree so early. I have only bisected these; I have no idea what the real fixes are but the following

Re: v3.11-rc1 USB regressions

2013-07-16 Thread Kishon Vijay Abraham I
Hi, On Tuesday 16 July 2013 12:03 PM, Daniel Mack wrote: Hi Aaro, On 16.07.2013 00:56, Aaro Koskinen wrote: Hi, I think USB tree introduced regressions in v3.11-rc1, at least for some OMAP devices using legacy boot. Thanks for checking the tree so early. I have only bisected these;

Re: v3.11-rc1 USB regressions

2013-07-16 Thread Felipe Balbi
Hi, On Tue, Jul 16, 2013 at 12:30:00PM +0530, Kishon Vijay Abraham I wrote: On Tuesday 16 July 2013 12:03 PM, Daniel Mack wrote: Hi Aaro, On 16.07.2013 00:56, Aaro Koskinen wrote: Hi, I think USB tree introduced regressions in v3.11-rc1, at least for some OMAP devices using legacy

Re: v3.11-rc1 USB regressions

2013-07-16 Thread Felipe Balbi
On Tue, Jul 16, 2013 at 11:08:36AM +0300, Felipe Balbi wrote: Hi, On Tue, Jul 16, 2013 at 12:30:00PM +0530, Kishon Vijay Abraham I wrote: On Tuesday 16 July 2013 12:03 PM, Daniel Mack wrote: Hi Aaro, On 16.07.2013 00:56, Aaro Koskinen wrote: Hi, I think USB tree introduced

Re: DWC3: Event Interrupt Mask issue

2013-07-16 Thread Felipe Balbi
Hi, On Tue, Jul 16, 2013 at 10:25:09AM +0800, Huang Rui wrote: sorry for the delay, but I just tested my 'testing' branch on OMAP5 uEVM board and it's working fine, then I merged my dwc3-no-oneshot branch on top and I still have dwc3 working. So this patch which I sent you is alright.

[GIT PULL] usb: fixes for v3.11-rc1

2013-07-16 Thread Felipe Balbi
Hi Greg, Here's my first set of fixes for v3.11-rc1, they have been boot tested on omap5-uevm (for dwc3 changes) and built tested for ARM and x86 with different .configs Consider merging this tree on your usb-linus branch. Let me know if you want any changes on this pull request cheers The

RE: linux-next: Tree for Jul 15 (usb gadget and rdnis)

2013-07-16 Thread Andrzej Pietrasiewicz
Hello, On Monday, July 15, 2013 5:55 PM Randy Dunlap wrote: On 07/14/13 20:45, Stephen Rothwell wrote: Hi all, The merge window is closed. Time to clean up all those unclean trees. The good among you can just fast forward to v3.11-rc1 ;-) Changes since 20130712: on x86_64:

Re: [PATCH v12 00/13] Add tested id switch and vbus connect detect support for Chipidea

2013-07-16 Thread Peter Chen
On Fri, Jul 12, 2013 at 03:18:31PM +0200, Marek Vasut wrote: Hi Peter, On Fri, Jul 12, 2013 at 06:04:43AM +0200, Marek Vasut wrote: Hi Peter, On Thu, Jul 11, 2013 at 07:57:19PM +0200, Marek Vasut wrote: Hi Peter, This patchset adds tested otg id switch function and

Re: [PATCH v12 06/13] usb: chipidea: add otg_cap attribute for otg capable

2013-07-16 Thread Peter Chen
On Fri, Jul 12, 2013 at 12:42:10PM +0300, Alexander Shishkin wrote: Peter Chen peter.c...@freescale.com writes: Hi Alex, do you have any other comments for this patchset? If you haven't, I will send the v13 patchset with current comments. -- Best Regards, Peter Chen -- To unsubscribe from

Re: [PATCH 1/2] usb: chipidea: kconfig: using USB_EHCI_HCD instead of USB for dependency

2013-07-16 Thread Peter Chen
On Mon, Jul 8, 2013 at 3:42 PM, Peter Chen peter.c...@freescale.com wrote: Chipidea controller is an EHCI compatible controller, with current kconfig, even CONIFG_USB_EHCI_HCD is not chosen, but CONFIG_USB is there, the chipidea code still will be compiled, in fact, it is useless.

Re: [PATCH v2 0/7] usb: phy: msm: Fixes and cleanups

2013-07-16 Thread Ivan T. Ivanov
Hi Felipe, On Tue, 2013-07-09 at 18:47 +0300, Ivan T. Ivanov wrote: From: Ivan T. Ivanov iiva...@mm-sol.com Changes since first version. * Extend commit messages a little bit. Following patches make initial cleanup of usb phy found in the Qualcomm chipsets. Changes include: * Build

Re: [Ksummit-2013-discuss] Topic: where should the border between userspace and kernel be?

2013-07-16 Thread Linus Walleij
On Tue, Jul 16, 2013 at 12:39 PM, Miklos Szeredi mik...@szeredi.hu wrote: Is the border always where it should be? Are there any candidates for moving things into, or out of the kernel? Are there some guidelines for deciding which functionality belongs to the kernel? Now I need to flag

DWC2 and/or S3C-HSOTG for STA2X11 board

2013-07-16 Thread Federico Vaga
Hello, I have an x86 board made by STMicroelectronics (STA2X11) with the Synopsis USB-OTG DesignWare 2 on it and connected through the PCI-e bus. I know that there are two drivers for the same controller: (host) drivers/staging/dwc2/* (device) drivers/usb/gadget/s3c-hsotg.{c|h} So,

Re: DWC2 and/or S3C-HSOTG for STA2X11 board

2013-07-16 Thread Felipe Balbi
Hi, On Tue, Jul 16, 2013 at 02:01:33PM +0200, Federico Vaga wrote: Hello, I have an x86 board made by STMicroelectronics (STA2X11) with the Synopsis USB-OTG DesignWare 2 on it and connected through the PCI-e bus. I know that there are two drivers for the same controller: (host)

[PATCH 00/21] Equivalent of g_mass_storage with configfs

2013-07-16 Thread Andrzej Pietrasiewicz
This series aims at integrating configfs into mass storage, the way it has been done for acm, ncm, ecm, eem, ecm subset, rndis, obex and phonet. It contains everything that is required to provide the equivalent of g_mass_storage.ko with configfs. Mass storage itself is quite large, so the

[PATCH 02/21] usb/gadget: f_phonet: remove unused preprocessor conditional

2013-07-16 Thread Andrzej Pietrasiewicz
The compatibility layer which the USBF_PHONET_INCLUDED was a part of is no longer present - the USBF_PHONET_INCLUDED is not #defined by anyone anymore, so the ifndef is always true. Removing it. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park

[PATCH 14/21] usb/gadget: f_mass_storage: use fsg_common_create_luns in fsg_common_init

2013-07-16 Thread Andrzej Pietrasiewicz
fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/f_mass_storage.c | 89

[PATCH 09/21] usb/gadget: f_mass_storage: use fsg_common_setup in fsg_common_init

2013-07-16 Thread Andrzej Pietrasiewicz
fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/f_mass_storage.c | 19 +++ 1

[PATCH 03/21] usb/gadget: configfs: add a method to unregister the gadget

2013-07-16 Thread Andrzej Pietrasiewicz
Add a method to unregister the gadget using its config_item. There can be functions (e.g. mass storage), which in some circumstances need the gadget stopped. Add a method of stopping the gadget. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park

[PATCH 01/21] usb/gadget: multi: fix error return code in cdc_do_config()

2013-07-16 Thread Andrzej Pietrasiewicz
Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Introduced by commit 59835a (usb: gadget: multi: use function framework for ACM.) Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park

[PATCH 10/21] usb/gadget: f_mass_storage: use fsg_common_set_num_buffers in fsg_common_init

2013-07-16 Thread Andrzej Pietrasiewicz
fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/f_mass_storage.c | 29

[PATCH 11/21] usb/gadget: f_mass_storage: use fsg_common_set_nluns in fsg_common_init

2013-07-16 Thread Andrzej Pietrasiewicz
fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/f_mass_storage.c | 22 +- 1

[PATCH 20/21] usb/gadget: storage_common: add methods to show/store 'cdrom' and 'removable'

2013-07-16 Thread Andrzej Pietrasiewicz
This will be required by configfs integration. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/storage_common.c | 42 +++ drivers/usb/gadget/storage_common.h |5 2

[PATCH 07/21] usb/gadget: f_mass_storage: use usb_gstrings_attach

2013-07-16 Thread Andrzej Pietrasiewicz
Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/f_mass_storage.c | 25 - 1 files changed, 12 insertions(+), 13 deletions(-) diff --git a/drivers/usb/gadget/f_mass_storage.c

[PATCH 17/21] usb/gadget: f_mass_storage: convert to new function interface with backward compatibility

2013-07-16 Thread Andrzej Pietrasiewicz
Converting mass storage to the new function interface requires converting the USB mass storage's function code and its users. This patch converts the f_mass_storage.c to the new function interface. The file is now compiled into a separate usb_f_mass_storage.ko module. The old function interface is

[PATCH 13/21] usb/gadget: f_mass_storage: use fsg_common_set_cdev in fsg_common_init

2013-07-16 Thread Andrzej Pietrasiewicz
fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/f_mass_storage.c | 23 ++- 1

[PATCH 12/21] usb/gadget: f_mass_storage: use fsg_common_set_ops/_private_data in fsg_common_init

2013-07-16 Thread Andrzej Pietrasiewicz
fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/f_mass_storage.c |5 +++-- 1 files changed, 3

[PATCH 18/21] usb/gadget: mass_storage: convert to new interface of f_mass_storage

2013-07-16 Thread Andrzej Pietrasiewicz
Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/Kconfig|1 + drivers/usb/gadget/mass_storage.c | 107 +++- 2 files changed, 81 insertions(+), 27 deletions(-) diff

[PATCH 05/21] usb/gadget: f_mass_storage: factor out a header file

2013-07-16 Thread Andrzej Pietrasiewicz
In order to prepare for the new function interface the f_mass_storage.c needs to be compiled as a module, and so a header file will be required. This patch factors out some code to a new f_mass_storage.h. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park

[PATCH 16/21] usb/gadget: f_mass_storage: use fsg_common_run_thread in fsg_common_init

2013-07-16 Thread Andrzej Pietrasiewicz
fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/f_mass_storage.c | 14 +++--- 1 files

[PATCH 08/21] usb/gadget: f_mass_storage: split fsg_common initialization into a number of functions

2013-07-16 Thread Andrzej Pietrasiewicz
When configfs is in place, the things related to intialization of struct fsg_common will be split over a number of places. This patch adds several functions which together cover the former intialization routine fsg_common_init. As a consequence, issuing of some debug messages needs to be adjusted.

[PATCH 06/21] usb/gadget: f_mass_storage: add a level of indirection for luns storage

2013-07-16 Thread Andrzej Pietrasiewicz
This is needed to prepare for configfs integration. So far the luns have been allocated during gadget's initialization, based on the nluns module parameter's value; the exact number is known when the gadget is initialized and that number of luns is allocated in one go; they all will be used.

[PATCH 19/21] usb/gadget: storage_common: make attribute operations more generic

2013-07-16 Thread Andrzej Pietrasiewicz
Show/store methods for sysfs attributes contain code which can be used also by configfs. Make them abstract the source the lun and rw_semaphore are taken from. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com ---

[PATCH 21/21] usb/gadget: f_mass_storage: add configfs support

2013-07-16 Thread Andrzej Pietrasiewicz
Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- .../ABI/testing/configfs-usb-gadget-mass-storage | 31 ++ drivers/usb/gadget/Kconfig | 11 + drivers/usb/gadget/f_mass_storage.c|

[PATCH 0/1] Equivalent of g_acm_ms.ko with configfs

2013-07-16 Thread Andrzej Pietrasiewicz
Here I present the conversion of everything that is required to provide the equivalent of g_acm_ms.ko with configfs. In fact this series consists of just one patch; everything required to provide the equivalent of g_acm_ms.ko with configfs has been done in the series related to the

[PATCH 1/1] usb/gadget: acm_ms: convert to new interface of f_mass_storage

2013-07-16 Thread Andrzej Pietrasiewicz
Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/Kconfig |1 + drivers/usb/gadget/acm_ms.c | 113 --- 2 files changed, 75 insertions(+), 39 deletions(-) diff

[PATCH 2/5] usb/gadget: multi: convert to new interface of f_rndis

2013-07-16 Thread Andrzej Pietrasiewicz
Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/Kconfig |3 +- drivers/usb/gadget/multi.c | 66 ++-- 2 files changed, 47 insertions(+), 22 deletions(-) diff

[PATCH 0/5] Equivalent of g_multi.ko with configfs

2013-07-16 Thread Andrzej Pietrasiewicz
Here I present the conversion of everything that is required to provide the equivalent of g_multi.ko with configfs. A branch will be available here since 16th July: git://git.infradead.org/users/kmpark/linux-samsung usb-gadget-configfs BACKWARD COMPATIBILITY == Please note

[PATCH 3/5] usb/gadget: multi: convert to new interface of f_mass_storage

2013-07-16 Thread Andrzej Pietrasiewicz
Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/Kconfig |1 + drivers/usb/gadget/multi.c | 112 2 files changed, 83 insertions(+), 30 deletions(-) diff --git

[PATCH 5/5] usb/gadget: mass_storage: merge usb_f_mass_storage module with u_ms module

2013-07-16 Thread Andrzej Pietrasiewicz
Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/Kconfig |7 --- drivers/usb/gadget/Makefile |4 +--- 2 files changed, 1 insertions(+), 10 deletions(-) diff --git a/drivers/usb/gadget/Kconfig

[PATCH 1/5] usb/gadget: multi: convert to new interface of f_ecm

2013-07-16 Thread Andrzej Pietrasiewicz
Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/Kconfig |1 + drivers/usb/gadget/multi.c | 61 +++ 2 files changed, 56 insertions(+), 6 deletions(-) diff --git

[PATCH 4/5] usb/gadget: f_mass_storage: remove compatibility layer

2013-07-16 Thread Andrzej Pietrasiewicz
There are no more old interface users left. Remove it. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/usb/gadget/f_mass_storage.c | 154 +-- drivers/usb/gadget/f_mass_storage.h | 21

Re: DWC2 and/or S3C-HSOTG for STA2X11 board

2013-07-16 Thread Federico Vaga
Thank you Felipe [add CC Giancarlo from ST] On Tuesday 16 July 2013 15:04:25 Felipe Balbi wrote: Hi, On Tue, Jul 16, 2013 at 02:01:33PM +0200, Federico Vaga wrote: Hello, I have an x86 board made by STMicroelectronics (STA2X11) with the Synopsis USB-OTG DesignWare 2 on it and

[GIT PULL] PHY Framework for 3.12 merge window

2013-07-16 Thread Kishon Vijay Abraham I
Hi Greg, Here's the pull request for the PHY Framework which I've been working on for a while. I've applied these patches on the current linux mainline HEAD (which has one commit after 3.11-rc1 is tagged). Even before this framework is merged, quite a few people started using it. I've queued

Re: DWC2 and/or S3C-HSOTG for STA2X11 board

2013-07-16 Thread Felipe Balbi
On Tue, Jul 16, 2013 at 02:27:43PM +0200, Federico Vaga wrote: Thank you Felipe [add CC Giancarlo from ST] On Tuesday 16 July 2013 15:04:25 Felipe Balbi wrote: Hi, On Tue, Jul 16, 2013 at 02:01:33PM +0200, Federico Vaga wrote: Hello, I have an x86 board made by

Re: [PATCH v2 0/4] usb: musb: fix USB enumeration issue in OMAP3 platform

2013-07-16 Thread Felipe Balbi
Hi, On Tue, Jul 16, 2013 at 11:22:38AM +0530, Kishon Vijay Abraham I wrote: Tony, On Monday 08 July 2013 04:44 PM, Kishon Vijay Abraham I wrote: This series fixes the USB enumeration issue caused because of the controller not able to get a reference to the PHY because of incorrect binding

Re: [PATCH v2 1/4] arm: omap: Add phy binding info for musb in plat data

2013-07-16 Thread Tony Lindgren
* Kishon Vijay Abraham I kis...@ti.com [130708 04:20]: In order for controllers to get PHY in case of non dt boot, the phy binding information (phy label) should be added in the platform data of the controller. Signed-off-by: Kishon Vijay Abraham I kis...@ti.com Acked-by: Felipe Balbi

Re: [PATCH v2 4/4] arm: omap: remove using usb_bind_phy for binding musb and phy

2013-07-16 Thread Tony Lindgren
* Kishon Vijay Abraham I kis...@ti.com [130708 04:21]: Now that MUSB for OMAP started using devm_usb_get_phy_by_name which does not require PHY library to already have the binding information, removed usb_bind_phy calls that binds the MUSB controller with the PHY from the board files.

dwc2: Invalid (?) initialization of PCGCTL register

2013-07-16 Thread Matthijs Kooijman
Hi Paul, could you perhaps look up some documentation on the PCGCTL register for me? The issue John and I have been facing is that on one particular board, the dwc2 driver fails to initialize saying: dwc2 101c.usb: Bad value for GSNPSID: 0x We've traced this back to the

Re: dwc2: Invalid (?) initialization of PCGCTL register

2013-07-16 Thread John Crispin
Hi, Simply resetting the PCGCTL register in hcd_init or somewhere around there fixes the problem. However, John suggested an even more generic problem: using the kernel-wide reset controller driver to completely reset the dwc controller before initializing it (by calling device_reset()). On

Re: [PATCH 06/21] usb/gadget: f_mass_storage: add a level of indirection for luns storage

2013-07-16 Thread Alan Stern
On Tue, 16 Jul 2013, Andrzej Pietrasiewicz wrote: This is needed to prepare for configfs integration. So far the luns have been allocated during gadget's initialization, based on the nluns module parameter's value; the exact number is known when the gadget is initialized and that number of

Re: [PATCH 08/21] usb/gadget: f_mass_storage: split fsg_common initialization into a number of functions

2013-07-16 Thread Alan Stern
On Tue, 16 Jul 2013, Andrzej Pietrasiewicz wrote: When configfs is in place, the things related to intialization of struct fsg_common will be split over a number of places. This patch adds several functions which together cover the former intialization routine fsg_common_init. As a

[PATCH v3 6/7] USB: ohci-at91: add usb_clk for transition to common clk framework

2013-07-16 Thread Boris BREZILLON
The AT91 PMC (Power Management Controller) provides an USB clock used by USB Full Speed host (ohci) and USB Full Speed device (udc). The usb drivers (ohci and udc) must configure this clock to 48Mhz. This configuration was formely done in mach-at91/clock.c, but this implementation will be removed

[PATCH v3 7/7] usb: gadget: at91_udc: add usb_clk for transition to common clk framework

2013-07-16 Thread Boris BREZILLON
The AT91 PMC (Power Management Controller) provides an USB clock used by USB Full Speed host (ohci) and USB Full Speed device (udc). The usb drivers (ohci and udc) must configure this clock to 48Mhz. This configuration was formely done in mach-at91/clock.c, but this implementation will be removed

[PATCH v3 4/7] USB: gadget: atmel_usba: prepare clk before calling enable

2013-07-16 Thread Boris BREZILLON
Replace clk_enable/disable with clk_prepare_enable/disable_unprepare to avoid common clk framework warnings. Signed-off-by: Boris BREZILLON b.brezil...@overkiz.com --- drivers/usb/gadget/atmel_usba_udc.c | 28 +--- 1 file changed, 21 insertions(+), 7 deletions(-) diff

Re: [PATCH] usb: udc: add gadget state kobject uevent

2013-07-16 Thread Greg KH
On Tue, Jul 16, 2013 at 11:49:07AM +0800, Rong Wang wrote: Hi Greg, The USB on our platform can change roles between HOST and GADGET, but it is not capable of OTG. That kind of sounds like the definition of OTG :) When the USB changes between roles the udev will run some scripts

Re: [PATCH v3 6/7] USB: ohci-at91: add usb_clk for transition to common clk framework

2013-07-16 Thread Alan Stern
On Tue, 16 Jul 2013, Boris BREZILLON wrote: The AT91 PMC (Power Management Controller) provides an USB clock used by USB Full Speed host (ohci) and USB Full Speed device (udc). The usb drivers (ohci and udc) must configure this clock to 48Mhz. This configuration was formely done in

Re: [PATCH v3 6/7] USB: ohci-at91: add usb_clk for transition to common clk framework

2013-07-16 Thread boris brezillon
Hello Alan, On 16/07/2013 18:48, Alan Stern wrote: On Tue, 16 Jul 2013, Boris BREZILLON wrote: The AT91 PMC (Power Management Controller) provides an USB clock used by USB Full Speed host (ohci) and USB Full Speed device (udc). The usb drivers (ohci and udc) must configure this clock to

Re: v3.11-rc1 USB regressions

2013-07-16 Thread Aaro Koskinen
Hi, On Tue, Jul 16, 2013 at 08:33:06AM +0200, Daniel Mack wrote: fe4cb0912f8e737f8e4b8b38b9e692f8062f5423 usb: musb: gadget: remove hcd initialization (Reverting this fixes error cdc_ether: probe of 4-1:1.0 failed with error -110 seen on the host side.) Which role does

Re: [PATCH v2 1/2] ARM: dts: tegra20: Rename USB UTMI parameters according to new definitions

2013-07-16 Thread Stephen Warren
On 06/26/2013 06:34 AM, Mikko Perttunen wrote: This patch changes the Tegra20 USB PHY nodes to use the UTMI configuration parameter names as specified in the device tree binding documentation after patch ARM: tegra: finalize USB EHCI and PHY bindings. So, I think the chances of this patch

RE: dwc2: Invalid (?) initialization of PCGCTL register

2013-07-16 Thread Paul Zimmerman
From: Matthijs Kooijman [mailto:matth...@stdin.nl] Sent: Tuesday, July 16, 2013 7:51 AM Hi Paul, could you perhaps look up some documentation on the PCGCTL register for me? The issue John and I have been facing is that on one particular board, the dwc2 driver fails to initialize

Re: [GIT PULL] usb: fixes for v3.11-rc1

2013-07-16 Thread Greg KH
On Tue, Jul 16, 2013 at 11:49:37AM +0300, Felipe Balbi wrote: Hi Greg, Here's my first set of fixes for v3.11-rc1, they have been boot tested on omap5-uevm (for dwc3 changes) and built tested for ARM and x86 with different .configs Consider merging this tree on your usb-linus branch.

Re: v3.11-rc1 USB regressions

2013-07-16 Thread Aaro Koskinen
Hi, On Tue, Jul 16, 2013 at 11:22:47AM +0300, Felipe Balbi wrote: On Tue, Jul 16, 2013 at 12:30:00PM +0530, Kishon Vijay Abraham I wrote: On Tuesday 16 July 2013 12:03 PM, Daniel Mack wrote: Hi Aaro, On 16.07.2013 00:56, Aaro Koskinen wrote: Hi, I think USB tree

Re: v3.11-rc1 USB regressions

2013-07-16 Thread Daniel Mack
On 16.07.2013 20:26, Aaro Koskinen wrote: Hi, On Tue, Jul 16, 2013 at 11:22:47AM +0300, Felipe Balbi wrote: On Tue, Jul 16, 2013 at 12:30:00PM +0530, Kishon Vijay Abraham I wrote: On Tuesday 16 July 2013 12:03 PM, Daniel Mack wrote: Hi Aaro, On 16.07.2013 00:56, Aaro Koskinen wrote: Hi,

Re: v3.11-rc1 USB regressions

2013-07-16 Thread Aaro Koskinen
On Tue, Jul 16, 2013 at 08:30:21PM +0200, Daniel Mack wrote: On 16.07.2013 20:26, Aaro Koskinen wrote: Hi, On Tue, Jul 16, 2013 at 11:22:47AM +0300, Felipe Balbi wrote: On Tue, Jul 16, 2013 at 12:30:00PM +0530, Kishon Vijay Abraham I wrote: On Tuesday 16 July 2013 12:03 PM, Daniel Mack

Re: Audio I/O parameters

2013-07-16 Thread Alan Stern
On Mon, 15 Jul 2013, James Stone wrote: James, did you ever provide a usbmon trace for 3.8 doing playback only and using 64 frames/period? I don't recall seeing any. It might help. OK - will send it to you off-list. I got it. It explains a lot. The audio-out stream uses a pipeline

RE: DWC2 and/or S3C-HSOTG for STA2X11 board

2013-07-16 Thread Paul Zimmerman
From: Federico Vaga [mailto:federico.v...@gmail.com] Sent: Tuesday, July 16, 2013 5:02 AM I have an x86 board made by STMicroelectronics (STA2X11) with the Synopsis USB-OTG DesignWare 2 on it and connected through the PCI-e bus. I know that there are two drivers for the same controller:

Re: [PATCH v3 6/7] USB: ohci-at91: add usb_clk for transition to common clk framework

2013-07-16 Thread Alan Stern
On Tue, 16 Jul 2013, boris brezillon wrote: + uclk = clk_get(pdev-dev, usb_clk); + if (IS_ERR(uclk)) { + uclk = NULL; + dev_warn(pdev-dev, failed to get usb_clk\n); + } Is this really what you want for backward compatibility? Here are some proposition to remove

[PATCH] ARM: omap2: fix musb usage for n8x0

2013-07-16 Thread Daniel Mack
Commit b7e2e75a8c (usb: gadget: drop unused USB_GADGET_MUSB_HDRC) dropped a config symbol that was unused by the musb core, but it turns out that board support code had references to it. As the core now has a fall-back to host-only mode if support for dual-role is not compiled in, so we can just

Re: Audio I/O parameters

2013-07-16 Thread Clemens Ladisch
Alan Stern wrote: The audio-out stream uses a pipeline of only 2 URBs. The URBs start out alternating between 8 and 7 packets apiece. This yields a total latency around 1.9 ms (equivalent to 2 periods of about 41 frames at 44.1 KHz), which is smaller than I would expect (2 periods of 64

[PATCH] staging: dwc2: add driver parameter to set AHB config register value

2013-07-16 Thread Paul Zimmerman
The dwc2 driver sets the value of the DWC2 GAHBCFG register to 0x6, which is GAHBCFG_HBSTLEN_INCR4. But different platforms may require different values. In particular, the Broadcom 2835 SOC used in the Raspberry Pi needs a value of 0x10, otherwise the DWC2 controller stops working after a short

Re: [PATCH] ARM: omap2: fix musb usage for n8x0

2013-07-16 Thread Felipe Balbi
On Tue, Jul 16, 2013 at 08:52:57PM +0200, Daniel Mack wrote: Commit b7e2e75a8c (usb: gadget: drop unused USB_GADGET_MUSB_HDRC) dropped a config symbol that was unused by the musb core, but it turns out that board support code had references to it. As the core now has a fall-back to host-only

Re: [PATCH v3 6/7] USB: ohci-at91: add usb_clk for transition to common clk framework

2013-07-16 Thread boris brezillon
On 16/07/2013 20:47, Alan Stern wrote: On Tue, 16 Jul 2013, boris brezillon wrote: + uclk = clk_get(pdev-dev, usb_clk); + if (IS_ERR(uclk)) { + uclk = NULL; + dev_warn(pdev-dev, failed to get usb_clk\n); + } Is this really what you want for

Re: Audio I/O parameters

2013-07-16 Thread Alan Stern
On Tue, 16 Jul 2013, Clemens Ladisch wrote: Alan Stern wrote: The audio-out stream uses a pipeline of only 2 URBs. The URBs start out alternating between 8 and 7 packets apiece. This yields a total latency around 1.9 ms (equivalent to 2 periods of about 41 frames at 44.1 KHz), which is

[no subject]

2013-07-16 Thread Julio Cesar Pompa Puente
'Tanto si piensas que puedes, como si piensas que no puedes, estás en lo cierto' Henry Ford -- 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-info.html

Kedves Email felhasznaloi

2013-07-16 Thread Admin
Kedves Email felhasznaloi, A postafiok korulbelul haladja meg az egy vagy tobb merethatar a rendszergazda altal beallitott. Mivel ezek a hibauzenet, hatassal lesz a szallitas a bejovo uzenet (ek), vagy az elkuldott uzenet nem kezbesitheto. on tanacsot ujraervenyesitesehez es frissitsd a Mailbox,

Re: Audio I/O parameters

2013-07-16 Thread James Stone
On Tue, Jul 16, 2013 at 7:36 PM, Alan Stern st...@rowland.harvard.edu wrote: On Mon, 15 Jul 2013, James Stone wrote: James, did you ever provide a usbmon trace for 3.8 doing playback only and using 64 frames/period? I don't recall seeing any. It might help. OK - will send it to you

Re: Incompleteness of HW capability registers (Was: [PATCH] staging: dwc2: fix thinko in dwc2_hc_set_even_odd_frame())

2013-07-16 Thread Stephen Warren
On 07/15/2013 02:27 AM, Matthijs Kooijman wrote: Hi Stephen, I'm not sure how many of the below driver parameters are actually required. I think as least the three fifo size parameters, plus max_transfer_size and max_packet_count, are needed. I'm curious why any of them are required; why

Re: [PATCH] usb: udc: add gadget state kobject uevent

2013-07-16 Thread Peter Chen
On Mon, Jul 15, 2013 at 11:31:17PM -0700, Greg KH wrote: On Tue, Jul 16, 2013 at 11:49:07AM +0800, Rong Wang wrote: Hi Greg, The USB on our platform can change roles between HOST and GADGET, but it is not capable of OTG. That kind of sounds like the definition of OTG :) When the

RE: question on skel_read func of usb_skeleton.c‏

2013-07-16 Thread WangChen
Oliver, For the func skel_read(), line 328~330, for the case rv ==0, can we just do retry, similar as the code line 301. I hope this can help read() get data in one time for blocking IO if the data to be read is short and can be received in quick time.  The existing retry logic upon should be

RE: question on skel_read func of usb_skeleton.c‏

2013-07-16 Thread WangChen
Sorry, I think existing code should already handle retry for blcoking case.  My idea to retry anyway @line328 is just intention to make the code looks better and handle the return of -EAGAIN in one place which is handled by retry logic. Anyway, this should be a minor suggestion, you can ignore

Re: [GIT PULL] PHY Framework for 3.12 merge window

2013-07-16 Thread Greg Kroah-Hartman
On Tue, Jul 16, 2013 at 06:08:26PM +0530, Kishon Vijay Abraham I wrote: Hi Greg, Here's the pull request for the PHY Framework which I've been working on for a while. I've applied these patches on the current linux mainline HEAD (which has one commit after 3.11-rc1 is tagged). Even

Re: [PATCH] xhci: Correct misplaced newlines

2013-07-16 Thread Sarah Sharp
Thanks Joe, I'll apply this to my tree after the merge window closes. Sarah Sharp On Tue, Jul 16, 2013 at 07:25:59PM -0700, Joe Perches wrote: Logging messages end in newlines, not have them put in the middle of messages. Signed-off-by: Joe Perches j...@perches.com ---

[PATCH] [RFC] EHCI: add to memory barrier to updating hw_next

2013-07-16 Thread 김기오
Hi, I have a missing urb completion problem on ARMv7 based platform. I thought the above problem was caused by coherent memory between the EHCI device and CPU so I tryied to allocates device type memory for EHCI via dma_declare_coherent_memory at machine initialization step so that EHCI always

Re: [PATCH] usb: udc: add gadget state kobject uevent

2013-07-16 Thread Rong Wang
Hi Greg, On Tue, Jul 16, 2013 at 2:31 PM, Greg KH gre...@linuxfoundation.org wrote: On Tue, Jul 16, 2013 at 11:49:07AM +0800, Rong Wang wrote: Hi Greg, The USB on our platform can change roles between HOST and GADGET, but it is not capable of OTG. That kind of sounds like the definition of