Re: [PATCH 3/3] davinci: add SRAM allocator

2009-05-22 Thread Kevin Hilman
Ring, Chris cr...@ti.com writes: Just a quick can we talk through this patch a bit note... I _think_ there are existing codecs on some DaVinci platforms (e.g. DM365) that are already using this SRAM memory. The current [easy] solution to their use case was to keep this memory out of the

RE: [PATCH 3/3] davinci: add SRAM allocator

2009-05-22 Thread Tivy, Robert
Brownell; linux-arm-ker...@lists.arm.linux.org.uk Subject: Re: [PATCH 3/3] davinci: add SRAM allocator Ring, Chris cr...@ti.com writes: Just a quick can we talk through this patch a bit note... I _think_ there are existing codecs on some DaVinci platforms (e.g. DM365) that are already

Re: [PATCH 3/3] davinci: add SRAM allocator

2009-05-22 Thread David Brownell
On Friday 22 May 2009, Tivy, Robert wrote: I think the way codecs are going to request SRAM is by asking for all of it, Yeech. I hope that's on the list of bugs to fix! which means that if the audio driver takes any of it through sram_alloc(), codecs won't have any.  And if codecs take

RE: [PATCH 3/3] davinci: add SRAM allocator

2009-05-22 Thread Ring, Chris
-Original Message- From: David Brownell [mailto:davi...@pacbell.net] On Friday 22 May 2009, Tivy, Robert wrote: I think the way codecs are going to request SRAM is by asking for all of it, Yeech. I hope that's on the list of bugs to fix! AFAIK, it's not, nor is it a bug.

Re: [PATCH 3/3] davinci: add SRAM allocator

2009-05-22 Thread Kevin Hilman
Ring, Chris cr...@ti.com writes: -Original Message- From: David Brownell [mailto:davi...@pacbell.net] On Friday 22 May 2009, Tivy, Robert wrote: I think the way codecs are going to request SRAM is by asking for all of it, Yeech. I hope that's on the list of bugs to fix!

RE: [PATCH 3/3] davinci: add SRAM allocator

2009-05-22 Thread Ring, Chris
-Original Message- From: Kevin Hilman [mailto:khil...@deeprootsystems.com] All of this discussion isn't relevant to the need for an in-kernel SRAM allocator upstream. What is being discussed here is various ways of using or not using it. IMHO, this discussion isn't relevant to

Re: [PATCH 3/3] davinci: add SRAM allocator

2009-05-22 Thread Kevin Hilman
[ removing linux-arm-kernel ] Ring, Chris cr...@ti.com writes: -Original Message- From: Kevin Hilman [mailto:khil...@deeprootsystems.com] All of this discussion isn't relevant to the need for an in-kernel SRAM allocator upstream. What is being discussed here is various ways of

RE: [PATCH 3/3] davinci: add SRAM allocator

2009-05-21 Thread Ring, Chris
Just a quick can we talk through this patch a bit note... I _think_ there are existing codecs on some DaVinci platforms (e.g. DM365) that are already using this SRAM memory. The current [easy] solution to their use case was to keep this memory out of the kernel, and simply give it to CMEM to

Re: [PATCH 3/3] davinci: add SRAM allocator

2009-05-21 Thread David Brownell
On Thursday 21 May 2009, Ring, Chris wrote: I _think_ there are existing codecs on some DaVinci platforms (e.g. DM365) that are already using this SRAM memory.  The current [easy] solution to their use case was to keep this memory out of the kernel, and simply give it to CMEM to manage (via