From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 19/19] imx: ventana: config: enable Falcon mode
Date: Thu, 21 May 2015 18:46:40 +0200 [thread overview]
Message-ID: <555E0BF0.3040402@denx.de> (raw)
In-Reply-To: <555E0526.8020100@denx.de>
Hi Stefano,
On 21.05.2015 18:17, Stefano Babic wrote:
> On 19/05/2015 15:26, Tim Harvey wrote:
>
>> Note that Falcon mode also requires you have a function in the SPL
>> that decides to boot U-Boot or to skip it and boot the OS directly. In
>> the Gateworks Ventana case I didn't want to make this completely
>> dependent on a GPIO/Button because this bootloader supports about 8
>> different PCB designs in the Ventana family - some of which do not
>> have buttons or may not have buttons loaded on the board. Instead, I
>> pulled env support into the SPL and use 'boot_os' env var to decide as
>> other boards do (see spl_start_uboot()).
>
> Anyway, this can be dangerous. The reason having a GPIO (or any trigger
> from external, for example a received char from the console) is to
> switch to U-Boot if the kernel does not work (but the kernel image is
> not corrupted), mainly go to panic, and the board can be restored into
> U-Boot.
>
> If you use an environment variable and the environment is put into the
> NAND (with MMC you can change the card), there is nothing you can do
> anymore. Board tries to boot until next panic, and then again and again.
Some boards just don't have such a means as a push button or jumper to
select the Falcon mode. That's why I introduced this environment
variable dependent way to select Falcon mode quite some time ago for a
PPC based board.
You are correct. If something goes completely wrong, and the board tries
to boot into the OS with a non-working OS installed, then you are
doomed. JTAG comes in handy then as I have experienced myself. ;)
But this is usually only while developing this feature and kernel. Once
its tested and mature, this really works.
Again, its a trade-off since another, better method to switch between
U-Boot- and OS- booting is not available.
Thanks,
Stefan
next prev parent reply other threads:[~2015-05-21 16:46 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-09 1:28 [U-Boot] [PATCH 00/19] imx: ventana: misc updates Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 01/19] imx: ventana: set dtype env var to boot media Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 02/19] imx: ventana: display SPL boot device Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 03/19] imx: ventana: config: enable gpio command Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 04/19] imx: ventana: config: enable driver model Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 05/19] imx: ventana: register gpio's with gpio_request Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 06/19] imx: ventana: enable DM_SERIAL Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 07/19] imx: ventana: config: enable Thermal support Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 08/19] imx: ventana: config: use MMC SPL RAW support Tim Harvey
2015-05-09 1:36 ` Fabio Estevam
2015-05-12 23:23 ` Tim Harvey
2015-05-15 14:20 ` Stefano Babic
2015-05-09 1:28 ` [U-Boot] [PATCH 09/19] imx: ventana: (cosmetic) clean up size defines for improved readability Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 10/19] imx: ventana: fix pcie reset for GW522x Tim Harvey
2015-05-11 7:59 ` Stefano Babic
2015-05-11 19:12 ` Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 11/19] imx: ventana: default msata/pci mux to pci before PCI enumeration Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 12/19] imx: ventana: split out common functions between SPL and uboot Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 13/19] imx: ventana: move GSC boot watchdog disable function to gsc.c Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 14/19] imx: ventana: detect pmic using i2c probe instead of board model Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 15/19] imx: ventana: use common uart and i2c setup functions in SPL Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 16/19] imx: ventana: add gpio setup to SPL Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 17/19] imx: ventana: add pmic_setup " Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 18/19] imx: ventana: add GSC boot watchdog disable " Tim Harvey
2015-05-09 1:28 ` [U-Boot] [PATCH 19/19] imx: ventana: config: enable Falcon mode Tim Harvey
2015-05-18 23:25 ` Fabio Estevam
2015-05-19 13:26 ` Tim Harvey
2015-05-19 13:30 ` Fabio Estevam
2015-05-21 16:17 ` Stefano Babic
2015-05-21 16:46 ` Stefan Roese [this message]
2015-05-21 17:06 ` Stefano Babic
2015-05-21 17:07 ` Tim Harvey
2015-05-15 14:25 ` [U-Boot] [PATCH 00/19] imx: ventana: misc updates Stefano Babic
2015-05-15 16:30 ` Tim Harvey
2015-05-15 16:56 ` Stefano Babic
2015-05-15 18:57 ` Otavio Salvador
2015-05-21 16:56 ` Tom Rini
2015-05-21 17:02 ` Stefano Babic
2015-05-15 16:14 ` [U-Boot] [PATCH v2 08/19] imx: ventana: config: use MMC SPL RAW support Tim Harvey
2015-05-19 13:06 ` Stefano Babic
2015-05-15 16:17 ` [U-Boot] [PATCH v2 09/19] imx: ventana: (cosmetic) clean up size defines for improved readability Tim Harvey
2015-05-19 13:06 ` Stefano Babic
2015-05-15 16:18 ` [U-Boot] [PATCH v2 17/19] imx: ventana: add pmic_setup to SPL Tim Harvey
2015-05-19 13:07 ` 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=555E0BF0.3040402@denx.de \
--to=sr@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox