[U-Boot] [RFC] usb: musb-new: omap2430: Enable DM_USB and OF support

[Lukasz Majewski <lukma@denx.de>]

Hi Adam,

> On Sun, Jul 22, 2018 at 11:45 AM Adam Ford <aford173@gmail.com> wrote:
> >
> > With upcoming changes that require CONFIG_BLK, this broke
> > USB Mass Storage on the OMAP3 boards because if CONFIG_BLK is
> > enabled, it assumes that DM_USB is enabled, but it wasn't yet
> > available on omap3 and omap4 boards.
> >
> > This patch converts the OMAP2430 MUSB glue to support DM_USB and
> > extracts the necessary information based on the device tree.
> >
> > It's based on the ti-musb driver, but there are enough significant
> > differences in both the architecture and device tree entires between
> > am33xx and OMAP3/OMAP4, that I think it makes sense to continue to
> > keep the separate.
> >  
> 
> I forgot to mention that, I based this patch on top of an existing
> patch, that hasn't yet been reviewed which strips out a bunch of dead
> code
>  https://patchwork.ozlabs.org/patch/943813/
> 
> I did it to make it easier for me to read.
> 
> > This RFC has functional USB Host working, but it's hard coded to be
> > host or peripheral based on the config file.  Unfortunately,
> > USB Peripherals like fastboot, etc don't appear to be
> > working at all.  At least on omap3_logic (DM3730/OMAP35xx).
> > Any suggestions are appreciated in how to setup the MUSB in OTG mode
> > and get it to start USB in either host or gadget mode.

Has it been working from the outset? Is the old musb driver working
correctly with gadget infrastructure?

> >
> > Unless an OTG adapter is connected with a USB device, the only
> > message I get when issing 'usb start' is
> >
> >         USB0:   Port not available.

I assume that without your patches it works? It was not clear from your
commit message.

