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
prev parent 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.