Re: [PATCH v2] cmem: Adding New recipe for cmem

[Denys Dmytriyenko <denys@ti.com>]

On Wed, Nov 20, 2013 at 01:11:18PM +0000, Maupin, Chase wrote:
> Sam,
> 
> Were you going to post an updated set of these patches to meta-ti so you can 
> align with Mrinmayee? I think your approach of splitting out the packages in 
> linux-utils is a good one and will be cleaner going forward.  Since both of 
> your groups share these packages aligning between the two would be helpful.
> 
> As a side note I assume you have other recipes that depend on the cmem 
> libraries that your cmem recipe provides.  Can you post one of those or 
> share it with Mrinmayee as an example of how you are able to point to the 
> right staging location for the libraries and headers for cmem in other 
> recipes.  I'm hoping we can start getting rid of ti-paths.inc but I'd like 
> to see how you were working around this.
> 
> Denys,
> 
> Since ti-linuxutils has other modules in it for edma, sdma, and irq but 
> those are only for legacy devices can we just replace the whole 
> ti-linuxutils with the cmem package which seems to be the main package 
> needed?  Or do you want to see additional recipes for the sdma, edma, and 
> irq moduled?

I'd like to see other parts of linuxutils modularized, but that can be done on 
as needed basis for any of the new platforms when they require those. We can 
keep the old linuxutils around for a little longer, but it can only be used on 
the old 35x and 37x plaftorms only and there are many other components (like 
SGX) removing support for those old platforms already...

-- 
Denys


> >-----Original Message-----
> >From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> >bounces@arago-project.org] On Behalf Of Nelson, Sam
> >Sent: Wednesday, November 06, 2013 11:04 PM
> >To: meta-arago@arago-project.org; Nelson, Sam
> >Subject: [meta-arago] [PATCH v2] cmem: Adding New recipe for cmem
> >
> >- Cmem module is used for user space contiguous memory alloation
> >
> >Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> >---
> > recipes-bsp/cmem/cmem-mod_git.bb |   26
> >++++++++++++++++++++++++++
> > recipes-bsp/cmem/cmem.inc        |   12 ++++++++++++
> > recipes-bsp/cmem/cmem_git.bb     |   16 ++++++++++++++++
> > 3 files changed, 54 insertions(+)
> > create mode 100644 recipes-bsp/cmem/cmem-mod_git.bb
> > create mode 100644 recipes-bsp/cmem/cmem.inc
> > create mode 100644 recipes-bsp/cmem/cmem_git.bb
> >
> >diff --git a/recipes-bsp/cmem/cmem-mod_git.bb b/recipes-
> >bsp/cmem/cmem-mod_git.bb
> >new file mode 100644
> >index 0000000..c402b72
> >--- /dev/null
> >+++ b/recipes-bsp/cmem/cmem-mod_git.bb
> >@@ -0,0 +1,26 @@
> >+DESCRIPTION="Contiguous memory allocation kernel module for
> >contiguous memory allocation from userspace "
> >+
> >+include cmem.inc
> >+
> >+NAME="cmem"
> >+
> >+# 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}"
> >+
> >+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}"
> >+}
> >diff --git a/recipes-bsp/cmem/cmem.inc b/recipes-bsp/cmem/cmem.inc
> >new file mode 100644
> >index 0000000..5d17d8e
> >--- /dev/null
> >+++ b/recipes-bsp/cmem/cmem.inc
> >@@ -0,0 +1,12 @@
> >+HOMEPAGE="http://processors.wiki.ti.com/index.php/Category:CMEM"
> >+
> >+LICENSE = "GPLv2"
> >+LIC_FILES_CHKSUM =
> >"file://include/ti/cmem.h;beginline=1;endline=30;md5=b86138d4028fb
> >8310b3b983024edc620"
> >+
> >+COMPATIBLE_MACHINE = "keystone-evm"
> >+
> >+BRANCH ?= "master"
> >+#SRCREV = "4.00.00.06"
> >+SRCREV = "c2cf2406702bcb889c10d0e6e7d298ba7b84ae7d"
> >+
> >+SRC_URI =
> >"git://git.ti.com/ipc/ludev.git;protocol=git;branch=${BRANCH}"
> >diff --git a/recipes-bsp/cmem/cmem_git.bb b/recipes-
> >bsp/cmem/cmem_git.bb
> >new file mode 100644
> >index 0000000..f415630
> >--- /dev/null
> >+++ b/recipes-bsp/cmem/cmem_git.bb
> >@@ -0,0 +1,16 @@
> >+DESCRIPTION="The cmem component supports contiguous memory
> >allocation from userspace "
> >+
> >+include cmem.inc
> >+
> >+RDEPENDS_${PN} = "cmem-mod"
> >+
> >+S = "${WORKDIR}/git"
> >+
> >+PR = "r0"
> >+
> >+PACKAGES =+ "${PN}-test"
> >+
> >+FILES_${PN}-test = "${bindir}/*"
> >+
> >+inherit autotools
> >+
> >--
> >1.7.9.5
> >
> >_______________________________________________
> >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