From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marek.vasut@gmail.com>
Date: Thu, 25 Aug 2011 05:38:39 +0200
Subject: [U-Boot] [PATCH] beagleboard: enable HUB power on all variants
	of the BeagleBoard
In-Reply-To: <CA+T6QPnfzwRhwq7Pq3CxYXoFKWowyosYEtH8VK=SuqFjvwa8zw@mail.gmail.com>
References: <1314226061-6933-1-git-send-email-agnel.joel@gmail.com>
	<201108250522.09262.marek.vasut@gmail.com>
	<CA+T6QPnfzwRhwq7Pq3CxYXoFKWowyosYEtH8VK=SuqFjvwa8zw@mail.gmail.com>
Message-ID: <201108250538.40014.marek.vasut@gmail.com>
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 Thursday, August 25, 2011 05:30:59 AM Jason Kridner wrote:
> On Wed, Aug 24, 2011 at 11:22 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
> > On Thursday, August 25, 2011 12:47:41 AM Joel A Fernandes wrote:
> >> From: Christian Spielberger <c.spielberger@bct-electronic.com>
> >> 
> >> Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
> >> Cc: "Eric B??nard" <eric@eukrea.com>
> >> Cc: "Christian Spielberger" <c.spielberger@bct-electronic.com>
> >> ---
> >> Pushing this patch myself as no one else did.
> >> 
> >> Previous discussions:
> >> http://article.gmane.org/gmane.comp.boot-loaders.u-boot/105663/
> >> http://article.gmane.org/gmane.comp.boot-loaders.u-boot/101540/
> >> 
> >>  board/ti/beagle/beagle.c |   10 +++++++++-
> >>  1 files changed, 9 insertions(+), 1 deletions(-)
> >> 
> >> diff --git a/board/ti/beagle/beagle.c b/board/ti/beagle/beagle.c
> >> index 13fe39b..9d65e9e 100644
> >> --- a/board/ti/beagle/beagle.c
> >> +++ b/board/ti/beagle/beagle.c
> >> @@ -332,7 +332,15 @@ int misc_init_r(void)
> >>               setenv(expansion_config.env_var,
> >> expansion_config.env_setting);
> >> 
> >>       twl4030_power_init();
> >> -     twl4030_led_init(TWL4030_LED_LEDEN_LEDAON |
> >> TWL4030_LED_LEDEN_LEDBON); +     switch (get_board_revision()) {
> >> +     case REVISION_XM_C:
> >> +     case REVISION_C4:
> >> +             twl4030_led_init(TWL4030_LED_LEDEN_LEDAON |
> >> TWL4030_LED_LEDEN_LEDBON); +             break;
> >> +     default:
> >> +             twl4030_led_init(TWL4030_LED_LEDEN_LEDBON);
> >> +             break;
> >> +     }
> 
> Please excuse if this is a duplicate, but I'm configuring a new mail
> tool and I don't think my other message got sent.
> 
> It is XM_A and XM_B that are odd-balls, so the correct code should be
> something like:
> 
> +     switch (get_board_revision()) {
> +     case REVISION_XM_A:
> +     case REVISION_XM_B:
> +             twl4030_led_init(TWL4030_LED_LEDEN_LEDBON); /* LEDA
> signal set low */
> +             break;
> +     default:
> +             twl4030_led_init(TWL4030_LED_LEDEN_LEDAON |
> TWL4030_LED_LEDEN_LEDBON);
> +             break;
> 
> >> +     }
> > 
> > Am I just too sleepy or the Subject doesn't correlate with what the patch
> > does ...
> > 
> > My understanding of Subject is that it enables power on HUB ... USB HUB ?
> > But the patch enables some LEDs ?
> 
> The LED drivers on the power management device (TWL4030-compatible)
> are used to enable the hub power.

Oh uh ... ok, now I understand. Please add a comment explaining it.

Also, change the subject se it's clear you're operating with USB HUB here.

Thanks!

> 
> > I might be wrong.
> > 
> > CHeers
> > 
> >>       /* Set GPIO states before they are made outputs */
> >>       writel(GPIO23 | GPIO10 | GPIO8 | GPIO2 | GPIO1,
> > 
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot at lists.denx.de
> > http://lists.denx.de/mailman/listinfo/u-boot