* Arnd Bergmann <arnd@xxxxxxxx> [171206 21:51]: > On Wed, Dec 6, 2017 at 5:29 PM, Dan Murphy <dmurphy@xxxxxx> wrote: > > Arnd > > > > On 12/06/2017 09:57 AM, Tony Lindgren wrote: > >> * Arnd Bergmann <arnd@xxxxxxxx> [171206 14:18]: > >>> In configurations without CONFIG_OMAP3 but with secure RAM support, > >>> we now run into a link failure: > >>> > >>> arch/arm/mach-omap2/omap-secure.o: In function `omap3_save_secure_ram': > >>> omap-secure.c:(.text+0x130): undefined reference to `save_secure_ram_context' > >>> > >>> The omap3_save_secure_ram() function is only called from the OMAP34xx > >>> power management code, so we can simply hide that function in the > >>> appropriate #ifdef. > >>> > >>> Fixes: d09220a887f7 ("ARM: OMAP2+: Fix SRAM virt to phys translation for save_secure_ram_context") > >>> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> > >> > >> Thanks for fixing it, want to apply directly to ARM SoC fixes > >> where d09220a887f7 is now? If so: > >> > >> Acked-by: Tony Lindgren <tony@xxxxxxxxxxx> > > > > Found the same issue in our builds. > > Compile tested the patch against different TI defconfig configurations. > > > > Tested-by: Dan Murphy <dmurphy@xxxxxx> > > Thanks! > > Unfortunately, I found another build problem with CONFIG_PM=n, > sent an updated patch now, and will apply it to the fixes branch > tomorrow keeping both of your Acked-by/Tested-by tags, unless > I hear any objections or run into a third problem. OK thanks works for me. Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html