From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from arroyo.ext.ti.com (arroyo.ext.ti.com [192.94.94.40]) by arago-project.org (Postfix) with ESMTPS id C4C1652972 for ; Wed, 6 Nov 2013 21:13:08 +0000 (UTC) Received: from dflxv15.itg.ti.com ([128.247.5.124]) by arroyo.ext.ti.com (8.13.7/8.13.7) with ESMTP id rA6LD7Dj007346 for ; Wed, 6 Nov 2013 15:13:07 -0600 Received: from DFLE72.ent.ti.com (dfle72.ent.ti.com [128.247.5.109]) by dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id rA6LD7ja030122 for ; Wed, 6 Nov 2013 15:13:07 -0600 Received: from dflp33.itg.ti.com (10.64.6.16) by DFLE72.ent.ti.com (128.247.5.109) with Microsoft SMTP Server id 14.2.342.3; Wed, 6 Nov 2013 15:13:07 -0600 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp33.itg.ti.com (8.14.3/8.13.8) with ESMTP id rA6LD77Q019027; Wed, 6 Nov 2013 15:13:07 -0600 Date: Wed, 6 Nov 2013 16:13:06 -0500 From: Denys Dmytriyenko To: "Nelson, Sam" Message-ID: <20131106211306.GB4510@edge> References: <1383160055-32615-1-git-send-email-sam.nelson@ti.com> <5271DA2C.8070401@ti.com> <7D46E86EC0A8354091174257B2FED101597EF49C@DLEE11.ent.ti.com> <5274A42E.1020405@ti.com> <20131104170604.GT24696@edge> <4F9216F96E5DF9428610502835A9F49054988496@DLEE11.ent.ti.com> MIME-Version: 1.0 In-Reply-To: <4F9216F96E5DF9428610502835A9F49054988496@DLEE11.ent.ti.com> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "meta-arago@arago-project.org" 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: Wed, 06 Nov 2013 21:13:09 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Wed, Nov 06, 2013 at 10:53:08AM -0500, Nelson, Sam wrote: > > > > -----Original Message----- > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago- > > bounces@arago-project.org] On Behalf Of Dmytriyenko, Denys > > Sent: Monday, November 04, 2013 12:06 PM > > To: Heroor, Siddharth > > Cc: meta-arago@arago-project.org > > Subject: Re: [meta-arago] [PATCH 1/1] ti-cmem: New recipes for ti-cmem > > > > On Sat, Nov 02, 2013 at 12:35:18PM +0530, Siddharth Heroor wrote: > > > 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. > > > > Sid, > > > > I agree with your suggestions as long as this component is common and > > being > > reused by both MCSDK and GLSDK teams, as well as others. > > > > What I don't want to happen is to unify on a single recipe and later find out > > that there are so many different requirements and nuances, that none of the > > teams can use the recipe directly and need to override and customize most > > of > > it to their needs... > [Sam] I have discussed with the TISB team who owns the linuxutils. Moving > forward it makes sense to use the cmem recipes to be used. (Although it is a > component of linuxutils). Does it still makes sense to move to meta-ti? So, to rephrase - cmem is one of the sub-components of linuxutils, and linuxutils is used to be distributed and built as a single entity, even though it was a collection of smaller sub-components or utils. And moving forward, the component owner is now fine with splitting it up and having individual recipes for each sub-component. In that case it still makes sense to move it to meta-ti. I may need to start looking into restructuring recipes-ti/ directory (where linuxutils now lives) and figure out better places for things in there, as recipes-bsp is probably a better fit at least for cmem... > > So, essentially, what I'm looking for is a confirmation from teams (at least > > Sam and Sid/Mrinmayee, maybe others) that ipc/ludev.git is _the_ repository > > with the latest code for the cmem/linuxutils component that is being used > > from > > a common (master?) branch. Thank you. > [Sam] Yes. ludev.git will be the place all future releases of cmem will be > delivered. As long as it is shared between teams, it's fine with me - we don't want several different implementations of cmem. Denys > > > >>>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 > > > > > > _______________________________________________ > > > meta-arago mailing list > > > meta-arago@arago-project.org > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago > > _______________________________________________ > > meta-arago mailing list > > meta-arago@arago-project.org > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago