Re: Getting serial console output on new imx7d tqma7 board UART4

[Lars Pedersen <lapeddk@gmail.com>]

Hi. Thanks for your patience :D

/Lars Pedersen

On Wed, 5 Aug 2020 at 14:30, Ahmad Fatoum <a.fatoum@pengutronix.de> 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@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox