All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: Sam Nelson <sam.nelson@ti.com>
Cc: meta-ti@yoctoproject.org
Subject: Re: [PATCH v2 2/2] rm-lld: Rename recipe and update to new version
Date: Wed, 8 Oct 2014 13:46:18 -0400	[thread overview]
Message-ID: <20141008174618.GQ1731@edge> (raw)
In-Reply-To: <1412764049-7074-2-git-send-email-sam.nelson@ti.com>

Sam,

Thanks for addressing all the comments! This looks good and I'll push it to 
meta-ti today.

-- 
Denys


On Wed, Oct 08, 2014 at 06:27:29AM -0400, Sam Nelson wrote:
> - Removed ti-prefix based on review
> - Update to new version 2.1.0.6
> - Add dependency on libdaemon
> - Remove device dependency
> - Updated license to BSD-3-Clause
> 
> Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> ---
> Changes from previous patch
> - Combined two commits into one
> - Reset PR as PV is updated
> - Updated comments: Updated license to BSD-3-Clause
> ---
> ---
>  recipes-bsp/rm-lld/{ti-rm_git.bb => rm-lld_git.bb} |   24 +++++++-------------
>  1 file changed, 8 insertions(+), 16 deletions(-)
>  rename recipes-bsp/rm-lld/{ti-rm_git.bb => rm-lld_git.bb} (56%)
> 
> diff --git a/recipes-bsp/rm-lld/ti-rm_git.bb b/recipes-bsp/rm-lld/rm-lld_git.bb
> similarity index 56%
> rename from recipes-bsp/rm-lld/ti-rm_git.bb
> rename to recipes-bsp/rm-lld/rm-lld_git.bb
> index 276c01e..ed34478 100644
> --- a/recipes-bsp/rm-lld/ti-rm_git.bb
> +++ b/recipes-bsp/rm-lld/rm-lld_git.bb
> @@ -1,17 +1,17 @@
>  DESCRIPTION = "TI Resource Manager Low Level Driver"
> -LICENSE = "TI BSD"
> +LICENSE = "BSD-3-Clause"
>  LIC_FILES_CHKSUM = "file://${WORKDIR}/git/ti/drv/rm/COPYING.txt;md5=dc61631b65360e6beb73b6c337800afc"
>  
>  BRANCH="master"
>  SRC_URI = "git://git.ti.com/keystone-rtos/rm-lld.git;destsuffix=git/ti/drv/rm;protocol=git;branch=${BRANCH}"
> -# Below commit ID corresponds to DEV.RM_LLD.02.01.00.04
> -SRCREV = "59e3bf5fca3995dd0f79bbdd1af988d278a2a049"
> +# Below commit ID corresponds to DEV.RM_LLD.02.01.00.06
> +SRCREV = "dc0dba4a6b0358efcb861d50b3dae21beb5281a8"
> +PV = "02.01.00.06"
>  PR = "r0"
> -PV = "02.01.00.04"
>  
>  COMPATIBLE_MACHINE = "keystone"
>  
> -DEPENDS = "ti-ipc"
> +DEPENDS = "ti-ipc libdaemon"
>  
>  PACKAGES =+ "${PN}-test"
>  
> @@ -19,8 +19,6 @@ FILES_${PN}-test = "${bindir}/rmDspClientTest_*.out \
>                      ${bindir}/rmLinuxClientTest_*.out \
>                      ${bindir}/ti/drv/rm/test/dts_files/*.dtb"
>  
> -DEVICELIST = "k2h k2k"
> -
>  CHOICELIST = "yes no"
>  
>  BASEDIR = "${WORKDIR}/git"
> @@ -29,12 +27,9 @@ S = "${BASEDIR}/ti/drv/rm"
>  do_compile () {
>  #   Now build the lld
>      make -f makefile_armv7 clean lib PDK_INSTALL_PATH=${STAGING_INCDIR} RM_SRC_DIR=${S}
> -    for device in ${DEVICELIST}
> +    for choice in ${CHOICELIST}
>      do
> -      for choice in ${CHOICELIST}
> -      do
> -        make -f makefile_armv7 tests IPC_DEVKIT_INSTALL_PATH=${STAGING_INCDIR} PDK_INSTALL_PATH=${BASEDIR} DEVICE="$device" USEDYNAMIC_LIB="$choice"
> -      done
> +      make -f makefile_armv7 tests IPC_DEVKIT_INSTALL_PATH=${STAGING_INCDIR} PDK_INSTALL_PATH=${BASEDIR} USEDYNAMIC_LIB="$choice"
>      done
>  }
>  
> @@ -42,8 +37,5 @@ do_install () {
>      install -d ${D}${includedir}/ti/drv/rm
>      install -d ${D}${libdir}
>      install -d ${D}${bindir}
> -    for device in ${DEVICELIST}
> -    do
> -      make -f makefile_armv7 install installbin installbin_test INSTALL_INC_BASE_DIR=${D}${includedir} INSTALL_LIB_BASE_DIR=${D}${libdir} INSTALL_BIN_BASE_DIR=${D}${bindir} DEVICE="$device"
> -    done
> +    make -f makefile_armv7 install installbin installbin_test INSTALL_INC_BASE_DIR=${D}${includedir} INSTALL_LIB_BASE_DIR=${D}${libdir} INSTALL_BIN_BASE_DIR=${D}${bindir}
>  }
> -- 
> 1.7.9.5
> 
> -- 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti


      reply	other threads:[~2014-10-08 17:46 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-08 10:27 [PATCH v2 1/2] ti-rm: Move from meta-arago to meta-ti Sam Nelson
2014-10-08 10:27 ` [PATCH v2 2/2] rm-lld: Rename recipe and update to new version Sam Nelson
2014-10-08 17:46   ` Denys Dmytriyenko [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141008174618.GQ1731@edge \
    --to=denys@ti.com \
    --cc=meta-ti@yoctoproject.org \
    --cc=sam.nelson@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.