[U-Boot] [PATCH 0/3] PPC440EPX(sequoia) USB EHCI support.

This patch adds echi support for ppc440epx(sequoia board). Also a fix for ECHI host controller port reset.
Chris
Chris Zhang (3): Fix EHCI port reset. Add ppc440epx USB ehci support. Adds EHCI definitions to sequoia board configuration file.
drivers/usb/host/Makefile | 1 + drivers/usb/host/ehci-hcd.c | 3 ++ drivers/usb/host/ehci-ppc4xx.c | 48 ++++++++++++++++++++++++++++++++++++++++ include/configs/sequoia.h | 16 ++++++++++++- 4 files changed, 67 insertions(+), 1 deletions(-) create mode 100644 drivers/usb/host/ehci-ppc4xx.c

In USB ehci driver, the port reset is not terminated. EHCI spec says "A host controller must terminate the reset and stabilize the state of the port within 2 milliseconds". Without termination, a port stays at reset state. This is observed on ppc4xx(sequoia) boards.
Signed-off-by: Chris Zhang chris@seamicro.com --- drivers/usb/host/ehci-hcd.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c index ba85991..9ebeb4f 100644 --- a/drivers/usb/host/ehci-hcd.c +++ b/drivers/usb/host/ehci-hcd.c @@ -708,6 +708,9 @@ ehci_submit_root(struct usb_device *dev, unsigned long pipe, void *buffer, * root */ wait_ms(50); + /* terminate the reset */ + ehci_writel(status_reg, reg & ~EHCI_PS_PR); + wait_ms(2); portreset |= 1 << le16_to_cpu(req->index); } break;

Hi,
2010/1/6 Chris Zhang chris@seamicro.com:
In USB ehci driver, the port reset is not terminated. EHCI spec says "A host controller must terminate the reset and stabilize the state of the port within 2 milliseconds". Without termination, a port stays at reset state. This is observed on ppc4xx(sequoia) boards.
Signed-off-by: Chris Zhang chris@seamicro.com
Applied to u-boot-usb Thanks.
Remy

Currently ppc440epx uses OHCI for USB full-speed support. This change adds support for EHCI.
Signed-off-by: Chris Zhang chris@seamicro.com --- drivers/usb/host/Makefile | 1 + drivers/usb/host/ehci-ppc4xx.c | 48 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 49 insertions(+), 0 deletions(-) create mode 100644 drivers/usb/host/ehci-ppc4xx.c
diff --git a/drivers/usb/host/Makefile b/drivers/usb/host/Makefile index 940d4a8..255679a 100644 --- a/drivers/usb/host/Makefile +++ b/drivers/usb/host/Makefile @@ -36,6 +36,7 @@ COBJS-$(CONFIG_USB_SL811HS) += sl811-hcd.o # echi COBJS-$(CONFIG_USB_EHCI) += ehci-hcd.o COBJS-$(CONFIG_USB_EHCI_FSL) += ehci-fsl.o +COBJS-$(CONFIG_USB_EHCI_PPC4XX) += ehci-ppc4xx.o COBJS-$(CONFIG_USB_EHCI_IXP4XX) += ehci-ixp.o COBJS-$(CONFIG_USB_EHCI_KIRKWOOD) += ehci-kirkwood.o COBJS-$(CONFIG_USB_EHCI_PCI) += ehci-pci.o diff --git a/drivers/usb/host/ehci-ppc4xx.c b/drivers/usb/host/ehci-ppc4xx.c new file mode 100644 index 0000000..966a629 --- /dev/null +++ b/drivers/usb/host/ehci-ppc4xx.c @@ -0,0 +1,48 @@ +/* + * (C) Copyright 2010, Chris Zhang chris@seamicro.com + * + * Author: Chris Zhang chris@seamicro.com + * This code is based on ehci freescale driver + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of + * the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, + * MA 02111-1307 USA + */ +#include <common.h> +#include <usb.h> + +#include "ehci.h" +#include "ehci-core.h" + +/* + * Create the appropriate control structures to manage + * a new EHCI host controller. + */ +int ehci_hcd_init(void) +{ + hccr = (struct ehci_hccr *)(CONFIG_SYS_PPC4XX_USB_ADDR); + hcor = (struct ehci_hcor *)((uint32_t) hccr + + HC_LENGTH(ehci_readl(&hccr->cr_capbase))); + usb_dev_init(); + return 0; +} + +/* + * Destroy the appropriate control structures corresponding + * the the EHCI host controller. + */ +int ehci_hcd_stop(void) +{ + return 0; +}

Hi,
2010/1/6 Chris Zhang chris@seamicro.com:
Currently ppc440epx uses OHCI for USB full-speed support. This change adds support for EHCI.
Signed-off-by: Chris Zhang chris@seamicro.com
Applied to u-boot-usb Thanks.
Remy

Adds required definitions for EHCI support in sequoia configuration file. But still keeps the OHCI as default driver.
Signed-off-by: Chris Zhang chris@seamicro.com --- include/configs/sequoia.h | 16 +++++++++++++++- 1 files changed, 15 insertions(+), 1 deletions(-)
diff --git a/include/configs/sequoia.h b/include/configs/sequoia.h index 5788d58..568d9fc 100644 --- a/include/configs/sequoia.h +++ b/include/configs/sequoia.h @@ -282,8 +282,20 @@
/* USB */ #ifdef CONFIG_440EPX + +#undef CONFIG_USB_EHCI /* OHCI by default */ + +#ifdef CONFIG_USB_EHCI +#define CONFIG_USB_EHCI_PPC4XX +#define CONFIG_SYS_PPC4XX_USB_ADDR 0xe0000300 +#define CONFIG_EHCI_HCD_INIT_AFTER_RESET +#define CONFIG_EHCI_MMIO_BIG_ENDIAN +#define CONFIG_EHCI_DESC_BIG_ENDIAN +#ifdef CONFIG_4xx_DCACHE +#define CONFIG_EHCI_DCACHE +#endif +#else /* CONFIG_USB_EHCI */ #define CONFIG_USB_OHCI_NEW -#define CONFIG_USB_STORAGE #define CONFIG_SYS_OHCI_BE_CONTROLLER
#undef CONFIG_SYS_USB_OHCI_BOARD_INIT @@ -291,7 +303,9 @@ #define CONFIG_SYS_USB_OHCI_REGS_BASE CONFIG_SYS_USB_HOST #define CONFIG_SYS_USB_OHCI_SLOT_NAME "ppc440" #define CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS 15 +#endif
+#define CONFIG_USB_STORAGE /* Comment this out to enable USB 1.1 device */ #define USB_2_0_DEVICE

Hi,
2010/1/6 Chris Zhang chris@seamicro.com:
Adds required definitions for EHCI support in sequoia configuration file. But still keeps the OHCI as default driver.
Signed-off-by: Chris Zhang chris@seamicro.com
Applied to u-boot-usb Thanks.
Remy
participants (2)
-
Chris Zhang
-
Remy Bohmer