On 5 April 2017 at 21:44, Simon Glass <s...@chromium.org> wrote: > Hi Philipp, > > On 1 April 2017 at 04:59, Philipp Tomsich > <philipp.toms...@theobroma-systems.com> wrote: >> The RK3399-Q7 ("Puma") SoM exposes UART0 as the Qseven UART (i.e. the >> serial line available via standardised pins on the edge connector and >> available on a RS232 connector). >> >> To support boards (such as the RK3399-Q7) that require UART0 as a >> debug console, we match CONFIG_DEBUG_UART_BASE and add the appropriate >> iomux setup to the rk3399 SPL code. >> >> As we are already touching this code, we also move the board-specific >> UART setup (i.e. iomux setup) into board_debug_uart_init(). This will >> be called from the debug UART init when CONFIG_DEBUG_UART_BOARD_INIT >> is set. >> >> As the RK3399 needs to use its board_debug_uart_init() function, we >> have Kconfig enable it by default for RK3399 builds. >> >> With everything set up to define CONFIG_BAUDRATE via defconfig and >> with to have the SPL debug UART either on UART0 or UART2, the configs >> for the RK3399 EVB are then update (the change for the RK3399-Q7 is >> left for later to not cause issues on applying the change). >> >> Signed-off-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com> >> Reviewed-by: Kever Yang <kever.y...@rock-chips.com> > > Acked-by: Simon Glass <s...@chromium.org> > > Really nice commit message, thank you. >> >> --- >> >> Changes in v3: >> - update Kconfig to auto-select CONFIG_DEBUG_UART_BOARD_INIT for the >> RK3399 to ensure there's no build failues (even as the defconfig >> files are not updated) >> - drop the CONFIG_DEBUG_UART_BOARD_INIT settings from defconfig >> (eliminating the 3rd patch from the original series) >> >> Changes in v2: >> - changed hex constant to lowercase >> >> arch/arm/include/asm/arch-rockchip/grf_rk3399.h | 8 +++++++ >> arch/arm/mach-rockchip/Kconfig | 1 + >> arch/arm/mach-rockchip/rk3399-board-spl.c | 29 >> ++++++++++++++++++------- >> 3 files changed, 30 insertions(+), 8 deletions(-)
Applied to u-boot-rockchip, thanks! _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot