On Thu, Jan 17, 2019 at 08:13:21AM -0600, Andrew F. Davis wrote: > On 1/16/19 3:14 PM, Tom Rini wrote: > > On Wed, Dec 05, 2018 at 11:51:33AM -0600, Andrew F. Davis wrote: > > > >> Like AM33xx and AM43xx, DRA7xx and AM57xx devices may need to > >> have an non-standard boot address in memory. This may be due > >> to the device being a high security variant, which place the > >> Initial SoftWare (ISW) after certificates and secure software. > >> > >> Allow these devices to set this from Kconfig. > >> > >> Signed-off-by: Andrew F. Davis <a...@ti.com> > >> Reviewed-by: Tom Rini <tr...@konsulko.com> > >> --- > >> arch/arm/mach-omap2/Kconfig | 13 +++++++++++++ > >> arch/arm/mach-omap2/am33xx/Kconfig | 15 --------------- > >> include/configs/ti_omap5_common.h | 2 +- > >> 3 files changed, 14 insertions(+), 16 deletions(-) > > > > Turns out this breaks OMAP3 among others as the question is asked there > > now and we have no default value (nor I suspect a reasonable one). > > The only thing changed is ti_omap5_common.h, the default provided in > that file is 0x40301350, would it be okay to just make that the default > for ISW_ENTRY_ADDR for all platforms that included that file?
Well, ISW_ENTRY_ADDR doesn't make sense for OMAP3, right? I suspect the problem is that in moving it from mach-omap2/am33xx/Kconfig to mach-omap2/Kconfig you need to update the depends on guards so that it doesn't show up where it's not used. -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot