public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
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 21:22:43 +0200	[thread overview]
Message-ID: <20130414212243.1f8ae3ec@lilith> (raw)
In-Reply-To: <1365793160-18247-5-git-send-email-mikedunn@newsguy.com>

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>

Amicalement,
-- 
Albert.

  parent reply	other threads:[~2013-04-14 19:22 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 [this message]
2013-04-14 20:12     ` Marek Vasut
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=20130414212243.1f8ae3ec@lilith \
    --to=albert.u.boot@aribaud.net \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox