On Do, 2020-01-02 at 11:23 +0100, Rouven Czerwinski wrote: > Mux UART1 and UART2 correctly to allow usage of DEBUG_LL. There is only one UART intended as console on this board. Why are you setting up 2 UARTs here? Regards, Lucas > This commit has only been tested on nitrogen6q. > > Signed-off-by: Rouven Czerwinski <r.czerwinski@xxxxxxxxxxxxxx> > --- > .../boundarydevices-nitrogen6/lowlevel.c | 24 > +++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/arch/arm/boards/boundarydevices-nitrogen6/lowlevel.c > b/arch/arm/boards/boundarydevices-nitrogen6/lowlevel.c > index 74ff71fc24..47c231396d 100644 > --- a/arch/arm/boards/boundarydevices-nitrogen6/lowlevel.c > +++ b/arch/arm/boards/boundarydevices-nitrogen6/lowlevel.c > @@ -1,16 +1,40 @@ > #include <common.h> > +#include <debug_ll.h> > #include <mach/generic.h> > #include <mach/esdctl.h> > #include <asm/barebox-arm.h> > +#include <serial/imx-uart.h> > > extern char __dtb_imx6q_nitrogen6x_start[]; > > +static inline void setup_uart(void) > +{ > + void __iomem *iomuxbase = (void *)MX6_IOMUXC_BASE_ADDR; > + > + //UART 1 > + writel(0x1, iomuxbase + 0x02a8); > + writel(0x1, iomuxbase + 0x02ac); > + > + //UART 2 > + writel(0x4, iomuxbase + 0x00bc); > + writel(0x4, iomuxbase + 0x00c0); > + writel(0x0, iomuxbase + 0x0928); > + > + imx6_ungate_all_peripherals(); > + imx6_uart_setup((void *)MX6_UART2_BASE_ADDR); > + pbl_set_putc(imx_uart_putc, (void *)MX6_UART2_BASE_ADDR); > +} > + > + > ENTRY_FUNCTION(start_imx6q_nitrogen6x_1g, r0, r1, r2) > { > void *fdt; > > imx6_cpu_lowlevel_init(); > > + if (IS_ENABLED(CONFIG_DEBUG_LL)) > + setup_uart(); > + > fdt = __dtb_imx6q_nitrogen6x_start + get_runtime_offset(); > > imx6q_barebox_entry(fdt); _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox