U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 3/3] fastboot: ARM: OMAP5: Enable reboot-bootloader
Date: Fri, 27 Mar 2015 10:19:55 -0400	[thread overview]
Message-ID: <20150327141955.GG32541@bill-the-cat> (raw)
In-Reply-To: <1427236492-18032-3-git-send-email-dileep.katta@linaro.org>

On Wed, Mar 25, 2015 at 04:04:52AM +0530, Dileep Katta wrote:

> Implemented fb_set_reboot_flag() for OMAP5 to set
> an environment variable 'dofastboot' when reboot-bootloader called.
> 
> This environment variable will be checked in boot command and fastboot
> will be called if the variable is set.
> If the bootcmd env variable of OMAP5 common is overwritten with board-specific
> command, then these changes will not apply.
[snip]
> +	printf("Setting reboot to fastboot flag ...\n");
> +	setenv("dofastboot", "1");
> +	saveenv();
> +	udelay(60);
[snip]
>  #define CONFIG_BOOTCOMMAND \
> +	"if test ${dofastboot} -eq 1; then " \
> +		"echo Boot fastboot requested, resetting dofastboot ...;" \
> +		"setenv dofastboot 0; saveenv; sleep 1;" \
> +		"echo Booting into fastboot ...; fastboot;" \

No.  When saveenv returns environment is saved.  If it doesn't there's a
bug that must be addressed.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150327/04be634e/attachment.sig>

  reply	other threads:[~2015-03-27 14:19 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-03 11:10 [U-Boot] [PATCH v1 1/4] ARM: DRA7XX: Add config file for Android with fastboot support Dileep Katta
2015-03-03 11:10 ` [U-Boot] [PATCH v2 2/4] fastboot: call board_usb_init() to enable usb Dileep Katta
2015-03-04 13:19   ` Lukasz Majewski
2015-03-04 17:05   ` Stegmaier, Angela
2015-03-04 18:29     ` Tom Rini
2015-03-04 19:06       ` Dileep Katta
2015-03-05 14:37         ` Tom Rini
2015-03-12  6:36           ` Dileep Katta
2015-03-12  8:55             ` Lukasz Majewski
2015-03-12 17:28               ` Marek Vasut
2015-03-12 21:12                 ` Dileep Katta
2015-03-13  7:53                   ` Lukasz Majewski
2015-03-13 19:28                     ` Dileep Katta
2015-03-13 20:55                       ` Lukasz Majewski
2015-03-27 17:53                         ` Dileep Katta
2015-03-27 18:15                           ` Tom Rini
2015-03-27 18:25                             ` Marek Vasut
2015-03-30  8:38                               ` Lukasz Majewski
2015-03-03 11:10 ` [U-Boot] [PATCH v3 3/4] ARM: DRA7: Set serial number environment variable Dileep Katta
2015-03-04 13:23   ` Lukasz Majewski
2015-03-04 16:53   ` Tom Rini
2015-03-07  7:00   ` Nishanth Menon
2015-03-03 11:10 ` [U-Boot] [PATCH v1 4/4] ARM: DRA7: fastboot: Implement reboot-bootloader command - Implemented fb_set_reboot_flag() for DRA7 - Defined a weak function, fb_check_reboot_flag() - Implemented for DRA7 Dileep Katta
2015-03-03 14:15   ` Rob Herring
2015-03-04 16:57     ` Tom Rini
2015-03-04 13:31   ` Lukasz Majewski
2015-03-03 11:53 ` [U-Boot] [PATCH v1 1/4] ARM: DRA7XX: Add config file for Android with fastboot support Dileep Katta
2015-03-04 13:18 ` Lukasz Majewski
2015-03-04 16:52 ` Tom Rini
2015-03-12  6:31   ` Dileep Katta
2015-03-12 19:29     ` Tom Rini
2015-03-17 18:38       ` [U-Boot] [PATCH v2 1/1] " Dileep Katta
2015-03-18 16:11         ` Tom Rini
2015-03-18 21:12           ` Dileep Katta
2015-03-19 13:15             ` Tom Rini
2015-03-23 22:41               ` Dileep Katta
2015-03-24 22:34                 ` [U-Boot] [PATCH v3 1/3] ARM: DRA7XX: Enable Fastboot Dileep Katta
2015-03-24 22:34                   ` [U-Boot] [PATCH v3 2/3] ARM: DRA7: Set serial number environment variable Dileep Katta
2015-03-27 14:19                     ` Tom Rini
2015-04-23 22:03                     ` [U-Boot] [U-Boot, v3, " Tom Rini
2015-03-24 22:34                   ` [U-Boot] [PATCH v3 3/3] fastboot: ARM: OMAP5: Enable reboot-bootloader Dileep Katta
2015-03-27 14:19                     ` Tom Rini [this message]
2015-03-27 17:28                       ` Dileep Katta
2015-03-27 17:36                         ` [U-Boot] [PATCH v4 " Dileep Katta
2015-03-27 18:02                           ` Tom Rini
2015-04-23 22:03                           ` [U-Boot] [U-Boot, v4, " Tom Rini
2015-03-27 14:19                   ` [U-Boot] [PATCH v3 1/3] ARM: DRA7XX: Enable Fastboot Tom Rini
2015-04-23 22:03                   ` [U-Boot] [U-Boot,v3,1/3] " Tom Rini
2015-03-23  8:50         ` [U-Boot] [PATCH v2 1/1] ARM: DRA7XX: Add config file for Android with fastboot support Lukasz Majewski

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=20150327141955.GG32541@bill-the-cat \
    --to=trini@konsulko.com \
    --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