From mboxrd@z Thu Jan  1 00:00:00 1970
From: Mike Frysinger <vapier@gentoo.org>
Date: Sat, 31 Oct 2009 17:49:11 -0400
Subject: [U-Boot] [PATCH 09/10] OMAP3 USB Initialize twl4030 only if
	required
In-Reply-To: <1257010667-10834-10-git-send-email-Tom.Rix@windriver.com>
References: <1257010667-10834-1-git-send-email-Tom.Rix@windriver.com>
	<1257010667-10834-9-git-send-email-Tom.Rix@windriver.com>
	<1257010667-10834-10-git-send-email-Tom.Rix@windriver.com>
Message-ID: <200910311749.12796.vapier@gentoo.org>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Saturday 31 October 2009 13:37:46 Tom Rix wrote:
> OMAP3EVM uses ISP1504 phy and so twl4030 related init is not required.
> 
> --- a/drivers/usb/musb/omap3.c
> +++ b/drivers/usb/musb/omap3.c
> @@ -94,12 +94,17 @@ int musb_platform_init(void)
>  	if (platform_needs_initialization) {
>  		u32 stdby;
> 
> +		/*
> +		 * OMAP3EVM uses ISP1504 phy and so
> +		 * twl4030 related init is not required.
> +		 */
> +#ifdef CONFIG_TWL4030_USB
>  		if (twl4030_usb_ulpi_init()) {
>  			serial_printf("ERROR: %s Could not initialize PHY\n",
>  				__PRETTY_FUNCTION__);
>  			goto end;
>  		}
> -
> +#endif

shouldnt this be abstracted away (weaks/phy-common name/etc...) instead of 
sprinkling ifdef's everywhere ?
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20091031/11d4eb60/attachment.pgp