the system: bugzilla.kernel.org/show_bug.cgi?id=76291
On Lynx Point-LP on the contrary switching ports to EHCI seems to be
necessary to fix these spurious interrupts.
Signed-off-by: Denis Turischev
Reported-by: Wulf Richartz
---
drivers/usb/host/xhci-pci.c | 4 +++-
1 file changed, 3 insertions(+),
Hi Mathias,
Just want to remind you about the patch, thanks.
Denis
On 03/21/2014 02:10 PM, Mathias Nyman wrote:
> On 03/21/2014 12:01 PM, Denis Turischev wrote:
>> The same issue like with Panther Point chipsets. If the USB ports are
>> switched to xHCI on shutdown, the xHCI h
an be avoided if the USB ports are switched back to EHCI on
shutdown.
Signed-off-by: Denis Turischev
---
drivers/usb/host/xhci-pci.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
index 3c898c1..9233d12 100644
--- a/drivers/usb/host
On 03/12/2014 12:08 PM, Fabio Estevam wrote:
Hi Denis,
Hi,
As you add me in the From field, you also need to add:
Signed-off-by: Fabio Estevam above your
Signed-off-by line.
Thanks.
+ usbphy {
+ #address-cells = <1>;
+ #size-cell
Signed-off-by: Denis Carikli
---
Changelog v3->v4:
- Moved the compatible of usbphy on top to match the other nodes.
- the usb phy's names were renamed from usbphy to usb-phy.
- The patch renaming the fsl,usbphy property in usb-phy was removed.
So this patch was adapted to that.
Chan
Signed-off-by: Denis Carikli
---
Changelog v2->v3:
- Extra gadget drivers additions were removed from this patch.
Changelog v1->v2:
- With the clock fix patches, the usb gadget also work.
So I've addeed it to this patch too.
- CONFIG_USB_OTG_FSM=y was not needed, so it was removed
Signed-off-by: Denis Carikli
---
Changelog v1->v2:
- With the clock fix patches, the usb gadget also work.
So I've set the otg port to otg instead of host.
- Before I forgott to set dr_mode to host in the usbhost port.
That is now fixed.
---
.../boot/dts/imx35-eukrea-mbimxsd35-baseb
This adds the i.MX25 and the i.MX35 support in the
ChipIdea usbmisc driver.
The i.MX25 and i.MX35 usb controllers are similar enough to be
able to use the same code.
Signed-off-by: Denis Carikli
---
Changelog v3->v4:
- The MXC_EHCI_INTERFACE_* were renamed in MX25_EHCI_INTERFACE_*
- Since
Signed-off-by: Denis Carikli
---
Changelog v1->v2:
- With the clock fix patches, the usb gadget also work.
So I've set the otg port to otg instead of host.
---
.../boot/dts/imx25-eukrea-mbimxsd25-baseboard.dts | 13 +
1 file changed, 13 insertions(+)
diff --git a/arch/
From: Fabio Estevam
Like other imx SoCs only one USB clock is needed on mx35.
Signed-off-by: Fabio Estevam
---
arch/arm/boot/dts/imx35.dtsi |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/imx35.dtsi b/arch/arm/boot/dts/imx35.dtsi
index e59ccb4..47
From: Fabio Estevam
Signed-off-by: Fabio Estevam
Signed-off-by: Denis Carikli
---
Changelog v3->v4:
- Added Fabio Estevam's Signed-off-by: it was given
as a mail response to this patch.
- Moved the compatible of usbphy on top to match the other nodes.
- the usb phy's names were
From: Fabio Estevam
Like other imx SoCs only one USB clock is needed on mx25.
Signed-off-by: Fabio Estevam
---
arch/arm/boot/dts/imx25.dtsi |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
index 77bb743..82
Signed-off-by: Denis Carikli
---
Changelog v2->v3:
- Extra gadget drivers additions were removed from this patch.
Changelog v1->v2:
- With the clock fix patches, the usb gadget also work.
So I've addeed it to this patch too.
- CONFIG_USB_OTG_FSM=y was not needed, so it was removed
Signed-off-by: Denis Carikli
---
Changelog v1->v2:
- With the clock fix patches, the usb gadget also work.
So I've set the otg port to otg instead of host.
- Before I forgott to set dr_mode to host in the usbhost port.
That is now fixed.
---
.../boot/dts/imx35-eukrea-mbimxsd35-baseb
Signed-off-by: Denis Carikli
---
Changelog v2->v3:
- rebased on top of the "usb: chipidea: Use standard usb-phy property." patch.
- Fixed the usbphy nodes index and added and added a reg property.
Changelog v1->v2:
- The usbphy nodes were made to look like the ones in imx53.dtsi
This adds the i.MX25 and the i.MX35 support in the
ChipIdea usbmisc driver.
The i.MX25 and i.MX35 usb controllers are similar enough to be
able to use the same code.
Signed-off-by: Denis Carikli
---
Changelog v2->v3:
- Add a commit log
Changelog v1->v2:
- converted two remaining defines
From: Fabio Estevam
This patch was adapted from the thread named
"USB Host support for mx25" on linux-usb@vger.kernel.org
Signed-off-by: Denis Carikli
---
Changelog v2->v3:
- rebased on top of the "usb: chipidea: Use standard usb-phy property." patch.
- Fixed the usbph
From: Fabio Estevam
Like other imx SoCs only one USB clock is needed on mx35.
Signed-off-by: Fabio Estevam
---
arch/arm/boot/dts/imx35.dtsi |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/imx35.dtsi b/arch/arm/boot/dts/imx35.dtsi
index e59ccb4..47
Signed-off-by: Denis Carikli
---
Changelog v1->v2:
- With the clock fix patches, the usb gadget also work.
So I've set the otg port to otg instead of host.
---
.../boot/dts/imx25-eukrea-mbimxsd25-baseboard.dts | 13 +
1 file changed, 13 insertions(+)
diff --git a/arch/
This converts the Chipidea usbmisc driver to
use the standard usb-phy property.
It also adapt the dts that uses it.
Signed-off-by: Denis Carikli
---
arch/arm/boot/dts/imx23.dtsi |2 +-
arch/arm/boot/dts/imx27.dtsi |4 ++--
arch/arm/boot/dts/imx28.dtsi |4
From: Fabio Estevam
Like other imx SoCs only one USB clock is needed on mx25.
Signed-off-by: Fabio Estevam
---
arch/arm/boot/dts/imx25.dtsi |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
index 77bb743..82
Signed-off-by: Denis Carikli
---
Changelog v1->v2:
- With the clock fix patches, the usb gadget also work.
So I've set the otg port to otg instead of host.
- Before I forgott to set dr_mode to host in the usbhost port.
That is now fixed.
---
.../boot/dts/imx35-eukrea-mbimxsd35-baseb
Signed-off-by: Denis Carikli
---
Changelog v1->v2:
- converted two remaining defines to BIT()
- Removed a variable declaration that was not used in usbmisc_imx25_init
---
drivers/usb/chipidea/usbmisc_imx.c | 58
1 file changed, 58 insertions(+)
diff --
From: Fabio Estevam
Like other imx SoCs only one USB clock is needed on mx35.
Signed-off-by: Fabio Estevam
---
arch/arm/boot/dts/imx35.dtsi |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/imx35.dtsi b/arch/arm/boot/dts/imx35.dtsi
index e59ccb4..47
Signed-off-by: Denis Carikli
---
Changelog v1->v2:
- The usbphy nodes were made to look like the ones in imx53.dtsi
- The patch was rebased on top of the clock fixes commits.
---
arch/arm/boot/dts/imx35.dtsi | 16
1 file changed, 16 insertions(+)
diff --git a/arch/arm/b
Signed-off-by: Denis Carikli
---
Changelog v1->v2:
- With the clock fix patches, the usb gadget also work.
So I've set the otg port to otg instead of host.
---
.../boot/dts/imx25-eukrea-mbimxsd25-baseboard.dts | 13 +
1 file changed, 13 insertions(+)
diff --git a/arch/
From: Fabio Estevam
Like other imx SoCs only one USB clock is needed on mx25.
Signed-off-by: Fabio Estevam
---
arch/arm/boot/dts/imx25.dtsi |6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
index 77bb743..82
Signed-off-by: Denis Carikli
---
Changelog v1->v2:
- With the clock fix patches, the usb gadget also work.
So I've addeed it to this patch too.
- CONFIG_USB_OTG_FSM=y was not needed, so it was removed.
---
arch/arm/configs/imx_v4_v5_defconfig |9 +
1 file changed, 9 in
From: Fabio Estevam
This patch was adapted from the thread named
"USB Host support for mx25" on linux-usb@vger.kernel.org
Signed-off-by: Denis Carikli
---
Changelog v1->v2:
- The usbphy nodes were made to look like the ones in imx53.dtsi
- The patch was rebased on top of th
an be avoided if the USB ports are switched back to EHCI on
shutdown.
Signed-off-by: Denis Turischev
---
drivers/usb/host/xhci-pci.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
index 3c898c1..9233d12 100644
--- a/drivers/usb/host
Signed-off-by: Denis Carikli
---
arch/arm/configs/imx_v4_v5_defconfig |8
1 file changed, 8 insertions(+)
diff --git a/arch/arm/configs/imx_v4_v5_defconfig
b/arch/arm/configs/imx_v4_v5_defconfig
index f1aeb7d..f7c4fbf 100644
--- a/arch/arm/configs/imx_v4_v5_defconfig
+++ b/arch
Signed-off-by: Denis Carikli
---
arch/arm/configs/imx_v6_v7_defconfig |2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/configs/imx_v6_v7_defconfig
b/arch/arm/configs/imx_v6_v7_defconfig
index 09e9743..772f14f 100644
--- a/arch/arm/configs/imx_v6_v7_defconfig
+++ b/arch/arm
Signed-off-by: Denis Carikli
---
.../boot/dts/imx35-eukrea-mbimxsd35-baseboard.dts | 12
1 file changed, 12 insertions(+)
diff --git a/arch/arm/boot/dts/imx35-eukrea-mbimxsd35-baseboard.dts
b/arch/arm/boot/dts/imx35-eukrea-mbimxsd35-baseboard.dts
index 71197b9..d27f02e 100644
Signed-off-by: Denis Carikli
---
arch/arm/boot/dts/imx35.dtsi | 11 +++
1 file changed, 11 insertions(+)
diff --git a/arch/arm/boot/dts/imx35.dtsi b/arch/arm/boot/dts/imx35.dtsi
index e59ccb4..1c15b56 100644
--- a/arch/arm/boot/dts/imx35.dtsi
+++ b/arch/arm/boot/dts/imx35.dtsi
Signed-off-by: Denis Carikli
---
.../boot/dts/imx25-eukrea-mbimxsd25-baseboard.dts | 13 +
1 file changed, 13 insertions(+)
diff --git a/arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dts
b/arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dts
index 62fb3da..58c606f 100644
Signed-off-by: Denis Carikli
---
arch/arm/boot/dts/imx51.dtsi |8
1 file changed, 8 insertions(+)
diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi
index e508e6f..917b6ed 100644
--- a/arch/arm/boot/dts/imx51.dtsi
+++ b/arch/arm/boot/dts/imx51.dtsi
@@ -100,6
From: Fabio Estevam
This patch was adapted from the thread named
"USB Host support for mx25" on linux-usb@vger.kernel.org
Signed-off-by: Denis Carikli
---
arch/arm/boot/dts/imx25.dtsi |9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/arch/arm/boot/dts/
Signed-off-by: Denis Carikli
---
.../boot/dts/imx51-eukrea-mbimxsd51-baseboard.dts | 47
1 file changed, 47 insertions(+)
diff --git a/arch/arm/boot/dts/imx51-eukrea-mbimxsd51-baseboard.dts
b/arch/arm/boot/dts/imx51-eukrea-mbimxsd51-baseboard.dts
index 5cec4f3..4d970b3
Signed-off-by: Denis Carikli
---
drivers/usb/chipidea/usbmisc_imx.c | 59
1 file changed, 59 insertions(+)
diff --git a/drivers/usb/chipidea/usbmisc_imx.c
b/drivers/usb/chipidea/usbmisc_imx.c
index cd061ab..d956ad0 100644
--- a/drivers/usb/chipidea
an be avoided if the USB ports are switched back to EHCI on
shutdown.
Signed-off-by: Denis Turischev
---
drivers/usb/host/xhci-pci.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
index 3c898c1..9233d12 100644
--- a/drivers/usb/host
, so I got the same error.
Denis.
--
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
an be avoided if the USB ports are switched back to EHCI on
shutdown.
Signed-off-by: Denis Turischev
---
drivers/usb/host/xhci-pci.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
index 3c898c1..9233d12 100644
--- a/drivers/usb/host
an be avoided if the USB ports are switched back to EHCI on
shutdown.
Signed-off-by: Denis Turischev
---
drivers/usb/host/xhci-pci.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
index 3c898c1..9233d12 100644
--- a/drivers/usb/host
Hi,
Not sure if it's been posted already. I'm running the latest Linus tree.
[ 10.636228] INFO: trying to register non-static key.
[ 10.636782] the code is fine but needs lockdep annotation.
[ 10.637271] turning off the locking correctness validator.
[ 10.637772] CPU: 0 PID: 1216 Comm: sy
On 01/08/2014 01:11 AM, Sarah Sharp wrote:
> Denis, what do you mean by "works for Lynx Point"? Do you mean that
> adding the quirk to switch the ports on EHCI on shutdown (e95829f474)
> for the Intense-PC2 *instead of* the commit to put the host in D3 on
> shutdown (638298d
Hi Sarah,
On 01/03/2014 02:03 AM, Sarah Sharp wrote:
> Denis, do all of Compulab's Haswell systems reboot on shutdown? Are
> they all running a Phoenix BIOS? Can you send me the output of `sudo
> lspci -vvv -s` for the xHCI host?
oem@oem-Intense-PC2 ~ $ sudo lspci -vvv -s 00:14
On 12/21/2013 01:45 AM, Sarah Sharp wrote:
> On Fri, Dec 20, 2013 at 12:41:11PM +0200, Denis Turischev wrote:
>>> Also, which kernel are you experiencing this issue on? In 3.12, I
>>> queued a separate patch to deal with spurious reboot issues on Lynx
>
i
> Signed-off-by: Sarah Sharp
> Cc: sta...@vger.kernel.org
>
> This patch is in 3.12, but a patch to narrow the quirk to only apply HP
> systems
> will hit 3.13 shortly:
Sorry, I indeed tested not on the latest kernel version, Ubuntu 3.13-rc3 has
this
shutdown.
v1: add new device id locally, not in
Signed-off-by: Denis Turischev
diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
--- a/drivers/usb/host/xhci-pci.c 2013-12-19 11:36:12.049589400 +0200
+++ b/drivers/usb/host/xhci-pci.c 2013-12-19 11:37:27.261590385
shutdown.
Signed-off-by: Denis Turischev
diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
--- a/drivers/usb/host/xhci-pci.c 2013-12-19 11:36:12.049589400 +0200
+++ b/drivers/usb/host/xhci-pci.c 2013-12-19 11:37:27.261590385 +0200
@@ -91,8 +91,9
On 6/24/13 6:35 PM, Felipe Balbi wrote:
Hi,
On Mon, Jun 24, 2013 at 06:31:07PM +0400, Denis V. Lunev wrote:
with a length that isn't multiply by max pkt size for this endpoint.
The commit 689d6eac (USB: UHCI: add native scatter-gather support(v1))
introduced an implementation of sc
D in sg element.
Signed-off-by: Konstantin Filatov
Signed-off-by: Denis V. Lunev
Acked-by: Alan Stern
CC: Ming Lei
CC: Sebastian Andrzej Siewior
CC: linux-usb@vger.kernel.org
---
Changes from v3:
- fixed subject
Changes from v2:
- changed subject as suggested by Alan Stern
- edited description
-by: Konstantin Filatov
Signed-off-by: Denis V. Lunev
Acked-by: Alan Stern
CC: Ming Lei
CC: Sebastian Andrzej Siewior
CC: linux-usb@vger.kernel.org
---
Changes from v2:
- changed subject as suggested by Alan Stern
- edited description
- shortened the referenced commit descriptions
Changes from v1
On 6/21/13 7:19 PM, Alan Stern wrote:
On Fri, 21 Jun 2013, Konstantin Filatov wrote:
Your patch skips a chunk of data to transfer. This is a corruption of
data. It's unacceptable. We can return an error code if we treat the
request as malformed, but we can't transfer data selectively.
I agree
t only for the last TD in sg list,
but also for the last TD in sg element.
Signed-off-by: Konstantin Filatov
Signed-off-by: Denis V. Lunev
CC: Alan Stern
CC: linux-usb@vger.kernel.org
---
drivers/usb/host/uhci-q.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/usb/host/uhci-q.c
but I don't recall Nokia opensourcing any of these before
Burning The Platforms. For all practical purposes, that code is lost.
--
Rémi Denis-Courmont
http://www.remlab.net/
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to maj
imited knowledge, the code was only ever used for Nokia
N900, N950 and N9 hardware, with obsolescent kernel versions. I would not
know if Renesas has any use for this.
--
Rémi Denis-Courmont
Sent from my collocated server
--
To unsubscribe from this list: send the line "unsubscribe linux-
The symbol usb_bind_phy is exported and annotated __init.
It looks like section mismatch. Fix by removing
the __init annotation of usb_bind_phy.
Found by Linux Driver Verification project (linuxtesting.org).
Signed-off-by: Denis Efremov
---
drivers/usb/otg/otg.c | 2 +-
1 file changed, 1
On 09.08.2012 13:37, Felipe Balbi wrote:
Hi,
I have pushed a patch which I think solves this issue. Can you test ?
Hi,
my patch is unneeded since this is a threaded interrupt handler. And it
runs with
interrupt line masked globally on the controller(because of IRQF_ONESHOT).
And I think that
Hi Sarah,
Yes, it fixes unexpected wake-up on Intense-PC.
Thanks!
Denis.
On 08/07/2012 08:39 PM, Sarah Sharp wrote:
> The Intel desktop boards DH77EB and DH77DF have a hardware issue that
> can be worked around by BIOS. If the USB ports are switched to xHCI on
> shutdown, the xHCI
be 0 then the call is harmless, if it
> is not 0 then we free the memory we allocated but don't need.
>
> Signed-off-by: Jesper Juhl
Acked-by: Rémi Denis-Courmont
> ---
> drivers/net/usb/cdc-phonet.c | 1 +
> 1 file changed, 1 insertion(+)
>
> Only compile tested
Hi Sarah,
Attached dmidecode output of Intense-PC.
Denis Turischev
On 07/31/2012 01:34 AM, Sarah Sharp wrote:
> Hi Denis,
>
> Can you send me the output of `sudo dmidecode`? I'd like to see if I
> can make a more general patch apply to the Intense-PC.
>
> Sarah Sha
Sarah Sharp
>
> On Mon, Jul 23, 2012 at 02:46:32PM +0300, Denis Turischev wrote:
>> Intense-PC is Compulab's mini-desktop with Intel Panther Point
>> chipset.
>>
>> Unconditional switchover to xHCI provided by function
>> usb_enable_xhci_ports() leads to sur
Some devices with Intel Panther Point chipset may require switchover
usb ports from XHCI back to EHCI controller before shutdown.
Otherwise various BIOS bugs related to power management may be
triggered.
v2: deleted unnecessary variables.
Signed-off-by: Denis Turischev
---
drivers/usb/host/pci
Some devices with Intel Panther Point chipset may require switchover
usb ports from XHCI back to EHCI controller before shutdown.
Overwise various BIOS bugs related to power management may be
triggered.
Signed-off-by: Denis Turischev
---
drivers/usb/host/pci-quirks.c | 14
erved. Switching back to EHCI solves
the problem.
The patch switches usb ports back to EHCI during xhci shutdown
for Intense-PC.
Signed-off-by: Denis Turischev
---
drivers/usb/host/xhci.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/x
Linux Driver Verification project (linuxtesting.org).
Signed-off-by: Denis Efremov
---
drivers/usb/otg/twl4030-usb.c |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/usb/otg/twl4030-usb.c b/drivers/usb/otg/twl4030-usb.c
index c4a86da..c89a2f5 100644
--- a/drivers
On 07/17/2012 07:54 AM, Sarah Sharp wrote:
> On Mon, Jul 16, 2012 at 07:46:06PM +0300, Denis Turischev wrote:
>> Intense-PC is Compulab's mini-desktop with Intel Panther Point
>> chipset.
>>
>> Unconditional ports switching provided by function
>> usb_e
The patch avoids ports switching for Intense-PC.
Signed-off-by: Denis Turischev
---
drivers/usb/host/pci-quirks.c |7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/usb/host/pci-quirks.c b/drivers/usb/host/pci-quirks.c
index df0828c..6f72593 100644
--- a/drivers/usb/host/pci-qui
separately.
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]>
---
drivers/usb/atm/Kconfig|9 -
drivers/usb/gadget/Kconfig |9 +
drivers/usb/serial/Kconfig | 11 ---
3 files changed, 13 insertions(+), 16 deletions(-)
diff --git a/drivers/usb/atm/Kconfig b/drivers/u
70 matches
Mail list logo
