public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Matthias Fuchs <matthias.fuchs@esd.eu>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] i.MX28: Fix VDDIO and VDDA setup
Date: Tue, 31 Jan 2012 11:24:36 +0100	[thread overview]
Message-ID: <4F27C164.2030203@esd.eu> (raw)
In-Reply-To: <1327968001-13567-1-git-send-email-marek.vasut@gmail.com>

I tested both patch on my mx28evk. At least I cannot notice any
missbehavior :-) Downloading the patched u-boot via USB recovery
download still does not work.

Marek, did you find any further mentionable difference between bootlets
code and current u-boot implementation?

Matthias

On 31.01.2012 01:00, Marek Vasut wrote:
> The DC power STS shouldn't be checked if booting off 5V supply.
> 
> Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> Cc: Wolfgang Denk <wd@denx.de>
> Cc: Detlev Zundel <dzu@denx.de>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Robert Deli?n <robert@delien.nl>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: Matthias Fuchs <matthias.fuchs@esd.eu>
> ---
>  arch/arm/cpu/arm926ejs/mx28/spl_power_init.c |   16 ++++++++++++----
>  1 files changed, 12 insertions(+), 4 deletions(-)
> 
> Note: Guys, please give this a testrun. I think it might fix the issue Robert
> was observing. I dug into the imx-bootlets and the bootrom X-Files and found
> this paranormal piece of code. Adding it fixes the boot issue on my crappy
> board.
> 
> M
> 
> diff --git a/arch/arm/cpu/arm926ejs/mx28/spl_power_init.c b/arch/arm/cpu/arm926ejs/mx28/spl_power_init.c
> index ee0f237..c6c25d9 100644
> --- a/arch/arm/cpu/arm926ejs/mx28/spl_power_init.c
> +++ b/arch/arm/cpu/arm926ejs/mx28/spl_power_init.c
> @@ -802,7 +802,9 @@ void mx28_power_set_vddio(uint32_t new_target, uint32_t new_brownout)
>  			clrsetbits_le32(&power_regs->hw_power_vddioctrl,
>  				POWER_VDDIOCTRL_TRG_MASK, diff);
>  
> -			if (powered_by_linreg)
> +			if (powered_by_linreg ||
> +				(readl(&power_regs->hw_power_sts) &
> +					POWER_STS_VDD5V_GT_VDDIO))
>  				early_delay(1500);
>  			else {
>  				while (!(readl(&power_regs->hw_power_sts) &
> @@ -837,7 +839,9 @@ void mx28_power_set_vddio(uint32_t new_target, uint32_t new_brownout)
>  			clrsetbits_le32(&power_regs->hw_power_vddioctrl,
>  				POWER_VDDIOCTRL_TRG_MASK, diff);
>  
> -			if (powered_by_linreg)
> +			if (powered_by_linreg ||
> +				(readl(&power_regs->hw_power_sts) &
> +					POWER_STS_VDD5V_GT_VDDIO))
>  				early_delay(1500);
>  			else {
>  				while (!(readl(&power_regs->hw_power_sts) &
> @@ -895,7 +899,9 @@ void mx28_power_set_vddd(uint32_t new_target, uint32_t new_brownout)
>  			clrsetbits_le32(&power_regs->hw_power_vdddctrl,
>  				POWER_VDDDCTRL_TRG_MASK, diff);
>  
> -			if (powered_by_linreg)
> +			if (powered_by_linreg ||
> +				(readl(&power_regs->hw_power_sts) &
> +					POWER_STS_VDD5V_GT_VDDIO))
>  				early_delay(1500);
>  			else {
>  				while (!(readl(&power_regs->hw_power_sts) &
> @@ -930,7 +936,9 @@ void mx28_power_set_vddd(uint32_t new_target, uint32_t new_brownout)
>  			clrsetbits_le32(&power_regs->hw_power_vdddctrl,
>  					POWER_VDDDCTRL_TRG_MASK, diff);
>  
> -			if (powered_by_linreg)
> +			if (powered_by_linreg ||
> +				(readl(&power_regs->hw_power_sts) &
> +					POWER_STS_VDD5V_GT_VDDIO))
>  				early_delay(1500);
>  			else {
>  				while (!(readl(&power_regs->hw_power_sts) &


-- 
------------------------------------------------------------------------
Dipl.-Ing. Matthias Fuchs
Head of System Design

esd electronic system design gmbh
Vahrenwalder Str. 207 - 30165 Hannover - GERMANY
Phone: +49-511-37298-0 - Fax: +49-511-37298-68
Please visit our homepage http://www.esd.eu
Quality Products - Made in Germany

Besuchen Sie uns auf der embedded world 2012
in Halle 4, Stand 129
vom 28. Februar - 01. M?rz 2012 in N?rnberg!
------------------------------------------------------------------------
Gesch?ftsf?hrer: Klaus Detering
Amtsgericht Hannover HRB 51373 - VAT-ID DE 115672832
-------------------------------------------------------------------------

  reply	other threads:[~2012-01-31 10:24 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-31  0:00 [U-Boot] [PATCH] i.MX28: Fix VDDIO and VDDA setup Marek Vasut
2012-01-31 10:24 ` Matthias Fuchs [this message]
2012-01-31 11:38   ` Marek Vasut
2012-01-31 13:03     ` Matthias Fuchs
2012-01-31 12:26 ` Fabio Estevam
2012-01-31 12:45   ` Marek Vasut
2012-01-31 17:21     ` Robert Deliën
2012-01-31 18:08       ` Marek Vasut
2012-01-31 18:33       ` Marek Vasut
2012-02-01 16:55         ` Robert Deliën
2012-02-02 15:30         ` Robert Deliën
2012-01-31 15:13   ` Robert Deliën
2012-02-07 17:03 ` Stefano Babic

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=4F27C164.2030203@esd.eu \
    --to=matthias.fuchs@esd.eu \
    --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