[U-Boot] [PATCH v5 01/11] usb, davinci: add enable_vbus() weak function

Igor Grinberg grinberg at compulab.co.il
Mon Oct 31 08:02:15 CET 2011


Hi Heiko,

On 10/31/11 06:53, Heiko Schocher wrote:
> Signed-off-by: Heiko Schocher <hs at denx.de>
> Cc: Remy Bohmer <linux at bohmer.net>
> Cc: Albert ARIBAUD <albert.u.boot at aribaud.net>
> Cc: Igor Grinberg <grinberg at compulab.co.il>
> 
> ---
> changes for v4:
> - patch is new since v4, as Igor Grinberg suggested.
> 
>  drivers/usb/musb/davinci.c |   12 ++++++++++++
>  1 files changed, 12 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/usb/musb/davinci.c b/drivers/usb/musb/davinci.c
> index f56f2df..27dc4bc 100644
> --- a/drivers/usb/musb/davinci.c
> +++ b/drivers/usb/musb/davinci.c
> @@ -78,6 +78,18 @@ static void phy_off(void)
>  	writel(USBPHY_OSCPDWN | USBPHY_PHYPDWN, USBPHY_CTL_PADDR);
>  }
>  
> +#ifndef DAVINCI_DM365EVM
> +__attribute__((weak))
> +void enable_vbus(void)
> +{
> +	/*
> +	 *  nothing to do, vbus is handled through the cpu.
> +	 *  Define this function in board code, if it is
> +	 *  different on your board.
> +	 */
> +}
> +#endif

I'd suggest, instead of closing the weak declaration inside
ifdef/ifndef, make it available to all boards with alias.

> +
>  /*
>   * This function performs Davinci platform specific initialization for usb0.
>   */

-- 
Regards,
Igor.


More information about the U-Boot mailing list