Hi Daniel, On 24.05.2014 19:24, Daniel Lezcano wrote: > A look at the code reveals use of S5P_VA_SYSRAM macro, in case of certain SoC > revisions, which is not valid any longer, after SYSRAM started to be mapped > dynamically. The new dynamic mapping is stored in sysram_base_addr variable, > which is declared static in platsmp.c > > This fix makes sysram_base_addr non-static, declared it in common.h and used > in pm.c instead of S5P_VA_SYSRAM. > > Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx> > Suggested-by: Tomasz Figa <t.figa@xxxxxxxxxxx> > --- > arch/arm/mach-exynos/common.h | 1 + > arch/arm/mach-exynos/platsmp.c | 2 +- > arch/arm/mach-exynos/pm.c | 4 ++-- > 3 files changed, 4 insertions(+), 3 deletions(-) > Looks good to me. Reviewed-by: Tomasz Figa <t.figa@xxxxxxxxxxx> Best regards, Tomasz -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html