From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bear.ext.ti.com (bear.ext.ti.com [192.94.94.41]) by arago-project.org (Postfix) with ESMTPS id 9298852972 for ; Sat, 2 Nov 2013 07:05:21 +0000 (UTC) Received: from dflxv15.itg.ti.com ([128.247.5.124]) by bear.ext.ti.com (8.13.7/8.13.7) with ESMTP id rA275KYm000573 for ; Sat, 2 Nov 2013 02:05:20 -0500 Received: from DFLE73.ent.ti.com ([128.247.5.110]) by dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id rA275KdM023474 for ; Sat, 2 Nov 2013 02:05:20 -0500 Received: from dflp32.itg.ti.com (10.64.6.15) by DFLE73.ent.ti.com (128.247.5.110) with Microsoft SMTP Server id 14.2.342.3; Sat, 2 Nov 2013 02:05:20 -0500 Received: from [172.24.158.58] (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp32.itg.ti.com (8.14.3/8.13.8) with ESMTP id rA275Ik3008835; Sat, 2 Nov 2013 02:05:19 -0500 Message-ID: <5274A42E.1020405@ti.com> Date: Sat, 2 Nov 2013 12:35:18 +0530 From: Siddharth Heroor User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.0 MIME-Version: 1.0 To: "Maupin, Chase" , "meta-arago@arago-project.org" References: <1383160055-32615-1-git-send-email-sam.nelson@ti.com> <5271DA2C.8070401@ti.com> <7D46E86EC0A8354091174257B2FED101597EF49C@DLEE11.ent.ti.com> In-Reply-To: <7D46E86EC0A8354091174257B2FED101597EF49C@DLEE11.ent.ti.com> Subject: Re: [PATCH 1/1] ti-cmem: New recipes for ti-cmem X-BeenThere: meta-arago@arago-project.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Arago metadata layer for TI SDKs - OE-Core/Yocto compatible List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 02 Nov 2013 07:05:21 -0000 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit On 10/31/2013 7:11 PM, Maupin, Chase wrote: >> -----Original Message----- >> From: meta-arago-bounces@arago-project.org [mailto:meta-arago- >> bounces@arago-project.org] On Behalf Of Heroor, Siddharth >> Sent: Wednesday, October 30, 2013 11:19 PM >> To: meta-arago@arago-project.org >> Subject: Re: [meta-arago] [PATCH 1/1] ti-cmem: New recipes for ti- >> cmem >> >> On 10/31/2013 12:37 AM, Sam Nelson wrote: >>> - Recipes for ti-cmem which provides user space api for >> contigous memory allocaion >>> - ti-cmem_mod provides the kernel module >>> - ti-mem provides the user space library >> Why do we need to create a new recipe? The ti-linuxutils in meta- >> ti >> should be the provider for cmem. Shouldn't that be updated for >> newer >> versions? >> Also, I would suggest that we make it machine compatible with >> omap-a15. > I don't know that linuxutils is really the right solution. I think it mainly comes down to where/how is cmem being released. So really the question is whether the SRC_URI used below is the official cmem release point or is the SRC_URI from linuxutils right? We do not need multiple versions of cmem where everyone has their own fork of the cmem code. > > Another point is shouldn't this be in meta-ti for at least the cmem module? ludev in the SRC_URI stands for linuxutils dev. The SRC_URI in the linuxutils is a tarball release. One option is for us to drop the tarball release from SRC_URI and use the git tree going forward. I'm in favour of this primarily because that's where the development happens and the tree is tagged. On meta-arago vs. meta-ti - I agree that the recipe should be in meta-ti. This patch should update linuxutils accordingly. > >>> Signed-off-by: Sam Nelson >>> --- >>> .../recipes-bsp/ti-cmem/ti-cmem-mod.bb | 40 >> ++++++++++++++++++++ >>> meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem.bb | 20 >> ++++++++++ >>> meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem.inc | 10 >> +++++ >>> 3 files changed, 70 insertions(+) >>> create mode 100755 meta-arago-extras/recipes-bsp/ti-cmem/ti- >> cmem-mod.bb >>> create mode 100644 meta-arago-extras/recipes-bsp/ti-cmem/ti- >> cmem.bb >>> create mode 100644 meta-arago-extras/recipes-bsp/ti-cmem/ti- >> cmem.inc >>> diff --git a/meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem- >> mod.bb b/meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem-mod.bb >>> new file mode 100755 >>> index 0000000..0839c20 >>> --- /dev/null >>> +++ b/meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem-mod.bb >>> @@ -0,0 +1,40 @@ >>> +DESCRIPTION="The cmem component module for contiguous memory >> allocation from userspace " >>> + >>> >> +HOMEPAGE="http://processors.wiki.ti.com/index.php/Category:CMEM" >>> + >>> +LICENSE = "GPLv2" >>> + >>> +include ti-cmem.inc >>> + >>> +RDEPENDS_${PN} += "kernel (=${KERNEL_VERSION})" >>> + >>> +DEPENDS += "virtual/kernel" >>> + >>> +NAME="cmem" >>> +MODULE_BUILD_DIR ="src/cmem/module" >>> + >>> +# This package builds a kernel module, use kernel PR as base >> and append a local >>> +PR = "${MACHINE_KERNEL_PR}" >>> +PR_append = "a" >>> + >>> +PKG_${PN} = "kernel-module-${PN}" >>> + >>> +KERNEL_VERSION = >> "${@base_read_file('${STAGING_KERNEL_DIR}/kernel-abiversion')}" >>> + >>> +S = "${WORKDIR}/git" >>> + >>> +inherit module >>> + >>> +do_compile () { >>> + cd ${S} >>> + make -f lu.mak module_clean >>> + make -f lu.mak module >> KERNEL_INSTALL_DIR=${STAGING_KERNEL_DIR} >> TOOLCHAIN_PREFIX=${CROSS_COMPILE} >>> +} >>> + >>> +do_install () { >>> + cd ${S} >>> + make -f lu.mak module_install >> KERNEL_INSTALL_DIR="${STAGING_KERNEL_DIR}" >> EXEC_DIR="${D}/lib/modules/${KERNEL_VERSION}/extra" >> INSTALL_MOD_PATH="${D}" >>> +} >>> + >>> +PACKAGE_STRIP = "no" >>> + >>> diff --git a/meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem.bb >> b/meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem.bb >>> new file mode 100644 >>> index 0000000..a106aa0 >>> --- /dev/null >>> +++ b/meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem.bb >>> @@ -0,0 +1,20 @@ >>> +DESCRIPTION="The cmem component supports contiguous memory >> allocation from userspace " >>> + >>> >> +HOMEPAGE="http://processors.wiki.ti.com/index.php/Category:CMEM" >>> + >>> +LICENSE = "GPLv2" >>> + >>> +include ti-cmem.inc >>> + >>> +DEPENDS = "ti-cmem-mod" >>> + >>> +S = "${WORKDIR}/git" >>> + >>> +PR = "r1" >>> + >>> +PACKAGES =+ "${PN}-test" >>> + >>> +FILES_${PN}-test = "${bindir}/*" >>> + >>> +inherit autotools >>> + >>> diff --git a/meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem.inc >> b/meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem.inc >>> new file mode 100644 >>> index 0000000..9308b2d >>> --- /dev/null >>> +++ b/meta-arago-extras/recipes-bsp/ti-cmem/ti-cmem.inc >>> @@ -0,0 +1,10 @@ >>> >> +HOMEPAGE="http://processors.wiki.ti.com/index.php/Category:CMEM" >>> + >>> +LIC_FILES_CHKSUM = >> "file://products.mak;beginline=2;endline=30;md5=195feadf798bb4165b >> cb1a23ffd50dbb" >>> + >>> +COMPATIBLE_MACHINE = "keystone-evm" >>> + >>> +BRANCH ?= "master" >>> +SRCREV = "4.00.00.06" >>> + >>> +SRC_URI = >> "git://git.ti.com/ipc/ludev.git;protocol=git;branch=${BRANCH}" >> >> _______________________________________________ >> meta-arago mailing list >> meta-arago@arago-project.org >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago