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
-------------------------------------------------------------------------
next prev parent 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 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.