[U-Boot] [PATCH v4] arm: imx: add i.MX53 Beckhoff CX9020 Embedded PC

[Patrick Brünn]

Hi Stefano,
>
>On 04/11/2016 11:57, linux-kernel-dev at beckhoff.com wrote:
>> From: Patrick Bruenn <p.bruenn at beckhoff.com>
>>
>> Add CX9020 board based on mx53loco.
>> Add simplified imx53 base device tree from kernel v4.8-rc8, to reuse
>> serial_mxc with DTE and prepare for device tree migration of other
>> functions and imx53 devices.
>>
>> The CX9020 differs from i.MX53 Quick Start Board by:
>> - use uart2 instead of uart1
>> - DVI-D connector instead of VGA
>> - no audio
>> - CCAT FPGA connected to emi
>> - enable rtc
>>
>> Signed-off-by: Patrick Bruenn <p.bruenn at beckhoff.com>
>>
>> ---
>> Version 4 should address most of the issues Stefan, Jagan and Simon
>pointed out.
>
>There is still some small issues that avoid the patches to be merged.
>Board cannot be built due to missing CONFIG_:
>
>       arm:  +   mx53cx9020
>+drivers/video/cfb_console.c: In function 'cfg_video_init':
>+drivers/video/cfb_console.c:2022:23: error:
>'CONFIG_SYS_CONSOLE_FG_COL'
>undeclared (first use in this function)
>+drivers/video/cfb_console.c:2022:23: note: each undeclared identifier
>is reported only once for each function it appears in
>+drivers/video/cfb_console.c:2025:23: error:
>'CONFIG_SYS_CONSOLE_BG_COL'
>undeclared (first use in this function)
>+make[3]: *** [drivers/video/cfb_console.o] Error 1
>+make[2]: *** [drivers/video] Error 2
>+make[1]: *** [drivers] Error 2
>
>
>I have checked why and I could fix with the following patch:
>
Thank you for your effort!
>
>---
> configs/mx53cx9020_defconfig | 6 ++++++
> include/configs/mx53cx9020.h | 4 ----
> 2 files changed, 6 insertions(+), 4 deletions(-)
>
>diff --git a/configs/mx53cx9020_defconfig b/configs/mx53cx9020_defconfig
>index 3603907..6954cf5 100644
>--- a/configs/mx53cx9020_defconfig
>+++ b/configs/mx53cx9020_defconfig
>@@ -11,6 +11,7 @@ CONFIG_CMD_BOOTZ=y
> CONFIG_CMD_MMC=y
> #CONFIG_CMD_USB=y
> # CONFIG_CMD_SETEXPR is not set
>+CONFIG_VIDEO=y
> CONFIG_CMD_DHCP=y
> CONFIG_CMD_MII=y
> CONFIG_CMD_PING=y
>@@ -27,3 +28,8 @@ CONFIG_FPGA_CYCLON2=y
> CONFIG_CMD_FPGA=y
> CONFIG_PINCTRL=y
> CONFIG_PINCTRL_IMX5=y
>+# CONFIG_VIDEO_SW_CURSOR is not set
>+CONFIG_CFB_CONSOLE=y
>+CONFIG_VGA_AS_SINGLE_DEVICE=y
>+CONFIG_SYS_CONSOLE_BG_COL=0x00
>+CONFIG_SYS_CONSOLE_FG_COL=0xa0
>diff --git a/include/configs/mx53cx9020.h b/include/configs/mx53cx9020.h
>index ac8b2a2..87c75d4 100644
>--- a/include/configs/mx53cx9020.h
>+++ b/include/configs/mx53cx9020.h
>@@ -187,11 +187,7 @@
>
> /* Framebuffer and LCD */
> #define CONFIG_PREBOOT
>-#define CONFIG_VIDEO
> #define CONFIG_VIDEO_IPUV3
>-#define CONFIG_CFB_CONSOLE
>-#define CONFIG_VGA_AS_SINGLE_DEVICE
>-#define CONFIG_SYS_CONSOLE_IS_IN_ENV
> #define CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE
> #define CONFIG_VIDEO_BMP_RLE8
> #define CONFIG_SPLASH_SCREEN
>--
>2.7.4
>
>Now on you: if you agree, I can integrate this fix into your patch and
>merge it into the u-boot-imx tree. If you disagree, please check it and
>send an update version, thanks !

Agreed!
I applied it and did a test run. Everything works as it is supposed to be.

Best regards,
Patrick

Signed-off-by: Patrick Bruenn <p.bruenn at beckhoff.com>

>
>Best regards,
>Stefano Babic
>
>--
>===========================================================
>==========
>DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
>HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
>Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
>===========================================================
>==========

Beckhoff Automation GmbH & Co. KG | Managing Director: Dipl. Phys. Hans Beckhoff
Registered office: Verl, Germany | Register court: Guetersloh HRA 7075