All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 4/8] arm: bootm: call udc_disable() before booting linux
Date: Sun, 14 Apr 2013 22:12:36 +0200	[thread overview]
Message-ID: <201304142212.36618.marex@denx.de> (raw)
In-Reply-To: <20130414212243.1f8ae3ec@lilith>

Dear Albert ARIBAUD,

> Hi Mike,
> 
> On Fri, 12 Apr 2013 11:59:16 -0700, Mike Dunn <mikedunn@newsguy.com>
> 
> wrote:
> > On the pxa270, if the udc device is not disabled before jumping to linux,
> > the device fails to initialize in linux because it was left in a running
> > state, and the linux driver assumes that it is in a disabled state.
> > 
> > Signed-off-by: Mike Dunn <mikedunn@newsguy.com>
> > ---
> > 
> > Changelog:
> >   v3: no change
> >   v2: no change
> > 
> > Arguably, this is a bug in the linux driver, but it seemed pretty simple
> > and benign to just disable it in u-boot.  I'll also send a patch to
> > upstream kernel to fix its driver.
> > 
> >  arch/arm/include/asm/bootm.h |    1 +
> >  arch/arm/lib/bootm.c         |    1 +
> >  2 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/arch/arm/include/asm/bootm.h b/arch/arm/include/asm/bootm.h
> > index db2ff94..68189cc 100644
> > --- a/arch/arm/include/asm/bootm.h
> > +++ b/arch/arm/include/asm/bootm.h
> > @@ -21,6 +21,7 @@
> > 
> >  #ifdef CONFIG_USB_DEVICE
> >  extern void udc_disconnect(void);
> > 
> > +extern void udc_disable(void);
> > 
> >  #endif
> >  
> >  #endif
> > 
> > diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c
> > index f3b30c5..6daa7bf 100644
> > --- a/arch/arm/lib/bootm.c
> > +++ b/arch/arm/lib/bootm.c
> > @@ -105,6 +105,7 @@ static void announce_and_cleanup(void)
> > 
> >  #ifdef CONFIG_USB_DEVICE
> >  
> >  	udc_disconnect();
> > 
> > +	udc_disable();
> > 
> >  #endif
> >  
> >  	cleanup_before_linux();
> >  
> >  }
> 
> Acked-By: Albert ARIBAUD <albert.u.boot@aribaud.net>

Tom, will this not break any of your musb-toys?

Best regards,
Marek Vasut

  reply	other threads:[~2013-04-14 20:12 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-12 18:59 [U-Boot] [PATCH v3 0/8] palm treo 680 smartphone board support Mike Dunn
2013-04-12 18:59 ` [U-Boot] [PATCH v3 1/8] pxa_lcd: add the ACX544AKN lcd device Mike Dunn
2013-04-12 18:59 ` [U-Boot] [PATCH v3 2/8] pxa_lcd: make lcd_enable() a weak pointer Mike Dunn
2013-04-12 18:59 ` [U-Boot] [PATCH v3 3/8] pxa27x_udc: remove call to unimplemented set_GPIO_mode() Mike Dunn
2013-04-12 18:59 ` [U-Boot] [PATCH v3 4/8] arm: bootm: call udc_disable() before booting linux Mike Dunn
2013-04-14 17:26   ` Marek Vasut
2013-04-14 19:22   ` Albert ARIBAUD
2013-04-14 20:12     ` Marek Vasut [this message]
2013-04-15 12:45       ` Tom Rini
2013-04-15 12:56         ` Albert ARIBAUD
2013-04-15 13:08           ` Marek Vasut
2013-04-12 18:59 ` [U-Boot] [PATCH 5/8] lib: import bitrev library from the linux kernel Mike Dunn
2013-04-12 18:59 ` [U-Boot] [PATCH v3 6/8] mtd: nand: add driver for diskonchip g4 nand flash Mike Dunn
2013-04-12 19:21   ` Scott Wood
2013-04-13 14:42     ` Marek Vasut
2013-04-15 17:51       ` Scott Wood
2013-04-12 18:59 ` [U-Boot] [PATCH v3 7/8] add support for palm treo 680 board Mike Dunn
2013-04-14 17:34   ` Marek Vasut
2013-04-15 17:34     ` Mike Dunn
2013-04-15 18:33       ` Marek Vasut
2013-04-15 19:11         ` Mike Dunn
2013-04-12 18:59 ` [U-Boot] [PATCH v3 8/8] palmtreo680: add utility that writes u-boot to flash Mike Dunn
2013-04-14 17:38   ` Marek Vasut
2013-04-16 17:50     ` Mike Dunn
2013-04-16 18:06       ` Marek Vasut

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=201304142212.36618.marex@denx.de \
    --to=marex@denx.de \
    --cc=u-boot@lists.denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.