[U-Boot] [PATCH] mkimage: add option for adding dtc binary path via argument

Simon Glass sjg at chromium.org
Wed Aug 14 20:26:34 CEST 2013


Hi,

On Wed, Aug 14, 2013 at 10:58 AM, Harvey Chapman <hchapman at 3gfp.com> wrote:
> On May 6, 2013, at 8:59 AM, Luka Perkov <luka at openwrt.org> wrote:
>
>> On Mon, May 06, 2013 at 02:32:51PM +0200, Wolfgang Denk wrote:
>>> It appears there is no really good reason for this patch, so I think
>>> we should drop it.
>>
>> Ok. Thanks for the review.
>
> Would there be any objection to adding an option to pass source paths to dtc from mkimage? Currently, in order to do that, I have to find the source for mkimage, grab the default dtc options out of mkimage.h and then pass that set on the command line. It would be nice and only slightly more future proof if I could just pass the source file search paths separately.

So you mean the -i option? I think that would be valuable, yes.
Without it you need to put all your files in one directory, or
maintain the same directory structure in your release directory. It
gets really ugly. The -i option was added to deal with this and I
think it makes sense to make it available in mkimage.

>
>
>  local t=${VSD_PATH}/app/flasher/uboot_flasher
>  local t2=${t}/tools
>  PATH=${t2}:$PATH ${t2}/mkimage -D "-I dts -O dtb -p 500 -i ${release_dir}" \
>                                 -f ${t}/update_field_mode.its spot_update.bin
>
> I'd like to pass one or more paths like "${release_dir}" without having to pass (or even know about) "-I dts -O dtb -p 500".
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

Regards,
Simon


More information about the U-Boot mailing list