On Wed, Dec 05, 2018 at 02:58:12PM +0100, Oleksij Rempel wrote: > Am 05.12.18 um 09:50 schrieb Antony Pavlov: > > I suppose there is a problem with simultaneous debug_ll support for AR9344 and AR9331 in one binary. > > the Kconfig DEBUG_AR9331_UART and DEBUG_AR9344_UART options are mutually exclusive. > > > > Your approach makes it possible to build barebox binary image for AR9331 board with debug_ll code for AR9344 > > and vice versa. > > yes, same issues as with most of DEBUG_LL related configurations in > common/Kconfig. It is possible to misconfigure the DEBUG_LL for board or > SoC. Currently I don't have a solution except of remove DEBUG_LL support > completely from Kconfig for MIPS, or introduce some dynamic > configuration, which is not related to this patch set. At least on ARM with multi image support you have to make sure to chose the correct DEBUG_LL UART that fits the board. Or to put the other way round: Enabling DEBUG_LL limits the build to a single board only, thus it should only be enabled during debugging. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 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
