diff mbox

[tegra-uboot-scripts,v2] Make folder for dtb files as an argument

Message ID aaa2e78cfda2a2afe91e9ac8dca4583e8a860123.1406238873.git.stefan@agner.ch
State Superseded, archived
Headers show

Commit Message

Stefan Agner July 24, 2014, 9:55 p.m. UTC
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.

Signed-off-by: Stefan Agner <stefan@agner.ch>
---
 gen-uboot-script.py | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

Comments

Stephen Warren July 25, 2014, 4:05 p.m. UTC | #1
On 07/24/2014 03:55 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

> @@ -107,7 +110,10 @@ if args.type == 'net':
>       prefix=''
>   else:
>       load='load ${devtype} ${devnum}:${rootpart}'
> -    prefix='/boot/'
> +    if not args.dtbdir:
> +        prefix='/boot/'
> +    else:
> +        prefix=args.dtbdir+'/'
>
>   f.write(load + ' ${kernel_addr_r} ' + prefix + 'zImage\n')

I don't think this is right; it makes --dtbdir affect the value of the 
prefix variable, which is used for all of zImage, DTBs, and initrd. I 
didn't think that's what you wanted, although if it is, then (a) the 
option doesn't have the right name, and (b), it should presumably affect 
network mode too.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/gen-uboot-script.py b/gen-uboot-script.py
index 8ca3d01..47de520 100755
--- a/gen-uboot-script.py
+++ b/gen-uboot-script.py
@@ -71,6 +71,9 @@  parser.add_argument('--noisy', action='store_true',
 parser.add_argument('--cmdline',
     help='Extra command-line options')
 
+parser.add_argument('--dtb-dir', dest='dtbdir', default=None,
+    help='Search directory for dtbs on target; defaults to /boot')
+
 args = parser.parse_args()
 if args.debug: print args
 
@@ -107,7 +110,10 @@  if args.type == 'net':
     prefix=''
 else:
     load='load ${devtype} ${devnum}:${rootpart}'
-    prefix='/boot/'
+    if not args.dtbdir:
+        prefix='/boot/'
+    else:
+        prefix=args.dtbdir+'/'
 
 f.write(load + ' ${kernel_addr_r} ' + prefix + 'zImage\n')