[U-Boot] PATCH add Beagleboard OTG host support

Wolfgang Denk wd at denx.de
Thu Feb 19 22:55:35 CET 2009


Dear yanfeng qin,

In message <254f00050902191102q72bfbe00n1ad677c02b807a42 at mail.gmail.com> you wrote:
> 
> This patch is adding usb(otg) host support for beagleboard.
> 
> Signed-off-by: yanfeng qin <connieqin1975 at gmail.com>
> Signed-off-by: garylu312 <garylu312 at gmail.com>
> Signed-off-by: shaofeng wang <wsf064 at gmail.com>
> ---
> Patch against omap3-dev-usb

Is this a branch name? If yes, in which repository?

>  drivers/usb/Makefile           |   12 +++---------
>  include/configs/omap3_beagle.h |    9 +++++----
>  include/usb.h                  |    6 +-----
>  3 files changed, 9 insertions(+), 18 deletions(-)
> 
> Index: u-boot-steve-omap3/drivers/usb/Makefile
> ===================================================================
> --- u-boot-steve-omap3.orig/drivers/usb/Makefile
> +++ u-boot-steve-omap3/drivers/usb/Makefile


Can you please use git-format-patch to create the patch? 

> @@ -31,17 +31,11 @@ COBJS-$(CONFIG_USB_OHCI_NEW) += usb_ohci
>  COBJS-$(CONFIG_USB_EHCI) += usb_ehci_core.o
> 
>  # host
> -COBJS-$(CONFIG_USB_ISP116X_HCD) += isp116x-hcd.o
> -COBJS-$(CONFIG_USB_R8A66597_HCD) += r8a66597-hcd.o
> -COBJS-$(CONFIG_USB_SL811HS) += sl811_usb.o
> -COBJS-$(CONFIG_USB_EHCI_FSL) += usb_ehci_fsl.o
> -COBJS-$(CONFIG_USB_EHCI_PCI) += usb_ehci_pci.o
> -COBJS-$(CONFIG_USB_EHCI_IXP4XX) += usb_ehci_ixp.o
> +ifdef CONFIG_USB_HOST
>  COBJS-$(CONFIG_MUSB_HCD) += musb_hcd.o musb_core.o
> -COBJS-$(CONFIG_USB_DAVINCI) += davinci_usb.o
> -COBJS-$(CONFIG_USB_EHCI_VCT) += usb_ehci_vct.o
>  COBJS-$(CONFIG_USB_OMAP3530) += omap3530_usb.o
> -
> +COBJS-$(CONFIG_TWL4030_USB) += twl4030_usb.o

It looks as if you were removing support for quite a lot of other USB
drivers here?


> Index: u-boot-steve-omap3/include/usb.h
> ===================================================================
> --- u-boot-steve-omap3.orig/include/usb.h
> +++ u-boot-steve-omap3/include/usb.h
> @@ -180,11 +180,7 @@ struct usb_device {
>   * this is how the lowlevel part communicate with the outer world
>   */
> 
> -#if defined(CONFIG_USB_UHCI) || defined(CONFIG_USB_OHCI) || \
> -    defined(CONFIG_USB_EHCI) || defined(CONFIG_USB_OHCI_NEW) || \
> -    defined(CONFIG_USB_SL811HS) || defined(CONFIG_USB_ISP116X_HCD) || \
> -    defined(CONFIG_USB_R8A66597_HCD) || defined(CONFIG_USB_DAVINCI) || \
> -    defined(CONFIG_USB_OMAP3530)
> +#if defined(CONFIG_MUSB_HCD)

It looks as if you were removing support for quite a lot of other USB
drivers here?


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
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
A day without sunshine is like night.


More information about the U-Boot mailing list