Jayan John jayanjoh...@gmail.com writes:
I am developing a custom USB device on a iMX6q platform (Wandboard)
Chipidea HDRC (highspeed dual role controller). The HID interface
consists of a single Interrupt IN ep and ep0. It is required to send
HID reports from Host to Gadget over ep0 (with
Peter Chen peter.c...@freescale.com writes:
Currently, connect gadget is unconditional after binding,
but some function drivers may want to connect gadget on the fly.
With this API, the function driver can disconnect gadget during
the initialization, and connect gadget when it wants.
During
Peter Chen peter.c...@freescale.com writes:
On Thu, Jan 29, 2015 at 12:27:23PM +0800, Peter Chen wrote:
Hi Felipe,
Hi Felipe, I see you tree is closed, but below three patches are
not in your tree, will you queue them now or at next rc? I have
some other patches based on them, so I would
andriy.shevche...@linux.intel.com
Reviewed-by: Alexander Shishkin alexander.shish...@linux.intel.com
--
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
a bypass list, which currently
consists of chipidea ids. Since now it's bigger than just chipidea.
Acked-by: Alan Stern st...@rowland.harvard.edu
Signed-off-by: Andy Shevchenko andriy.shevche...@linux.intel.com
Reviewed-by: Alexander Shishkin alexander.shish...@linux.intel.com
--
To unsubscribe
Peter Chen peter.c...@freescale.com writes:
During this API, the deactivation count will be update, and it
will try to connect or disconnect gadget. It can be used to
enable functions for gadget.
It's up to Felipe, but I'd like to see answers to the following
questions in this commit message:
Peter Chen peter.c...@freescale.com writes:
On Tue, Jan 27, 2015 at 11:21:56AM -0500, Alan Stern wrote:
On Tue, 27 Jan 2015, Peter Chen wrote:
Please fix this properly.
I don't know the right way to fix this. Alan, has you any suggestion?
It depends. How did the code
Andy Shevchenko andriy.shevche...@linux.intel.com writes:
On some Intel MID platforms the ChipIdea USB controller is used. The EHCI PCI
is in conflict with the proper driver. The patch makes ehci-pci to be ignored
in favour of ChipIdea controller.
Signed-off-by: Andy Shevchenko
Peter Chen peter.c...@freescale.com writes:
@@ -348,23 +356,14 @@ static void usb_gadget_remove_driver(struct usb_udc
*udc)
*/
void usb_del_gadget_udc(struct usb_gadget *gadget)
{
- struct usb_udc *udc = NULL;
-
- mutex_lock(udc_lock);
- list_for_each_entry(udc,
Peter Chen peter.c...@freescale.com writes:
This patch set cleans up udc drivers which define .pullup API, but
still call pullup operation at .udc_start or .udc_stop. In fact,
the related pullup operations are covered by udc core.
Oh, this driver is still around? Are we going to obsolete it
Peter Chen peter.c...@freescale.com writes:
+ mutex_lock(udc_lock);
+ list_for_each_entry(udc, udc_list, list)
+ if (udc-gadget == gadget)
+ goto found;
+ mutex_unlock(udc_lock);
Not entirely specific to this patchset, but this search pattern is
Greg KH gre...@linuxfoundation.org writes:
From: Greg Kroah-Hartman gre...@linuxfoundation.org
Alexander isn't able to maintain the Chipidea code anymore, and as Peter
has been acting as the de-facto maintainer anyway, make it official.
Cc: Alexander Shishkin alexander.shish
Hector Palacios hector.palac...@digi.com writes:
Dear Peter,
On 08/27/2013 04:51 AM, Peter Chen wrote:
On Mon, Aug 26, 2013 at 06:24:48PM +0200, Hector Palacios wrote:
Dear Peter,
On 08/12/2013 03:22 AM, Peter Chen wrote:
This patchset adds tested otg id switch function and vbus connect
Hi,
These are chipidea patches for v3.12. The highlight of the series is
vbus detection support. Other than that, a few cleanups and fixes,
mostly concerning imx platforms. Everything is sparse, smatch and
coccinelle clean and bisectable.
Fabio Estevam (3):
usb: chipidea: ci_hdrc_imx: remove
this function into probe.
After this change, USB Host1 port on mx53qsb board is functional.
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Tested-by: Arnaud Patard arnaud.pat...@rtp-net.org
Reviewed-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish
From: Fabio Estevam fabio.este...@freescale.com
'res' is not used anywhere, so let's get rid of it.
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Acked-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb
From: Lothar Waßmann l...@karo-electronics.de
This prevents the USB PHY refcount to be decremented below zero upon
unloading the ci-hdrc-imx module.
Signed-off-by: Lothar Waßmann l...@karo-electronics.de
Acked-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin
-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci_hdrc_imx.c | 26 ++
drivers/usb/chipidea/core.c| 23 +++
include/linux/usb/chipidea.h | 1 +
3 files changed, 26 insertions(+), 24 deletions(-)
diff
peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci_hdrc_imx.c | 1 -
drivers/usb/chipidea/ci_hdrc_msm.c | 1 -
drivers/usb/chipidea/udc.c | 24 ++--
include/linux/usb/chipidea.h | 1 -
4
From: Fabio Estevam fabio.este...@freescale.com
After the rename to ci_hdrc we ended up with two MODULE_ALIAS entries, so
remove the old one.
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Reviewed-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin
-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/core.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/chipidea/core.c b/drivers/usb/chipidea/core.c
index ab01e18..e9cfd31 100644
--- a/drivers/usb/chipidea/core.c
+++ b/drivers
From: Peter Chen peter.c...@freescale.com
It is useless at below cases:
- If we implement both usb host and device at chipidea driver.
- If we don't need phy-otg.
Tested-by: Marek Vasut ma...@denx.de
Signed-off-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin
From: Peter Chen peter.c...@freescale.com
For chipidea, the IP must know vbus before the controller
begins to run. So the .pullup should only be called when
the vbus is there.
Tested-by: Marek Vasut ma...@denx.de
Signed-off-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander
From: Peter Chen peter.c...@freescale.com
This file is mainly used to access otgsc currently, it may
add otg related things in the future.
Tested-by: Marek Vasut ma...@denx.de
Signed-off-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
Vasut ma...@denx.de
Signed-off-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci.h | 5 +
drivers/usb/chipidea/core.c | 28 +++-
drivers/usb/chipidea/otg.c | 42
-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/core.c | 10 --
drivers/usb/chipidea/host.c | 7 +++
drivers/usb/chipidea/host.h | 6 ++
drivers/usb/chipidea/udc.c | 36
From: Peter Chen peter.c...@freescale.com
Remove an unused macro leftover from the old initialization code.
Signed-off-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci_hdrc_imx.c | 3 ---
1 file changed, 3
From: Peter Chen peter.c...@freescale.com
Move otg related things to otg file.
Tested-by: Marek Vasut ma...@denx.de
Signed-off-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/core.c | 63
register.
- When the gadget starts (load one gadget module), the controller
can run if vbus is on (CI_HDRC_PULLUP_ON_VBUS), it also does not
relate to shared register.
Tested-by: Marek Vasut ma...@denx.de
Signed-off-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin
From: Lothar Waßmann l...@karo-electronics.de
If a role fails to start, propagate the error code up the call stack
from probe.
Signed-off-by: Lothar Waßmann l...@karo-electronics.de
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/core.c | 1 -
1
-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/host.c | 23 ++-
1 file changed, 22 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/chipidea/host.c b/drivers/usb/chipidea/host.c
index
-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/core.c | 37 ++---
include/linux/usb/chipidea.h | 5 +
2 files changed, 35 insertions(+), 7 deletions(-)
diff --git a/drivers/usb/chipidea/core.c b/drivers/usb/chipidea/core.c
...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci.h | 3 +++
drivers/usb/chipidea/core.c | 32
drivers/usb/chipidea/otg.c | 4
3 files changed, 39 insertions(+)
diff --git a/drivers/usb
-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci_hdrc_imx.c | 71 +---
drivers/usb/chipidea/ci_hdrc_imx.h | 17 ++-
drivers/usb/chipidea/usbmisc_imx.c | 95 +-
3 files changed, 76 insertions(+), 107 deletions
.
Signed-off-by: Lothar Waßmann l...@karo-electronics.de
Reviewed-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/Kconfig | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/usb
Peter Chen peter.c...@freescale.com writes:
On Mon, Aug 12, 2013 at 09:22:46AM +0800, Peter Chen wrote:
This patchset adds tested otg id switch function and vbus connect
and disconnect detection for chipidea driver. And fix kinds of
bugs found at chipidea drivers after enabling id and vbus
Sascha Hauer s.ha...@pengutronix.de writes:
The chipidea i.MX driver is split into two drivers. The ci_hdrc_imx driver
handles the chipidea cores and the usbmisc_imx driver handles the noncore
registers common to all chipidea cores (but SoC specific). Current flow is:
- usbmisc sets an ops
Fabio Estevam fabio.este...@freescale.com writes:
Currently hw_phymode_configure() is located inside hw_device_reset(), which is
only called by chipidea udc driver.
When operating in host mode, we also need to call hw_phymode_configure() in
order to properly configure the PHY mode, so move
Peter Chen peter.c...@freescale.com writes:
Signed-off-by: Peter Chen peter.c...@freescale.com
Added a commit message and applied. Thanks!
--
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
Peter Chen peter.c...@freescale.com writes:
Now, chipidea host has already depended on USB_EHCI_HCD
Signed-off-by: Peter Chen peter.c...@freescale.com
Alan, what do you think?
---
drivers/usb/host/Kconfig |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git
Fabio Estevam feste...@gmail.com writes:
From: Fabio Estevam fabio.este...@freescale.com
After the rename to ci_hdrc we ended up with two MODULE_ALIAS entries, so
remove the old one.
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Applied and pushed, thanks!
--
To unsubscribe
Peter Chen peter.c...@freescale.com writes:
Since we need otgsc to know vbus's status at some chipidea
controllers even it is peripheral-only mode. Besides, some
SoCs (eg, AR9331 SoC) don't have otgsc register even
the DCCPARAMS_DC and DCCPARAMS_HC are both 1 at CAP_DCCPARAMS.
We inroduce
Peter Chen peter.c...@freescale.com writes:
This file is mainly used to access otgsc currently, it may
add otg related things in the future.
Tested-by: Marek Vasut ma...@denx.de
Signed-off-by: Peter Chen peter.c...@freescale.com
---
drivers/usb/chipidea/Makefile |2 +-
Peter Chen peter.c...@freescale.com writes:
A few typos:
Move otg relate things to otg file.
related
Tested-by: Marek Vasut ma...@denx.de
Signed-off-by: Peter Chen peter.c...@freescale.com
---
drivers/usb/chipidea/core.c | 63
+--
Peter Chen peter.c...@freescale.com writes:
On Fri, Aug 09, 2013 at 03:55:56PM +0300, Alexander Shishkin wrote:
+ } else {
+ /*
+ * If the controller is not OTG capable, but support
+ * role switch, the defalt role is gadget
Peter Chen peter.c...@freescale.com writes:
CI_HDRC_REGS_SHARED stands for the controller registers is shared
with other USB drivers, if all USB drivers are at chipidea/, it doesn't
needed to set.
We still have the msm driver that uses REGS_SHARED.
CI_HDRC_PULLUP_ON_VBUS stands for pullup
Alan Stern st...@rowland.harvard.edu writes:
On Fri, 9 Aug 2013, Alexander Shishkin wrote:
Peter Chen peter.c...@freescale.com writes:
Now, chipidea host has already depended on USB_EHCI_HCD
Signed-off-by: Peter Chen peter.c...@freescale.com
Alan, what do you think
Peter Chen peter.c...@freescale.com writes:
- The old operation needs to call hw_alloc_regmap two times, and
the first operation is only used to know if the controller is
lpm supported, besides, there is a kfree before kzalloc, it is also
tricky.
Why tricky? kfree() is called from either
Hi,
Here are two small chipidea fixes for v3.11.
Fabio Estevam (1):
usb: chipidea: cast PORTSC_PTS and DEVLC_PTS macros
Peter Chen (1):
usb: chipidea: fix the build error with randconfig
drivers/usb/chipidea/Kconfig | 4 ++--
drivers/usb/chipidea/bits.h | 4 ++--
2 files changed, 4
]
Reported-by: Felipe Balbi ba...@ti.com
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/bits.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/usb/chipidea/bits.h b
the chipidea host depend on
ehci even it is built as module.
Signed-off-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/Kconfig | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/usb
Fabio Estevam feste...@gmail.com writes:
From: Fabio Estevam fabio.este...@freescale.com
'res' is not used anywhere, so let's get rid of it.
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Applied and pushed, thanks!
--
To unsubscribe from this list: send the line unsubscribe
: probably the parts using the DMA API should
be factored out, instead of disabling the whole USB gadget subsystem.
Indeed. How does this look instead?
This one is much better. FWIW,
Acked-by: Alexander Shishkin alexander.shish...@linux.intel.com
Regards,
--
Alex
--
To unsubscribe from
Peter Chen peter.c...@freescale.com writes:
Since we need otgsc to know vbus's status at some chipidea
controllers even it is peripheral-only mode. Besides, some
SoCs (eg, AR9331 SoC) don't have otgsc register even
the DCCPARAMS_DC and DCCPARAMS_HC are both 1 at CAP_DCCPARAMS.
We inroduce
Peter Chen peter.c...@freescale.com writes:
On Fri, Jul 12, 2013 at 11:12:01AM +0300, Alexander Shishkin wrote:
Peter Chen peter.c...@freescale.com writes:
Since we need otgsc to know vbus's status at some chipidea
controllers even it is peripheral-only mode. Besides, some
SoCs (eg
Peter Chen peter.c...@freescale.com writes:
Using below configs, the compile will have error:
ERROR: ehci_init_driver undefined!
.config:
CONFIG_USB_CHIPIDEA=m
CONFIG_USB_CHIPIDEA_HOST=y
CONFIG_USB_CHIPIDEA_DEBUG=y
The reason is chipidea host uses symbol from ehci, but ehci
is not
Fabio Estevam fabio.este...@freescale.com writes:
Fix the following build warnings on x86:
drivers/usb/chipidea/core.c: In function 'hw_phymode_configure':
drivers/usb/chipidea/core.c:226:3: warning: large integer implicitly
truncated to unsigned type [-Woverflow]
Peter Chen hzpeterc...@gmail.com writes:
On Thu, Mar 21, 2013 at 11:06 AM, Peter Chen peter.c...@freescale.com wrote:
On Wed, Mar 20, 2013 at 01:04:33PM +0200, Alexander Shishkin wrote:
Peter Chen peter.c...@freescale.com writes:
On Fri, Mar 15, 2013 at 05:17:08PM +0200, Alexander Shishkin
Felipe Balbi ba...@ti.com writes:
On Wed, Jun 26, 2013 at 05:37:19PM +0530, George Cherian wrote:
On 6/26/2013 3:46 PM, Felipe Balbi wrote:
Hi,
On Wed, Jun 26, 2013 at 02:59:14PM +0530, George Cherian wrote:
There can be configurations in which DWC3 is hoooked up only to USB2 PHY.
In such
Fabio Estevam fabio.este...@freescale.com writes:
commit ea1418b5f1a (usb: chipidea: i.MX: use devm_usb_get_phy_by_phandle to
get
phy) causes the USB host to miss the disconnect/connect events.
In order to reproduce this problem:
- Insert a USB thumb into the USB host port (connection is
Hi,
Here is one more fix for the usb-next.
Fabio Estevam (1):
usb: chipidea: ci_hdrc_imx: access phy via private data
drivers/usb/chipidea/ci_hdrc_imx.c |9 -
1 file changed, 4 insertions(+), 5 deletions(-)
--
1.7.10.4
--
To unsubscribe from this list: send the line
...@freescale.com
Acked-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci_hdrc_imx.c |9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/usb/chipidea/ci_hdrc_imx.c
b/drivers/usb
Dunlap rdun...@infradead.org
It's actually Felipe's turf, so needs either his ack or target his tree.
I'm ok with this.
FWIW,
Acked-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
Changes for v2:
- Using IS_ENABLED to MACRO define
include/linux/usb/of.h | 16
Greg KH gre...@linuxfoundation.org writes:
On Thu, Jun 13, 2013 at 06:00:04PM +0300, Alexander Shishkin wrote:
ci13xxx is bad for at least the following reasons:
* people often mistype it
* it doesn't add any informational value to the names it's used in
* it needlessly attracts mail
Peter Chen peter.c...@freescale.com writes:
On Thu, Jun 13, 2013 at 06:00:04PM +0300, Alexander Shishkin wrote:
ci13xxx is bad for at least the following reasons:
* people often mistype it
* it doesn't add any informational value to the names it's used in
* it needlessly attracts mail
Barry Song 21cn...@gmail.com writes:
+if USB_CHIPIDEA_UDC USB_CHIPIDEA_HOST
+
+config USB_CHIPIDEA_SIRF
+ depends on ARCH_SIRF
+ bool SiRF USB controller ChipIdea driver binding
+ default y
+ help
+ Say Y here to enable sirf usb ChipIdea driver binding.
+
+config
Barry Song 21cn...@gmail.com writes:
2013/6/14 Alexander Shishkin alexander.shish...@linux.intel.com:
Barry Song 21cn...@gmail.com writes:
+if USB_CHIPIDEA_UDC USB_CHIPIDEA_HOST
+
+config USB_CHIPIDEA_SIRF
+ depends on ARCH_SIRF
+ bool SiRF USB controller ChipIdea driver binding
-by: Felipe Balbi ba...@ti.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/debug.c | 19 +++--
drivers/usb/chipidea/udc.c | 161 +-
drivers/usb/chipidea/udc.h | 11 +--
3 files changed, 129 insertions
'.
Updating only the local reg_vbus is wrong, since we currently check for
data-reg_vbus in the ci13xxx_imx_remove() function.
In order to avoid such issue, just use 'data-reg_vbus' directly.
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish
From: Fabio Estevam fabio.este...@freescale.com
Similarly as it is done in ci13xxx_imx_remove(), only calls of_node_put if
data-phy_np is not NULL.
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb
fabio.este...@freescale.com
Tested-by: Shawn Guo shawn@linaro.org
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci13xxx_imx.c |7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/usb/chipidea/ci13xxx_imx.c
b/drivers/usb/chipidea
Grzeschik m.grzesc...@pengutronix.de
Reviewed-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/core.c |2 +-
drivers/usb/chipidea/udc.c | 20 +++-
drivers/usb/chipidea/udc.h |1 +
3 files
-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
.../devicetree/bindings/usb/ci13xxx-imx.txt|1 +
drivers/usb/chipidea/core.c| 24 +++-
include/linux/usb/chipidea.h |2 +-
3 files changed, 20 insertions(+), 7
m...@pengutronix.de
Signed-off-by: Sascha Hauer s.ha...@pengutronix.de
Acked-by: Felipe Balbi ba...@ti.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/phy/Makefile |1 +
drivers/usb/phy/of.c | 47
structure.
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci13xxx_imx.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/drivers/usb/chipidea/ci13xxx_imx.c
b
.
Everything is sparse, smatch, checkpatch and coccinelle clean.
Alexander Shishkin (2):
usb: chipidea: get rid of camelcase names
usb: chipidea: drop 13xxx infix
Andy Shevchenko (2):
usb: chipidea: remove superfluous pci_set_drvdata(pci, NULL)
usb: chipidea: move to pcim_* functions
Fabio
' was not declared. Should it be static?
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/usbmisc_imx.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/usb/chipidea
-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci.h |1 +
drivers/usb/chipidea/core.c |1 -
drivers/usb/chipidea/udc.c | 192 +--
3 files changed, 111 insertions(+), 83 deletions(-)
diff --git a/drivers/usb
),
since chipidea core driver takes care of both host and device roles.
Signed-off-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/Kconfig |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
-by: Sascha Hauer s.ha...@pengutronix.de
Reviewed-by: Peter Chen peter.c...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci13xxx_imx.c | 19 +--
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/drivers
-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci13xxx_pci.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/usb/chipidea/ci13xxx_pci.c
b/drivers/usb/chipidea/ci13xxx_pci.c
index a2a6ac8..59fab90 100644
--- a/drivers/usb/chipidea/ci13xxx_pci.c
+++ b
Grzeschik m.grzesc...@pengutronix.de
Signed-off-by: Marc Kleine-Budde m...@pengutronix.de
Signed-off-by: Sascha Hauer s.ha...@pengutronix.de
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
.../devicetree/bindings/usb/ci13xxx-imx.txt|5 ++
drivers/usb/chipidea/bits.h
From: Philipp Zabel p.za...@pengutronix.de
This patch converts the driver to use the module_platform_driver
macro which makes the code smaller and a bit simpler.
Signed-off-by: Philipp Zabel p.za...@pengutronix.de
Signed-off-by: Michael Grzeschik m...@pengutronix.de
Signed-off-by: Alexander
From: Andy Shevchenko andriy.shevche...@linux.intel.com
This patch makes error path cleaner and probe function tidier.
Signed-off-by: Andy Shevchenko andriy.shevche...@linux.intel.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb/chipidea/ci13xxx_pci.c
From: Sascha Hauer s.ha...@pengutronix.de
This patch converts the driver to use devm_usb_get_phy_by_phandle
which makes the code smaller and a bit simpler.
Signed-off-by: Sascha Hauer s.ha...@pengutronix.de
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb
From: Fabio Estevam fabio.este...@freescale.com
There is no need to keep a local 'phy_np' as we can directly use the private
structure in data-phy_np.
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
Signed-off-by: Alexander Shishkin alexander.shish...@linux.intel.com
---
drivers/usb
Since someone has added camelcase detection to checkpatch.pl, chipidea
udc patches have been very noisy. To make everybody's life easier, this
patch changes camelcase names into something more appropriate to the
coding style. No functional changes.
Signed-off-by: Alexander Shishkin
Kerry Calvert kerry.calv...@micropower.com writes:
[adding linux-usb back to the loop]
Dude, what is with the spamming of this list? Cease and desist please
These things are called patches. This is how they are distributed in
linux kernel development process.
Regards,
--
Alex
--
To
Sergei Shtylyov sergei.shtyl...@cogentembedded.com writes:
Hello.
On 06/13/2013 06:59 PM, Alexander Shishkin wrote:
From: Michael Grzeschik m.grzesc...@pengutronix.de
The pdata structure gets copied anyway inside ci13xxx_add_device
by platform_device_add. We don't need to have it static
Hi,
Here are two fixes for v3.10-rc5, one of them is also applicable for
-stable.
Alexander Shishkin (2):
usb: chipidea: fix no transceiver case
usb: chipidea: fix id change handling
drivers/usb/chipidea/core.c |3 ++-
drivers/usb/chipidea/udc.c | 13 -
2 files changed
Re-enable chipidea irq even if there's no role changing to do. This is
a problem since b183c19f (USB: chipidea: re-order irq handling to avoid
unhandled irqs); when it manifests, chipidea irq gets disabled for good.
Cc: sta...@vger.kernel.org # v3.7
Signed-off-by: Alexander Shishkin
Andy Shevchenko andriy.shevche...@linux.intel.com writes:
This patch makes error path cleaner and probe function tidier.
...but does not apply to chipidea tree.
Signed-off-by: Andy Shevchenko andriy.shevche...@linux.intel.com
---
It assumes that removal of PCI_D0 and drvdata(NULL) were
Michael Grzeschik m...@pengutronix.de writes:
Hi,
On Mon, Jun 10, 2013 at 04:52:06PM +0300, Alexander Shishkin wrote:
Michael Grzeschik m...@pengutronix.de writes:
From: Michael Grzeschik m.grzesc...@pengutronix.de
This patch makes it possible to set the chipidea udc
into full-speed
Michael Grzeschik m...@pengutronix.de writes:
From: Michael Grzeschik m.grzesc...@pengutronix.de
This patch makes it possible to set the chipidea udc
into full-speed only mode. It can be set by the oftree
property force-full-speed.
Signed-off-by: Michael Grzeschik
Michael Grzeschik m...@pengutronix.de writes:
Hi Alexander,
On Fri, May 31, 2013 at 08:38:44PM +0200, Michael Grzeschik wrote:
From: Michael Grzeschik m.grzesc...@pengutronix.de
The pdata structure gets copied anyway inside ci13xxx_add_device
by platform_device_add. We don't need to have
Peter Chen hzpeterc...@gmail.com writes:
On Thu, May 9, 2013 at 3:51 AM, Randy Dunlap rdun...@infradead.org wrote:
On 05/08/13 03:02, Peter Chen wrote:
Randy Dunlap rdun...@infradead.org reported below problem
at i386:
drivers/built-in.o: In function `ci_hdrc_host_init':
(.text+0x2ce75c):
Michael Grzeschik m...@pengutronix.de writes:
Hey Alex,
On Mon, Jun 03, 2013 at 03:35:37PM +0300, Alexander Shishkin wrote:
Michael Grzeschik m...@pengutronix.de writes:
From: Michael Grzeschik m.grzesc...@pengutronix.de
This patch makes it possible to configure the PTW, PTS and STS
Peter Chen hzpeterc...@gmail.com writes:
On Mon, Jun 3, 2013 at 8:37 PM, Alexander Shishkin
alexander.shish...@linux.intel.com wrote:
Michael Grzeschik m...@pengutronix.de writes:
From: Sascha Hauer s.ha...@pengutronix.de
Even if a chipidea core is otg capable the board may
Chen Peter-B29397 b29...@freescale.com writes:
I did actually have this one in my tree, but since that time this
commit 7c8bfed7aaeba690de30835fe89882e1047a55fd
Author: David Rientjes rient...@google.com
Date: Fri Apr 26 13:25:01 2013 -0700
usb, chipidea: fix link error when
Peter Chen hzpeterc...@gmail.com writes:
On Tue, Jun 4, 2013 at 5:31 PM, Alexander Shishkin
alexander.shish...@linux.intel.com wrote:
Peter Chen hzpeterc...@gmail.com writes:
On Mon, Jun 3, 2013 at 8:37 PM, Alexander Shishkin
alexander.shish...@linux.intel.com wrote:
Michael Grzeschik m
1 - 100 of 357 matches
Mail list logo