[PATCH v2 2/2] ARM: OMAP: sram: Add am33xx SRAM support (minimal)
Peter Korsgaard
jacmet at sunsite.dk
Mon Feb 13 09:24:05 EST 2012
>>>>> "Afzal" == Afzal Mohammed <afzal at ti.com> writes:
Afzal> From: Vaibhav Bedia <vaibhav.bedia at ti.com>
Afzal> Update SRAM start & size for am33xx SoC's.
Afzal> Note: cpu_is_34xx() is true for am33xx also. Doing
Afzal> cpu_is_am33xx() check after cpu_is_34xx() will not
Afzal> achieve what we want due to the above reason.
Afzal> Hence cpu_is_am33xx() is done before cpu_is_34xx()
Afzal> } else {
Afzal> - if (cpu_is_omap34xx()) {
Afzal> + if (cpu_is_am33xx()) {
Afzal> + omap_sram_start = AM33XX_SRAM_PA;
Afzal> + omap_sram_size = 0x10000; /* 64K */
Afzal> + } else if (cpu_is_omap34xx()) {
Afzal> omap_sram_start = OMAP3_SRAM_PA;
Afzal> omap_sram_size = 0x10000; /* 64K */
I don't particular know the omap sram stuff, but doesn't the 33xx have
2x 64K blocks of SRAM?
Afzal> +static inline int am33xx_sram_init(void)
Afzal> +{
Afzal> + return 0;
I know you mentioned it in the commit message, but it might be good with
a comment here as well that this dummy function is needed to not get the 34xx
init function called for 33xx, so it doesn't get removed when somebody
decides to cleanup.
Afzal> +}
Afzal> +
Afzal> int __init omap_sram_init(void)
Afzal> {
Afzal> omap_detect_sram();
Afzal> @@ -379,6 +387,8 @@ int __init omap_sram_init(void)
Afzal> omap242x_sram_init();
Afzal> else if (cpu_is_omap2430())
Afzal> omap243x_sram_init();
Afzal> + else if (cpu_is_am33xx())
Afzal> + am33xx_sram_init();
Afzal> else if (cpu_is_omap34xx())
Afzal> omap34xx_sram_init();
--
Bye, Peter Korsgaard
More information about the linux-arm-kernel
mailing list