public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Eric Witcher <ewitcher@mindspring.com>
Cc: Steve Sakoman <sakoman@gmail.com>, linux-omap@vger.kernel.org
Subject: Re: ARM: OMAP3: Beagleboard: Fix EHCI initialization
Date: Mon, 21 Sep 2009 18:35:51 -0700	[thread overview]
Message-ID: <20090922013550.GL27699@atomide.com> (raw)
In-Reply-To: <979D9760-ABA4-48A1-87DE-5E335E7C24F9@mindspring.com>

* Eric Witcher <ewitcher@mindspring.com> [090907 18:17]:
>
> On Sep 1, 2009, at 5:55 PM, Steve Sakoman wrote:
>
>> Enable USB power, use correct gpio for usb_ehci_init
>>
>> Signed-off-by: Steve Sakoman <steve@sakoman.com>
>
> Tested-by: Eric Witcher <ewitcher@mindspring.com>
> This patch worked at commit
> 3dff8750255bf5a1e9a69931b1d8739a65a933d0
> of linux-omap-2.6.git master

This one should also be updated to pass the configuration
in the platform data unless already done with what's in
the linux-omap ehci and master branches. Can you please
check?

> I also needed
>
> [PATCH 4/8] ehci: fix phy_reset init in ehci probe
> [PATCH] omapfb: Reorder Register_framebuffer call
>
> to get a clean boot.

I've added this into omap-fixes queue and merged it
also into the linux-omap master branch.

Regards,

Tony


> Build configuration was omap3_beagle_defconfig.
>
>
>> ---
>> arch/arm/mach-omap2/board-omap3beagle.c |    5 +++--
>> 1 files changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/board-omap3beagle.c
>> b/arch/arm/mach-omap2/board-omap3beagle.c
>> index 07aee90..65934ea 100644
>> --- a/arch/arm/mach-omap2/board-omap3beagle.c
>> +++ b/arch/arm/mach-omap2/board-omap3beagle.c
>> @@ -161,7 +161,7 @@ static int beagle_twl_gpio_setup(struct device  
>> *dev,
>>
>> 	/* TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, active low)  
>> */
>> 	gpio_request(gpio + TWL4030_GPIO_MAX, "nEN_USB_PWR");
>> -	gpio_direction_output(gpio + TWL4030_GPIO_MAX, 1);
>> +	gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
>>
>> 	/* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
>> 	gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
>> @@ -412,7 +412,8 @@ static void __init omap3_beagle_init(void)
>> 	gpio_direction_output(170, true);
>>
>> 	usb_musb_init();
>> -	usb_ehci_init(EHCI_HCD_OMAP_MODE_PHY, true, true, 57, 61);
>> +	usb_ehci_init(EHCI_HCD_OMAP_MODE_PHY, false, true,
>> +			-EINVAL, 147);
>> 	omap3beagle_flash_init();
>>
>> 	/* Ensure SDRC pins are mux'd for self-refresh */
>> -- 
>> 1.6.0.4
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-omap" 
>> in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

      reply	other threads:[~2009-09-22  1:35 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-01 21:55 ARM: OMAP3: Beagleboard: Fix EHCI initialization Steve Sakoman
2009-09-08  1:17 ` Eric Witcher
2009-09-22  1:35   ` Tony Lindgren [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090922013550.GL27699@atomide.com \
    --to=tony@atomide.com \
    --cc=ewitcher@mindspring.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=sakoman@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox