Re: [PATCH tegra-uboot-scripts 2/2] Make folder for dtb files as an argument

[Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>]

On 07/23/2014 02:27 PM, Stefan Agner wrote:
> Some distribution use different folders to store the dtb files
> rather then storing them directly in /boot. This adds a new
> argument to define which folder U-Boot should search the dtb
> file.

> diff --git a/gen-uboot-script.py b/gen-uboot-script.py

> +parser.add_argument('--dtb-dir', dest='dtbdir', default='/boot',
> +    help='Search directory for dtbs on target; defaults to /boot')

I'd prefer the default to be None rather that a hard-coded path here.

That way, after the following code that already exists:

if args.type == 'net':
    f.write('dhcp\n')
    load='tftpboot'
    prefix=''
else:
    load='load ${devtype} ${devnum}:${rootpart}'
    prefix='/boot/'

... we can add something like:

if not args.dtbdir:
    args.dtbdir = prefix

... which will exactly preserve the existing behaviour in the
network-boot case.

> -f.write(load + ' ${fdt_addr_r} ' + prefix + '${_fdt}\n')
> +f.write(load + ' ${fdt_addr_r} ' + args.dtbdir + '/${_fdt}\n')
>  f.write('set _fdt\n')

As an aside, I guess I should write a gen-extlinux-conf.py too!