On Wed, Jun 12, 2013 at 05:30:03PM -0600, Stephen Warren wrote:
[...]
> diff --git a/tegra-uboot-flasher b/tegra-uboot-flasher
> index 447ff37..ef73e2f 100755
> --- a/tegra-uboot-flasher
> +++ b/tegra-uboot-flasher
> @@ -75,6 +75,17 @@ gen_flashcmds = {
> 'spi': gen_flashcmd_spi,
> }
>
> +def find_config_dir():
> + if not configs.has_key(args.configname):
> + print 'Unknown config "%s"' % args.configname
> + sys.exit(1)
> +
> + global boardname, socname, out_board_dir
Python supports returning touples, so maybe globals can be avoided here?
> +
> + boardname = configs[args.configname]['board']
> + socname = boards[boardname]['soc']
> + out_board_dir = os.path.join(args.data_dir, boardname)
> +
> parser = argparse.ArgumentParser(description='Write an image to a Tegra
> board\'s flash')
> parser.add_argument('--debug', action='store_true',
> help='Turn on debugging prints')
> @@ -122,14 +133,7 @@ if args.list_confignames:
> print configname
> sys.exit(0)
>
> -if not configs.has_key(args.configname):
> - print 'Unknown config "%s"' % args.configname
> - sys.exit(1)
> -
> -boardname = configs[args.configname]['board']
> -socname = boards[boardname]['soc']
> -
> -out_board_dir = os.path.join(args.data_dir, boardname)
> +find_config_dir()
So this'll turn into something like:
boardname, socname, out_board_dir = find_config_dir()
Thierry
pgpKVUwZnWub4.pgp
Description: PGP signature
