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] mxs: fixed battery boot on imx233-olinuxino-micro
Date: Sat, 24 May 2014 12:44:48 +0200	[thread overview]
Message-ID: <201405241244.48213.marex@denx.de> (raw)
In-Reply-To: <1400637715-24117-1-git-send-email-peter.schumann@jeepxj.de>

On Wednesday, May 21, 2014 at 04:01:55 AM, Peter Schumann wrote:
> This patch makes it possible to boot from battery on olinuxino Boards
> from Olimex.
> 
> We should not set DOUBLE_FETS in HW_POWER_MINPWR

Please explain how you arrived to this conclusion.

> , it makes my system
> reset with in seconds while running on battery power.
>
> Also  mxs_power_enable_4p2() should not be called if running from
> battery. It switches VDDD and VDDA to LinReg mode and than turns off
> the VDDIO DCDC supply. At this point the system resets.

This has been tested on i.MX28 , can you please check what the differences are 
between these two chips and why it works only on one of them ?

> If we run on battery this is no good idea because the LinRegs are
> chained behind VDDIO and battery voltage seems not enough to power
> the system from LinRegs.
> Also is the power system already running on DCDC after BootROM hands
> over to u-boot, so no sense in switching back to LinRegs.

The BootROM never hands over to U-Boot, so this really makes no sense. Can you 
please explain ? The BootROM handles over to U-Boot SPL, which configures the 
power block.

Also, I find it a very bad idea to depend on the BootROM to actually start the 
DCDC converter. I don't think we can universally say the DCDC converter is 
running when entering U-Boot SPL. I also don't think we can even depend on the 
power block configuration when exiting the BootROM -- for example in JTAG Boot 
Mode, the BootROM halts early in the boot process and will likely not configure 
anything with regards to the power block.

[...]

THanks!

Best regards,
Marek Vasut

  reply	other threads:[~2014-05-24 10:44 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-21  2:01 [U-Boot] [PATCH] mxs: fixed battery boot on imx233-olinuxino-micro Peter Schumann
2014-05-24 10:44 ` Marek Vasut [this message]
2014-05-25 20:02   ` Peter Schumann
2014-06-05 22:18 ` Marek Vasut
2014-06-13  1:04   ` Marek Vasut
2014-06-13  7:36     ` Peter Schumann
2014-06-13 10:33       ` Marek Vasut
2014-06-15 13:11 ` [U-Boot] [PATCH v2] " Peter Schumann
2014-06-16 22:18   ` Marek Vasut
2014-06-16 22:19     ` Marek Vasut
2014-06-17 16:00       ` Stefano Babic
2014-06-18 18:17         ` Peter Schumann
2014-06-19  7:12           ` Marek Vasut
2014-08-20 10:30             ` 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=201405241244.48213.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.