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

[Denys Dmytriyenko <denys@ti.com>]

On Thu, Nov 07, 2013 at 09:00:08PM -0500, Tivy, Robert wrote:
> Please see one response below...
> 
> Regards,
> 
> - Rob
> 
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Thursday, November 07, 2013 5:53 PM
> > To: Nelson, Sam
> > Cc: Maupin, Chase; meta-arago@arago-project.org; Ring, Chris; Tivy,
> > Robert
> > Subject: Re: [meta-arago] [PATCH v2] cmem: Adding New recipe for cmem
> > 
> > 
> > > > >+}
> > > > >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"
> > > >
> > > > As Sidd pointed out is there any reason not to add the other
> > devices like
> > > > omap-a15 as compatible machines?
> > >
> > > [Sam] I am fine with remove the compatible machine. Although this is
> > not
> > > tested with other machines.
> > 
> > So, Chris and Rob should comment on which platforms this version of
> > cmem is
> > supposed to be compatible with, even if it's not tested yet.
> > 
> 
> CMEM is completely architecture-independent.  The only HW it touches is 
> memory, and the memory it touches is specified by the user/system 
> integrator.

Thanks. Then it doesn't make sense to artificially limit it to Keystone only.

-- 
Denys