[PATCH 2/7] OMAP2+: use global values for the SRAM PA addresses

[jean.pihet@newoldbits.com (Jean Pihet)]

Hi Nishant,

On Fri, Dec 17, 2010 at 2:34 PM, Nishanth Menon <nm@ti.com> wrote:
> jean.pihet at newoldbits.com had written, on 12/17/2010 04:08 AM, the
> following:
>>
>> From: Jean Pihet <j-pihet@ti.com>
>>
>> The SRAM PA addresses are locally defined and used at
>> different places, i.e. SRAM management code and idle sleep code.
>>
>> The macros are now defined at a centralized place, for
>> easier maintenance.
>>
>> Tested on N900 and Beagleboard with full RET and OFF modes,
>> using cpuidle and suspend.
>>
>> Signed-off-by: Jean Pihet <j-pihet@ti.com>
>> ---
>> ?arch/arm/mach-omap2/sdrc.h ? ? ? ? ? ? | ? ?1 -
>> ?arch/arm/mach-omap2/sleep34xx.S ? ? ? ?| ? ?1 +
>> ?arch/arm/plat-omap/include/plat/sram.h | ? 11 +++++++++++
>> ?arch/arm/plat-omap/sram.c ? ? ? ? ? ? ?| ? ?7 ++-----
>> ?4 files changed, 14 insertions(+), 6 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/sdrc.h b/arch/arm/mach-omap2/sdrc.h
>> index 68f57bb..b3f8379 100644
>> --- a/arch/arm/mach-omap2/sdrc.h
>> +++ b/arch/arm/mach-omap2/sdrc.h
>> @@ -74,5 +74,4 @@ static inline u32 sms_read_reg(u16 reg)
>> ?*/
>> ?#define SDRC_MPURATE_LOOPS ? ? ? ? ? ? 96
>> ?-
>
> spurious change.
Cosmetic change!

>
> [...]
>
>> diff --git a/arch/arm/plat-omap/sram.c b/arch/arm/plat-omap/sram.c
>> index 819ea0c..1a686c8 100644
>> --- a/arch/arm/plat-omap/sram.c
>> +++ b/arch/arm/plat-omap/sram.c
>> @@ -41,15 +41,12 @@
>> ?#define OMAP1_SRAM_PA ? ? ? ? 0x20000000
>> ?#define OMAP1_SRAM_VA ? ? ? ? ?VMALLOC_END
>> -#define OMAP2_SRAM_PA ? ? ? ? ?0x40200000
>> -#define OMAP2_SRAM_PUB_PA ? ? ?0x4020f800
>> +#define OMAP2_SRAM_PUB_PA ? ? ?(OMAP2_SRAM_PA + 0xf800)
>> ?#define OMAP2_SRAM_VA ? ? ? ? ?0xfe400000
>> ?#define OMAP2_SRAM_PUB_VA ? ? ?(OMAP2_SRAM_VA + 0x800)
>> -#define OMAP3_SRAM_PA ? ? ? ? ? 0x40200000
>> ?#define OMAP3_SRAM_VA ? ? ? ? ? 0xfe400000
>> -#define OMAP3_SRAM_PUB_PA ? ? ? 0x40208000
>> +#define OMAP3_SRAM_PUB_PA ? ? ? (OMAP3_SRAM_PA + 0x8000)
>
> Though not directly related to the patch, information sake: the PUB physical
> address on a GP device != public SRAM address on a HS/EMU device - in theory
> PPA can reconfigure how much is available for a HS/EMU device here.
Ok good to know.

>
>> ?#define OMAP3_SRAM_PUB_VA ? ? ? (OMAP3_SRAM_VA + 0x8000)
>> -#define OMAP4_SRAM_PA ? ? ? ? ?0x40300000
>> ?#define OMAP4_SRAM_VA ? ? ? ? ?0xfe400000
>> ?#define OMAP4_SRAM_PUB_PA ? ? ?(OMAP4_SRAM_PA + 0x4000)
>> ?#define OMAP4_SRAM_PUB_VA ? ? ?(OMAP4_SRAM_VA + 0x4000)
>
> Tested-by: Nishanth Menon<nm@ti.com>
> Tested on:
> SDP3630
> SDP3430
> Test script:
> http://pastebin.mozilla.org/889933
Nice test script btw

>
> --
> Regards,
> Nishanth Menon
>

Thanks!
Jean