On 12 November 2015 at 18:42, Ariel D'Alessandro ariel@vanguardiasur.com.ar wrote:
CONFIG_DEBUG_UART is enabled in defconfig, but there's no Low-level debugging functions implemented, so build fails because of undefined references to `printch' in common/console.c. In order to fix this, enable CONFIG_DEBUG_LL.
I think you are fixing this the wrong way. This patch solves the problem as well, and I think it's the right way:
[patch format is probably wasted] @@ -34,6 +34,7 @@ CONFIG_REGULATOR_ACT8846=y CONFIG_RAM=y CONFIG_SPL_RAM=y CONFIG_DEBUG_UART=y +CONFIG_DEBUG_UART_NS16550=y CONFIG_DEBUG_UART_BASE=0xff690000 CONFIG_DEBUG_UART_CLOCK=24000000 CONFIG_DEBUG_UART_SHIFT=2
Without this fix, Altera JTAG UART is selected instead of NS16550. I expect other defconfigs (e.g. chromebook_jerry_defconfig) to suffer from this as well.
That said, I still haven't looked at the differences between DEBUG_LL and DEBUG_UART.