Hi. Thanks for your patience :D /Lars Pedersen On Wed, 5 Aug 2020 at 14:30, Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> wrote: > > Hello, > > On 8/5/20 1:12 PM, Lars Pedersen wrote: > >> Text base is ignored, when you are relocatable. If you aren't relocatable, > >> you need a valid address as your text base. Why did you turn it off? > > > > I didn't turn it off. It isn't selected by MACH_FREESCALE_MX7_SABRESD > > so it gets turned off if only this board is used. It is selected by 6 > > other boards, which is the reason it works when compiling for all > > boards. > > I don't think that's how it works. If it was =y and the select went > away because of CONFIG_MACH_CCMX51=n, it remains at =y. I just tested > it and that's the behavior I observed. If I use the imx_v7_defconfig and go and deselect "ConnectCore i.MX51" (MACH_CCMX51), then RELOCATABLE will change from =y to =n So in other words. Deselecting this one board (MACH_CCMX51) will set RELOCATABLE=n and now all the boards rely on CONFIG_TEXT_BASE to be set correct. And as you also mentioned later on, this will be an issue. > > > I am not sure what we could do better here. You can use barebox > as non-relocatable binary if you set CONFIG_TEXT_BASE to a sensible > value. You might want this if you are keen on reducing barebox size > further at cost of portability. So having a select here to ensure > it's always set won't work. > > We can also not set a suitable CONFIG_TEXT_BASE at Kconfig level, > because, due to multi-image support, we may be building different > boards with different TEXT_BASEs... > > We could have RELOCATABLE be default y as it's a more sensible default > though. Sascha, what do you think? > > > > > Symbol: RELOCATABLE [=y] > > Type : bool > > Defined at common/Kconfig:336 > > Prompt: generate relocatable barebox binary > > Depends on: PPC || ARM [=y] > > Location: > > (2) -> General Settings > > Selected by [y]: > > - MACH_CCMX51 [=y] && ARCH_IMX [=y] && IMX_MULTI_BOARDS [=y] > > Selected by [n]: > > - ARCH_BCM283X [=n] && <choice> > > - ARCH_TEGRA [=n] && <choice> > > - ARCH_ZYNQMP [=n] && <choice> > > - MACH_VEXPRESS [=n] && <choice> > > - ARCH_ZYNQ7000 [=n] && ARCH_ZYNQ [=n] > > > > > >> > >> Cheers, > >> Ahmad > >> > >> -- > >> Pengutronix e.K. | | > >> Steuerwalder Str. 21 | http://www.pengutronix.de/ | > >> 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > >> Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | > > > > -- > Pengutronix e.K. | | > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
