[U-Boot] [PATCH] usb: ehci-omap: Add Support DM_USB and OF_CONTROL

Adam Ford aford173 at gmail.com
Mon Jul 29 12:16:47 UTC 2019


On Sun, Jun 30, 2019 at 5:21 PM Adam Ford <aford173 at gmail.com> wrote:
>
> The omap3.dtsi file shows the usbhshost node with two sub-nodes
> for ohci and ehci.  This patch file creates the usbhshost, and
> pulls the portX-mode information.  It then locates the EHCI
> sub-node, and initializes the EHCI controller with the info
> pulled from the usbhshost node.
>
> There is still more to do since there isn't an actual link
> between the 'phys' reference and the corresponding phy driver,
> and there is no nop-xceiv driver yet.
>
> In the meantime, the older style reference to
> CONFIG_OMAP_EHCI_PHYx_RESET_GPIO is still needed to pull
> the phy out of reset until the phy driver is completed and the
> phandle reference is made.
>
> Signed-off-by: Adam Ford <aford173 at gmail.com>
>

Marek,

Some subsequent patches that were depending on this have been applied
already, but they cannot be really useful until this patch is applied.
Is there any chance it can be applied?  It will give some of the OMAP3
boards the USB EHCI functionality which has been lost for a little
while.

thank you,

