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

[Eric Miao]

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?

>
> Best Regards,
> J.
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/