Re: [PATCH 1/1] davinci: changed SRAM allocator to shared ram.

[Subhasish Ghosh]

--------------------------------------------------
From: "Nori, Sekhar" <nsekhar@xxxxxx>
Sent: Wednesday, February 23, 2011 9:00 PM
To: "Subhasish Ghosh" <subhasish@xxxxxxxxxxxxxxxxxxxx>; 
<davinci-linux-open-source@xxxxxxxxxxxxxxxxxxxx>
Cc: <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>; "Watkins, Melissa" 
<m-watkins@xxxxxx>; <sachi@xxxxxxxxxxxxxxxxxxxx>; "Kevin Hilman" 
<khilman@xxxxxxxxxxxxxxxxxxx>; "Russell King" <linux@xxxxxxxxxxxxxxxx>; 
"Michael Williamson" <michael.williamson@xxxxxxxxxxxxxxxx>; "Chemparathy, 
Cyril" <cyril@xxxxxx>; "Sergei Shtylyov" <sshtylyov@xxxxxxxxxxxxx>; "open 
list" <linux-kernel@xxxxxxxxxxxxxxx>
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)
         I had tried NFS earlier, but I think there is some problem with 
NFS boot and udev with this kernel.
          So, currently I will not be able to do it, but will try if I get 
a chance.

> Thanks,
> Sekhar
>
> > Signed-off-by: Subhasish Ghosh <subhasish@xxxxxxxxxxxxxxxxxxxx>
> > ---
> >  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
--
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/