> >
> > Signed-off-by: Adam Ford <aford173@gmail.com>
> >
> > diff --git a/drivers/usb/musb-new/omap2430.c
> > b/drivers/usb/musb-new/omap2430.c index fd63c07789..8b56e1a42a
> > 100644 --- a/drivers/usb/musb-new/omap2430.c
> > +++ b/drivers/usb/musb-new/omap2430.c
> > @@ -9,14 +9,18 @@
> >   * This file is part of the Inventra Controller Driver for Linux.
> >   */
> >  #include <common.h>
> > +#include <dm.h>
> > +#include <dm/device-internal.h>
> > +#include <dm/lists.h>
> > +#include <linux/usb/otg.h>
> >  #include <asm/omap_common.h>
> >  #include <asm/omap_musb.h>
> >  #include <twl4030.h>
> >  #include <twl6030.h>
> >  #include "linux-compat.h"
> > -
> >  #include "musb_core.h"
> >  #include "omap2430.h"
> > +#include "musb_uboot.h"
> >
> >  static inline void omap2430_low_level_exit(struct musb *musb)
> >  {
> > @@ -43,6 +47,7 @@ static int omap2430_musb_init(struct musb *musb)
> >         u32 l;
> >         int status = 0;
> >         unsigned long int start;
> > +
> >         struct omap_musb_board_data *data =
> >                 (struct omap_musb_board_data *)musb->controller;
> >
> > @@ -129,3 +134,144 @@ const struct musb_platform_ops omap2430_ops =
> > { .enable         = omap2430_musb_enable,
> >         .disable        = omap2430_musb_disable,
> >  };
> > +
> > +#if defined(CONFIG_DM_USB)
> > +
> > +struct omap2430_musb_platdata {
> > +       void *base;
> > +       void *ctrl_mod_base;
> > +       struct musb_hdrc_platform_data plat;
> > +       struct musb_hdrc_config musb_config;
> > +       struct omap_musb_board_data otg_board_data;
> > +};
> > +
> > +static int omap2430_musb_ofdata_to_platdata(struct udevice *dev)
> > +{
> > +       struct omap2430_musb_platdata *platdata =
> > dev_get_platdata(dev);
> > +       const void *fdt = gd->fdt_blob;
> > +       int node = dev_of_offset(dev);
> > +
> > +       platdata->base = (void *)dev_read_addr_ptr(dev);
> > +
> > +       platdata->musb_config.multipoint = fdtdec_get_int(fdt, node,
> > +
> > "multipoint",
> > +                                                         -1);
> > +       if (platdata->musb_config.multipoint < 0) {
> > +               pr_err("MUSB multipoint DT entry missing\n");
> > +               return -ENOENT;
> > +       }
> > +
> > +       platdata->musb_config.dyn_fifo = 1;
> > +       platdata->musb_config.num_eps = fdtdec_get_int(fdt, node,
> > +                                                      "num-eps",
> > -1);
> > +       if (platdata->musb_config.num_eps < 0) {
> > +               pr_err("MUSB num-eps DT entry missing\n");
> > +               return -ENOENT;
> > +       }
> > +
> > +       platdata->musb_config.ram_bits = fdtdec_get_int(fdt, node,
> > +                                                       "ram-bits",
> > -1);
> > +       if (platdata->musb_config.ram_bits < 0) {
> > +               pr_err("MUSB ram-bits DT entry missing\n");
> > +               return -ENOENT;
> > +       }
> > +
> > +       platdata->plat.power = fdtdec_get_int(fdt, node,
> > +
> > "power", -1);
> > +       if (platdata->plat.power < 0) {
> > +               pr_err("MUSB power DT entry missing\n");
> > +               return -ENOENT;
> > +       }
> > +
> > +       platdata->otg_board_data.interface_type =
> > fdtdec_get_int(fdt, node,
> > +
> > "interface-type", -1);
> > +       if (platdata->otg_board_data.interface_type < 0) {
> > +               pr_err("MUSB interface-type DT entry missing\n");
> > +               return -ENOENT;
> > +       }
> > +
> > +#if 0 /* In a perfect world, mode would be set to OTG, mode 3 from
> > DT */
> > +       platdata->plat.mode = fdtdec_get_int(fdt, node,
> > +
> > "mode", -1);
> > +       if (platdata->plat.mode < 0) {
> > +               pr_err("MUSB mode DT entry missing\n");
> > +               return -ENOENT;
> > +       }
> > +#else /* MUSB_OTG, it doesn't work */
> > +#ifdef CONFIG_USB_MUSB_HOST /* Host seems to be the only option
> > that works */
> > +       platdata->plat.mode = MUSB_HOST;
> > +#else /* For that matter, MUSB_PERIPHERAL doesn't either */
> > +       platdata->plat.mode = MUSB_PERIPHERAL;
> > +#endif
> > +#endif
> > +       platdata->otg_board_data.dev = dev;
> > +       platdata->plat.config = &platdata->musb_config;
> > +       platdata->plat.platform_ops = &omap2430_ops;
> > +       platdata->plat.board_data = &platdata->otg_board_data;
> > +       return 0;
> > +}
> > +
> > +static int omap2430_musb_probe(struct udevice *dev)
> > +{
> > +#ifdef CONFIG_USB_MUSB_HOST
> > +       struct musb_host_data *host = dev_get_priv(dev);
> > +#endif
> > +       struct omap2430_musb_platdata *platdata =
> > dev_get_platdata(dev);
> > +       struct usb_bus_priv *priv = dev_get_uclass_priv(dev);
> > +       struct omap_musb_board_data *otg_board_data;
> > +       int ret;
> > +       void *base = dev_read_addr_ptr(dev);
> > +
> > +       priv->desc_before_addr = true;
> > +
> > +       otg_board_data = &platdata->otg_board_data;
> > +
> > +#ifdef CONFIG_USB_MUSB_HOST
> > +       host->host = musb_init_controller(&platdata->plat,
> > +                                         (struct device
> > *)otg_board_data,
> > +                                         platdata->base);
> > +       if (!host->host) {
> > +               return -EIO;
> > +       }
> > +
> > +       ret = musb_lowlevel_init(host);
> > +#else
> > +       ret = musb_register(&platdata->plat,
> > +                         (struct device *)otg_board_data,
> > +                         platdata->base);
> > +#endif
> > +       return ret;
> > +}
> > +
> > +static int omap2430_musb_remove(struct udevice *dev)
> > +{
> > +       struct musb_host_data *host = dev_get_priv(dev);
> > +
> > +       musb_stop(host->host);
> > +
> > +       return 0;
> > +}
> > +
> > +static const struct udevice_id omap2430_musb_ids[] = {
> > +       { .compatible = "ti,omap3-musb" },
> > +       { .compatible = "ti,omap4-musb" },
> > +       { }
> > +};
> > +
> > +U_BOOT_DRIVER(omap2430_musb) = {
> > +       .name   = "omap2430-musb",
> > +#ifdef CONFIG_USB_MUSB_HOST
> > +       .id             = UCLASS_USB,
> > +#else
> > +       .id             = UCLASS_USB_DEV_GENERIC,
> > +#endif
> > +       .of_match = omap2430_musb_ids,
> > +       .ofdata_to_platdata = omap2430_musb_ofdata_to_platdata,
> > +       .probe = omap2430_musb_probe,
> > +       .remove = omap2430_musb_remove,
> > +       .ops    = &musb_usb_ops,
> > +       .platdata_auto_alloc_size = sizeof(struct
> > omap2430_musb_platdata),
> > +       .priv_auto_alloc_size = sizeof(struct musb_host_data),
> > +};
> > +
> > +#endif /* CONFIG_DM_USB */
> > --
> > 2.17.1
> >  




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180722/8eddd873/attachment.sig>