All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] branding-processor-sdk: enable RESERVE_CMEM by default
@ 2016-10-06 17:31 Denys Dmytriyenko
  2016-10-06 18:18 ` Stiffler, Jacob
  0 siblings, 1 reply; 3+ messages in thread
From: Denys Dmytriyenko @ 2016-10-06 17:31 UTC (permalink / raw)
  To: meta-arago

From: Denys Dmytriyenko <denys@ti.com>

Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
 meta-arago-distro/conf/distro/include/branding-processor-sdk.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta-arago-distro/conf/distro/include/branding-processor-sdk.inc b/meta-arago-distro/conf/distro/include/branding-processor-sdk.inc
index bd51f76..d97518d 100644
--- a/meta-arago-distro/conf/distro/include/branding-processor-sdk.inc
+++ b/meta-arago-distro/conf/distro/include/branding-processor-sdk.inc
@@ -1,6 +1,7 @@
 ARAGO_TISDK_IMAGE = "processor-sdk-linux-image"
 
 # Set CMEM reserved memory region
+RESERVE_CMEM_omap-a15 = "1"
 CMEM_BASE_omap-a15 ?= "a0000000"
 CMEM_SIZE_omap-a15 ?= "08000000"
 
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] branding-processor-sdk: enable RESERVE_CMEM by default
  2016-10-06 17:31 [PATCH] branding-processor-sdk: enable RESERVE_CMEM by default Denys Dmytriyenko
@ 2016-10-06 18:18 ` Stiffler, Jacob
  2016-10-06 19:38   ` Denys Dmytriyenko
  0 siblings, 1 reply; 3+ messages in thread
From: Stiffler, Jacob @ 2016-10-06 18:18 UTC (permalink / raw)
  To: Denys Dmytriyenko, meta-arago

Seeing this reminds me how out of date this file has become. I'll try to send a patch by the end of next week.

But as opencl is the main consumer of CMEM, could we use that machine feature to determine whether or not to reserve memory for CMEM?

> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> Sent: Thursday, October 06, 2016 1:32 PM
> To: meta-arago@arago-project.org
> Subject: [meta-arago] [PATCH] branding-processor-sdk: enable
> RESERVE_CMEM by default
> 
> From: Denys Dmytriyenko <denys@ti.com>
> 
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
>  meta-arago-distro/conf/distro/include/branding-processor-sdk.inc | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta-arago-distro/conf/distro/include/branding-processor-
> sdk.inc b/meta-arago-distro/conf/distro/include/branding-processor-sdk.inc
> index bd51f76..d97518d 100644
> --- a/meta-arago-distro/conf/distro/include/branding-processor-sdk.inc
> +++ b/meta-arago-distro/conf/distro/include/branding-processor-sdk.inc
> @@ -1,6 +1,7 @@
>  ARAGO_TISDK_IMAGE = "processor-sdk-linux-image"
> 
>  # Set CMEM reserved memory region
> +RESERVE_CMEM_omap-a15 = "1"
>  CMEM_BASE_omap-a15 ?= "a0000000"
>  CMEM_SIZE_omap-a15 ?= "08000000"
> 
> --
> 2.7.4
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] branding-processor-sdk: enable RESERVE_CMEM by default
  2016-10-06 18:18 ` Stiffler, Jacob
@ 2016-10-06 19:38   ` Denys Dmytriyenko
  0 siblings, 0 replies; 3+ messages in thread
From: Denys Dmytriyenko @ 2016-10-06 19:38 UTC (permalink / raw)
  To: Stiffler, Jacob; +Cc: meta-arago

On Thu, Oct 06, 2016 at 06:18:15PM +0000, Stiffler, Jacob wrote:
> Seeing this reminds me how out of date this file has become. I'll try to 
> send a patch by the end of next week.

Sounds good, thanks.


> But as opencl is the main consumer of CMEM, could we use that machine 
> feature to determine whether or not to reserve memory for CMEM?

Well, I guess that's an option, as long as it stays in ProcessorSDK branding 
for now - CoreSDK still needs to build w/o CMEM enabled, but building OpenCL 
is fine, even though it won't work properly w/o CMEM...


> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > Sent: Thursday, October 06, 2016 1:32 PM
> > To: meta-arago@arago-project.org
> > Subject: [meta-arago] [PATCH] branding-processor-sdk: enable
> > RESERVE_CMEM by default
> > 
> > From: Denys Dmytriyenko <denys@ti.com>
> > 
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> >  meta-arago-distro/conf/distro/include/branding-processor-sdk.inc | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/meta-arago-distro/conf/distro/include/branding-processor-
> > sdk.inc b/meta-arago-distro/conf/distro/include/branding-processor-sdk.inc
> > index bd51f76..d97518d 100644
> > --- a/meta-arago-distro/conf/distro/include/branding-processor-sdk.inc
> > +++ b/meta-arago-distro/conf/distro/include/branding-processor-sdk.inc
> > @@ -1,6 +1,7 @@
> >  ARAGO_TISDK_IMAGE = "processor-sdk-linux-image"
> > 
> >  # Set CMEM reserved memory region
> > +RESERVE_CMEM_omap-a15 = "1"
> >  CMEM_BASE_omap-a15 ?= "a0000000"
> >  CMEM_SIZE_omap-a15 ?= "08000000"
> > 
> > --
> > 2.7.4
> > 
> > _______________________________________________
> > 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


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-10-06 19:38 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-06 17:31 [PATCH] branding-processor-sdk: enable RESERVE_CMEM by default Denys Dmytriyenko
2016-10-06 18:18 ` Stiffler, Jacob
2016-10-06 19:38   ` Denys Dmytriyenko

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.