From mboxrd@z Thu Jan 1 00:00:00 1970 From: plagnioj@jcrosoft.com (Jean-Christophe PLAGNIOL-VILLARD) Date: Wed, 17 Aug 2011 14:32:21 +0200 Subject: [PATCH V4 0/3] ARM: mmp: add audio sram support In-Reply-To: <4E48F420.3030101@marvell.com> References: <1313377794-26721-1-git-send-email-leoy@marvell.com> <1313377893.14434.5.camel@Lily> <20110815091245.GC32705@game.jcrosoft.org> <4E48F420.3030101@marvell.com> Message-ID: <20110817123221.GH28201@game.jcrosoft.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 18:25 Mon 15 Aug , Leo Yan wrote: > > > On 08/15/2011 05:35 PM, Eric Miao wrote: > >On Mon, Aug 15, 2011 at 5:12 PM, Jean-Christophe PLAGNIOL-VILLARD > > wrote: > >>On 16:43 Mon 15 Aug , Eric Miao wrote: > >>>On Mon, Aug 15, 2011 at 11:11 AM, Haojian Zhuang > >>> 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 > >>> > >>>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? on the chunk Best Regards, J.