No subject

--------------------------------------------------
From: "Nori, Sekhar" <nsekhar at ti.com>
Sent: Thursday, February 24, 2011 4:57 PM
To: "Subhasish Ghosh" <subhasish at mistralsolutions.com>; 
<davinci-linux-open-source at linux.davincidsp.com>
Cc: <linux-arm-kernel at lists.infradead.org>; "Watkins, Melissa" 
<m-watkins at ti.com>; <sachi at mistralsolutions.com>; "Kevin Hilman" 
<khilman at deeprootsystems.com>; "Russell King" <linux at arm.linux.org.uk>; 
"Michael Williamson" <michael.williamson at criticallink.com>; "Chemparathy, 
Cyril" <cyril at ti.com>; "Sergei Shtylyov" <sshtylyov at ru.mvista.com>; "open 
list" <linux-kernel at vger.kernel.org>
Subject: RE: [PATCH 1/1] davinci: changed SRAM allocator to shared ram.

> Hi Subhasish,
>
> On Thu, Feb 24, 2011 at 15:54:56, Subhasish Ghosh wrote:
>> --------------------------------------------------
>> From: "Nori, Sekhar" <nsekhar at ti.com>
>> Sent: Wednesday, February 23, 2011 9:00 PM
>> To: "Subhasish Ghosh" <subhasish at mistralsolutions.com>;
>> <davinci-linux-open-source at linux.davincidsp.com>
>> Cc: <linux-arm-kernel at lists.infradead.org>; "Watkins, Melissa"
>> <m-watkins at ti.com>; <sachi at mistralsolutions.com>; "Kevin Hilman"
>> <khilman at deeprootsystems.com>; "Russell King" <linux at arm.linux.org.uk>;
>> "Michael Williamson" <michael.williamson at criticallink.com>; "Chemparathy,
>> Cyril" <cyril at ti.com>; "Sergei Shtylyov" <sshtylyov at ru.mvista.com>; "open
>> list" <linux-kernel at vger.kernel.org>
>> Subject: RE: [PATCH 1/1] davinci: changed SRAM allocator to shared ram.
>>
>> > Hi Subhasish,
>> >
>> > On Fri, Feb 11, 2011 at 19:51:28, Subhasish Ghosh wrote:
>> >> This patch modifies the sram allocator to allocate memory
>> >> from the DA8XX shared RAM.
>> >
>> > It will be nice to know if you tried suspend-to-RAM
>> > after this change and found it to be working.
>> >
>>
>> SG -- My file system is currently mounted from MMC.
>>           Suspend to RAM seems to have bug with MMC.
>> 
>> (http://processors.wiki.ti.com/index.php/OMAP-L1_Linux_Drivers_Usage#Suspend-to-RAM)
>
> You can try Kevin's suggestion here:
>
> http://linux.davincidsp.com/pipermail/davinci-linux-open-source/2011-January/021807.html
>
>>           I had tried NFS earlier, but I think there is some problem with
>> NFS boot and udev with this kernel.
>
> Anyway there are some problems reported with Ethernet
> driver and suspend on the latest kernel. Folks inside
> TI are working on it.
>
>>            So, currently I will not be able to do it, but will try if I 
>> get
>> a chance.
>
> How about using ramdisk?
>
> Thanks,
> Sekhar
>
>> > Thanks,
>> > Sekhar
>> >
>> >>
>> >> Signed-off-by: Subhasish Ghosh <subhasish at mistralsolutions.com>
>> >> ---
>> >>  arch/arm/mach-davinci/da850.c              |    6 +++---
>> >>  arch/arm/mach-davinci/include/mach/da8xx.h |    1 +
>> >>  2 files changed, 4 insertions(+), 3 deletions(-)
>> >>
>> >> diff --git a/arch/arm/mach-davinci/da850.c
>> >> b/arch/arm/mach-davinci/da850.c
>> >> index 3443d97..8a4de97 100644
>> >> --- a/arch/arm/mach-davinci/da850.c
>> >> +++ b/arch/arm/mach-davinci/da850.c
>> >> @@ -711,7 +711,7 @@ static struct map_desc da850_io_desc[] = {
>> >>  },
>> >>  {
>> >>  .virtual = SRAM_VIRT,
>> >> - .pfn = __phys_to_pfn(DA8XX_ARM_RAM_BASE),
>> >> + .pfn = __phys_to_pfn(DA8XX_SHARED_RAM_BASE),
>> >>  .length = SZ_8K,
>> >>  .type = MT_DEVICE
>> >>  },
>> >> @@ -1083,8 +1083,8 @@ static struct davinci_soc_info
>> >> davinci_soc_info_da850 = {
>> >>  .gpio_irq = IRQ_DA8XX_GPIO0,
>> >>  .serial_dev = &da8xx_serial_device,
>> >>  .emac_pdata = &da8xx_emac_pdata,
>> >> - .sram_dma = DA8XX_ARM_RAM_BASE,
>> >> - .sram_len = SZ_8K,
>> >> + .sram_dma = DA8XX_SHARED_RAM_BASE,
>> >> + .sram_len = SZ_128K,
>> >>  .reset_device = &da8xx_wdt_device,
>> >>  };
>> >>
>> >> diff --git a/arch/arm/mach-davinci/include/mach/da8xx.h
>> >> b/arch/arm/mach-davinci/include/mach/da8xx.h
>> >> index cfcb223..c3c3339 100644
>> >> --- a/arch/arm/mach-davinci/include/mach/da8xx.h
>> >> +++ b/arch/arm/mach-davinci/include/mach/da8xx.h
>> >> @@ -70,6 +70,7 @@ extern unsigned int da850_max_speed;
>> >>  #define DA8XX_AEMIF_CTL_BASE 0x68000000
>> >>  #define DA8XX_DDR2_CTL_BASE 0xb0000000
>> >>  #define DA8XX_ARM_RAM_BASE 0xffff0000
>> >> +#define DA8XX_SHARED_RAM_BASE 0x80000000
>> >>
>> >>  void __init da830_init(void);
>> >>  void __init da850_init(void);
>> >> -- 
>> >> 1.7.2.3
>> >>
>> >>
>> >
>>
>