[U-Boot] [PATCH 15/22] musb: Add device-model support to the musb-host u-boot glue

[Simon Glass]

Hi Hans,

On 17 June 2015 at 13:33, Hans de Goede <hdegoede at redhat.com> wrote:
> Add device-model support to the musb-host u-boot glue, note this only
> adds device-model support to the musb-core glue code, it does not add
> support for device-model to any of the SoC specific musb glue code.
>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> ---
>  drivers/usb/musb-new/musb_uboot.c | 70 ++++++++++++++++++++++++++++++++++++++-
>  drivers/usb/musb-new/musb_uboot.h |  4 +++
>  2 files changed, 73 insertions(+), 1 deletion(-)
>

Acked-by: Simon Glass <sjg at chromium.org>

Nit below.

> diff --git a/drivers/usb/musb-new/musb_uboot.c b/drivers/usb/musb-new/musb_uboot.c
> index 9875100..9b56e90 100644
> --- a/drivers/usb/musb-new/musb_uboot.c
> +++ b/drivers/usb/musb-new/musb_uboot.c
> @@ -21,7 +21,9 @@ struct int_queue {
>         struct urb urb;
>  };
>
> +#ifndef CONFIG_DM_USB
>  struct musb_host_data musb_host;
> +#endif
>
>  static void musb_host_complete_urb(struct urb *urb)
>  {
> @@ -244,6 +246,7 @@ int musb_lowlevel_init(struct musb_host_data *host)
>         return 0;
>  }
>
> +#ifndef CONFIG_DM_USB
>  int usb_lowlevel_stop(int index)
>  {
>         if (!musb_host.host) {
> @@ -300,6 +303,71 @@ int usb_lowlevel_init(int index, enum usb_init_type init, void **controller)
>  {
>         return musb_lowlevel_init(&musb_host);
>  }
> +#endif /* !CONFIG_DM_USB */
> +
> +#ifdef CONFIG_DM_USB
> +static int musb_submit_control_msg(struct udevice *dev, struct usb_device *udev,
> +                                  unsigned long pipe, void *buffer, int length,
> +                                  struct devrequest *setup)
> +{
> +       struct musb_host_data *host = dev_get_priv(dev);
> +       return _musb_submit_control_msg(host, udev, pipe, buffer, length, setup);
> +}
> +
> +static int musb_submit_bulk_msg(struct udevice *dev, struct usb_device *udev,
> +                               unsigned long pipe, void *buffer, int length)
> +{
> +       struct musb_host_data *host = dev_get_priv(dev);
> +       return _musb_submit_bulk_msg(host, udev, pipe, buffer, length);
> +}
> +
> +static int musb_submit_int_msg(struct udevice *dev, struct usb_device *udev,
> +                              unsigned long pipe, void *buffer, int length,
> +                              int interval)
> +{
> +       struct musb_host_data *host = dev_get_priv(dev);
> +       return _musb_submit_int_msg(host, udev, pipe, buffer, length, interval);
> +}
> +
> +static struct int_queue *musb_create_int_queue(struct udevice *dev,
> +               struct usb_device *udev, unsigned long pipe, int queuesize,
> +               int elementsize, void *buffer, int interval)
> +{
> +       struct musb_host_data *host = dev_get_priv(dev);

Can we have newlines after declarations?

> +       return _musb_create_int_queue(host, udev, pipe, queuesize, elementsize,
> +                                     buffer, interval);
> +}
> +
> +static void *musb_poll_int_queue(struct udevice *dev, struct usb_device *udev,
> +                                struct int_queue *queue)
> +{
> +       struct musb_host_data *host = dev_get_priv(dev);
> +       return _musb_poll_int_queue(host, udev, queue);
> +}
> +
> +static int musb_destroy_int_queue(struct udevice *dev, struct usb_device *udev,
> +                                 struct int_queue *queue)
> +{
> +       struct musb_host_data *host = dev_get_priv(dev);
> +       return _musb_destroy_int_queue(host, udev, queue);
> +}
> +
> +static int musb_reset_root_port(struct udevice *dev, struct usb_device *udev)
> +{
> +       struct musb_host_data *host = dev_get_priv(dev);
> +       return _musb_reset_root_port(host, udev);
> +}
> +
> +struct dm_usb_ops musb_usb_ops = {
> +       .control = musb_submit_control_msg,
> +       .bulk = musb_submit_bulk_msg,
> +       .interrupt = musb_submit_int_msg,
> +       .create_int_queue = musb_create_int_queue,
> +       .poll_int_queue = musb_poll_int_queue,
> +       .destroy_int_queue = musb_destroy_int_queue,
> +       .reset_root_port = musb_reset_root_port,
> +};
> +#endif /* CONFIG_DM_USB */
>  #endif /* CONFIG_MUSB_HOST */
>
>  #ifdef CONFIG_MUSB_GADGET
> @@ -360,7 +428,7 @@ int musb_register(struct musb_hdrc_platform_data *plat, void *bdata,
>         struct musb **musbp;
>
>         switch (plat->mode) {
> -#ifdef CONFIG_MUSB_HOST
> +#if defined(CONFIG_MUSB_HOST) && !defined(CONFIG_DM_USB)
>         case MUSB_HOST:
>                 musbp = &musb_host.host;
>                 break;
> diff --git a/drivers/usb/musb-new/musb_uboot.h b/drivers/usb/musb-new/musb_uboot.h
> index 69b7977..6312cd2 100644
> --- a/drivers/usb/musb-new/musb_uboot.h
> +++ b/drivers/usb/musb-new/musb_uboot.h
> @@ -21,4 +21,8 @@ struct musb_host_data {
>         struct urb urb;
>  };
>
> +extern struct dm_usb_ops musb_usb_ops;
> +
> +int musb_lowlevel_init(struct musb_host_data *host);
> +
>  #endif
> --
> 2.4.3
>

Regards,
Simon