Re: [PATCH V4 0/3] ARM: mmp: add audio sram support

From: Leo Yan
Date: Mon Aug 15 2011 - 06:23:23 EST

On 08/15/2011 05:35 PM, Eric Miao wrote:
On Mon, Aug 15, 2011 at 5:12 PM, Jean-Christophe PLAGNIOL-VILLARD
<plagnioj@xxxxxxxxxxxx> wrote:
On 16:43 Mon 15 Aug , Eric Miao wrote:
On Mon, Aug 15, 2011 at 11:11 AM, Haojian Zhuang
<haojian.zhuang@xxxxxxxxxxx> wrote:
On Sun, 2011-08-14 at 20:09 -0700, Leo Yan wrote:
On mmp platform, there have two sram banks:
audio sram bank, and internal sram bank for video and PM.
So add the sram module to manage these sram banks.

And register the sram banks so can dynamically alloc/free
the buffer.

Leo Yan (3):
ARM: mmp: add sram allocator
ARM: mmp: register audio sram bank
ARM: mmp: register internal sram bank

arch/arm/Kconfig | 1 +
arch/arm/mach-mmp/Makefile | 2 +-
arch/arm/mach-mmp/brownstone.c | 11 ++
arch/arm/mach-mmp/include/mach/mmp2.h | 13 +++
arch/arm/mach-mmp/include/mach/sram.h | 35 +++++++
arch/arm/mach-mmp/mmp2.c | 3 +
arch/arm/mach-mmp/sram.c | 168 +++++++++++++++++++++++++++++++++
7 files changed, 232 insertions(+), 1 deletions(-)
create mode 100644 arch/arm/mach-mmp/include/mach/sram.h
create mode 100644 arch/arm/mach-mmp/sram.c

Acked-by: Haojian Zhuang<haojian.zhuang@xxxxxxxxx>

Looks good to me. Thanks Haojian. Applied to -devel.
I've some reserve on the named pool they need to be managed at generic level
not here

What are their names? And your suggestions to fix this?

For MMP platform have not only one bank, so now named the audio sram bank as "asram", and another sram bank as "isram" which is used by video/secure processor/pm.
If other modules want to use the sram, just use the name string to
get the gen pool handler, and then just call genalloc APIs.

I just wander if maintain the name in genalloc, then the name string
we should maintain in the pool's structure or chunk's structure?

Best Regards,

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at