public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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