[flasher PATCH] Add a command to specify post flash u-boot command

[flasher PATCH] Add a command to specify post flash u-boot command

[James Thomas]

Simple patch to allow a post-flash-cmd to be used after flashing is complete,
e.g --post-flash-cmd 'ums 0 mmc 0'

This defaults to 'reset' if not passed

James Thomas (1):
  Add a --post-flash-cmd command line argument

 tegra-uboot-flasher | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

-- 
1.9.3

[flasher PATCH] Add a --post-flash-cmd command line argument

[James Thomas]

This allows a user to specify the u-boot command to run after flashing
is complete (e.g 'ums 0 mmc 0'), defaults to 'reset' if not set
---
 tegra-uboot-flasher | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/tegra-uboot-flasher b/tegra-uboot-flasher
index b71f967..39a077a 100755
--- a/tegra-uboot-flasher
+++ b/tegra-uboot-flasher
@@ -218,7 +218,10 @@ def func_flash():
         bootcmd += 'echo >>> Flashing OK, rebooting... ; '
         # To update the bootloader, reset.
         # If wanting to run installer, set installer_args.configname in environment, 'run bootcmd'
-        bootcmd += 'reset'
+        if args.post_flash_cmd:
+            bootcmd += args.post_flash_cmd
+        else:
+            bootcmd += 'reset'
         print 'bootcmd:', bootcmd
         cmd = ['fdtput', '-p', '-t', 's', u_boot_dtb_runflash, '/config', 'bootcmd', bootcmd]
         run(workdir, cmd)
@@ -307,6 +310,8 @@ parser_flash.add_argument('--env', type=str, nargs=2, action='append',
     metavar=("VAR", "VALUE"),
     help='Set a U-Boot environment variable after flashing. This option ' +
         'may be given multiple times')
+parser_flash.add_argument('--post-flash-cmd', type=str,
+    help='The U-Boot command to run after flashing, defaults to \'reset\'')
 
 parser_exec = subparsers.add_parser('exec',
     help='Download and execute an unmodified bootloader binary, named ' +
-- 
1.9.3

Re: [flasher PATCH] Add a --post-flash-cmd command line argument

[Stephen Warren]

On 11/25/2014 03:14 PM, James Thomas wrote:
> This allows a user to specify the u-boot command to run after flashing
> is complete (e.g 'ums 0 mmc 0'), defaults to 'reset' if not set

You need to add a Signed-off-by line here to indicate that you're 
willing to license the patch according to the license at the top of the 
tegra-uboot-flasher file. We use the s-o-b line in the same way as e.g. 
the Linux kernel, although the actual license of the file is different. See:

http://developercertificate.org/