adam
> diff --git a/arch/arm/include/asm/ehci-omap.h b/arch/arm/include/asm/ehci-omap.h
> index 1549f7bf21..f970bba937 100644
> --- a/arch/arm/include/asm/ehci-omap.h
> +++ b/arch/arm/include/asm/ehci-omap.h
> @@ -123,6 +123,7 @@ struct omap_ehci {
>         u32 insreg08;           /* 0xb0 */
>  };
>
> +#if !CONFIG_IS_ENABLED(DM_USB) || !CONFIG_IS_ENABLED(OF_CONTROL)
>  /*
>   * FIXME: forward declaration of this structs needed because omap got the
>   * ehci implementation backwards. move out ehci_hcd_x from board files
> @@ -133,5 +134,6 @@ struct ehci_hcor;
>  int omap_ehci_hcd_init(int index, struct omap_usbhs_board_data *usbhs_pdata,
>                        struct ehci_hccr **hccr, struct ehci_hcor **hcor);
>  int omap_ehci_hcd_stop(void);
> +#endif
>
>  #endif /* _OMAP_COMMON_EHCI_H_ */
> diff --git a/drivers/usb/host/ehci-omap.c b/drivers/usb/host/ehci-omap.c
> index ab459c8cc9..bf83a4adb3 100644
> --- a/drivers/usb/host/ehci-omap.c
> +++ b/drivers/usb/host/ehci-omap.c
> @@ -9,7 +9,6 @@
>   *     Shashi Ranjan <shashiranjanmca05 at gmail.com>
>   *
>   */
> -
>  #include <common.h>
>  #include <usb.h>
>  #include <usb/ulpi.h>
> @@ -18,6 +17,10 @@
>  #include <asm/gpio.h>
>  #include <asm/arch/ehci.h>
>  #include <asm/ehci-omap.h>
> +#include <dm.h>
> +#include <dm/device-internal.h>
> +#include <dm/lists.h>
> +#include <power/regulator.h>
>
>  #include "ehci.h"
>
> @@ -177,8 +180,13 @@ int omap_ehci_hcd_stop(void)
>   * Based on "drivers/usb/host/ehci-omap.c" from Linux 3.1
>   * See there for additional Copyrights.
>   */
> +#if !CONFIG_IS_ENABLED(DM_USB) || !CONFIG_IS_ENABLED(OF_CONTROL)
> +
>  int omap_ehci_hcd_init(int index, struct omap_usbhs_board_data *usbhs_pdata,
>                        struct ehci_hccr **hccr, struct ehci_hcor **hcor)
> +#else
> +int omap_ehci_hcd_init(int index, struct omap_usbhs_board_data *usbhs_pdata)
> +#endif
>  {
>         int ret;
>         unsigned int i, reg = 0, rev = 0;
> @@ -285,10 +293,130 @@ int omap_ehci_hcd_init(int index, struct omap_usbhs_board_data *usbhs_pdata,
>         for (i = 0; i < OMAP_HS_USB_PORTS; i++)
>                 if (is_ehci_phy_mode(usbhs_pdata->port_mode[i]))
>                         omap_ehci_soft_phy_reset(i);
> -
> +#if !CONFIG_IS_ENABLED(DM_USB) || !CONFIG_IS_ENABLED(OF_CONTROL)
>         *hccr = (struct ehci_hccr *)(OMAP_EHCI_BASE);
>         *hcor = (struct ehci_hcor *)(OMAP_EHCI_BASE + 0x10);
> +#endif
>
>         debug("OMAP EHCI init done\n");
>         return 0;
>  }
> +
> +#if CONFIG_IS_ENABLED(DM_USB)
> +
> +struct ehci_omap_priv_data {
> +       struct ehci_ctrl ctrl;
> +       struct omap_ehci *ehci;
> +#ifdef CONFIG_DM_REGULATOR
> +       struct udevice *vbus_supply;
> +#endif
> +       enum usb_init_type init_type;
> +       int portnr;
> +       struct phy phy[OMAP_HS_USB_PORTS];
> +       int nports;
> +};
> +
> +static struct omap_usbhs_board_data usbhs_bdata = {
> +       .port_mode[0] = OMAP_USBHS_PORT_MODE_UNUSED,
> +       .port_mode[1] = OMAP_USBHS_PORT_MODE_UNUSED,
> +       .port_mode[2] = OMAP_USBHS_PORT_MODE_UNUSED,
> +};
> +
> +/* Match this order enums usbhs_omap_port_mode */
> +static char *phy_strings[] = {
> +       "",
> +       "ehci-phy",
> +       "ehci-tll",
> +       "ehci-hsic",
> +};
> +
> +static int find_phy_mode(const char *phy_type)
> +{
> +       int i;
> +
> +       for (i = 0; i < OMAP_HS_USB_PORTS; i++) {
> +               if (!strcmp(phy_type, phy_strings[i]))
> +                       return i;
> +       }
> +       return 0;
> +}
> +
> +static const struct ehci_ops omap_ehci_ops = {
> +};
> +
> +static int ehci_usb_probe(struct udevice *dev)
> +{
> +       struct usb_platdata *plat = dev_get_platdata(dev);
> +       const void *fdt = gd->fdt_blob;
> +       struct omap_ehci *ehci;
> +       struct ehci_omap_priv_data *priv = dev_get_priv(dev);
> +       struct ehci_hccr *hccr;
> +       struct ehci_hcor *hcor;
> +       ofnode node;
> +       ofnode parent_node = dev->node;
> +       int ret = -1;
> +       int off, i;
> +       const char *mode;
> +       char prop[11];
> +
> +       priv->ehci = (struct omap_ehci *)devfdt_get_addr(dev);
> +       priv->portnr = dev->seq;
> +       priv->init_type = plat->init_type;
> +
> +       /* Go through each port portX-mode to determing phy mode */
> +       for (i = 0; i < 3; i++) {
> +               snprintf(prop, sizeof(prop), "port%d-mode", i + 1);
> +               mode = dev_read_string(dev, prop);
> +               if (mode)
> +                       usbhs_bdata.port_mode[i] = find_phy_mode(mode);
> +       }
> +
> +       ofnode_for_each_subnode(node, parent_node) {
> +               /* Locate node for ti,ehci-omap */
> +               off = ofnode_to_offset(node);
> +               ret = fdt_node_check_compatible(fdt, off, "ti,ehci-omap");
> +
> +               if (!ret) {
> +                       /* If EHCI node is found, set the pointer to it */
> +                       ehci = (struct omap_ehci *)ofnode_get_addr(node);
> +                       hccr = (struct ehci_hccr *)&ehci->hccapbase;
> +                       hcor = (struct ehci_hcor *)&ehci->usbcmd;
> +
> +                       /* Do the actual EHCI initialization */
> +                       omap_ehci_hcd_init(0, &usbhs_bdata);
> +                       ret = ehci_register(dev, hccr, hcor, &omap_ehci_ops, 0,
> +                                           USB_INIT_HOST);
> +               }
> +       }
> +
> +       return ret;
> +}
> +
> +static const struct udevice_id omap_ehci_dt_ids[] = {
> +       { .compatible = "ti,usbhs-host" },
> +       { }
> +};
> +
> +static int ehci_usb_ofdata_to_platdata(struct udevice *dev)
> +{
> +       struct usb_platdata *plat = dev_get_platdata(dev);
> +
> +       plat->init_type = USB_INIT_HOST;
> +
> +       return 0;
> +}
> +
> +U_BOOT_DRIVER(usb_omaphs_host) = {
> +       .name   = "usbhs-host",
> +       .id     = UCLASS_USB,
> +       .of_match = omap_ehci_dt_ids,
> +       .ofdata_to_platdata = ehci_usb_ofdata_to_platdata,
> +       .probe  = ehci_usb_probe,
> +       .remove = ehci_deregister,
> +       .ops    = &ehci_usb_ops,
> +       .platdata_auto_alloc_size = sizeof(struct usb_platdata),
> +       .priv_auto_alloc_size = sizeof(struct ehci_omap_priv_data),
> +       .flags  = DM_FLAG_ALLOC_PRIV_DMA,
> +};
> +
> +#endif
> diff --git a/include/configs/omap3_logic.h b/include/configs/omap3_logic.h
> index e318a9f896..65d8487c18 100644
> --- a/include/configs/omap3_logic.h
> +++ b/include/configs/omap3_logic.h
> @@ -30,6 +30,8 @@
>  /* I2C */
>  #define CONFIG_SYS_I2C_EEPROM_ADDR     0x50    /* EEPROM AT24C64      */
>
> +#define CONFIG_OMAP_EHCI_PHY1_RESET_GPIO       4
> +
>  /* Board NAND Info. */
>  #ifdef CONFIG_NAND
>  #define CONFIG_SYS_MAX_NAND_DEVICE     1         /* Max number of */
> --
> 2.17.1
>


More information about the U-Boot mailing list