Re: [PATCH] tisdk-image: add support for DTB files

[Denys Dmytriyenko <denys@ti.com>]

On Wed, Apr 03, 2013 at 11:48:30AM -0500, Chase Maupin wrote:
> * If DTB files exist add support for copying those files into
>   the SDK prebuilt-images directory.
> * The files to be copied can be controlled using the DTB_FILTER
>   variable.
> 
> Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
> ---
>  meta-arago-distro/classes/tisdk-image.bbclass |   10 ++++++++++
>  1 files changed, 10 insertions(+), 0 deletions(-)
> 
> diff --git a/meta-arago-distro/classes/tisdk-image.bbclass b/meta-arago-distro/classes/tisdk-image.bbclass
> index a1ce1ad..4e3ed52 100644
> --- a/meta-arago-distro/classes/tisdk-image.bbclass
> +++ b/meta-arago-distro/classes/tisdk-image.bbclass
> @@ -528,6 +528,16 @@ do_sdk_image () {
>          return 1
>      fi

I was thinking of adding DTB_FILTER ?= "unknown" here, but then realized it 
would just waste few cycles trying to find something in the deploy directory, 
which we can avoid completely by testing if it set or not...


> +    # Copy the DTB files if they exist.
> +    # NOTE: For simplicity remove the uImage- prefix on the dtb files.  Get just the symlink
> +    #       files for a cleaner name.  Use the DTB_FILTER variable to allow finding the
> +    #       dtb files for only that MACHINE type

Wrap the below for loop in "if [ -z "${DTB_FILTER}" ]; then" here?


> +    for f in `find ${DEPLOY_DIR_IMAGE} -type l -name "*${DTB_FILTER}*.dtb"`
> +    do
> +        dtb_file=`basename $f | sed s/uImage-//`
> +        cp $f ${prebuilt_dir}/${dtb_file}
> +    done
> +
>      if [ "${SECONDARY_BOOTLOADER_NAME}" != "" ]
>      then
>          # Copy the secondary bootloader image if it exists
> -- 
> 1.7.0.4
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>