Re: [PATCH v3] imx: support i.MX8QM DMSSE20 a1 board

[Oliver Graute <oliver.graute@kococonnector.com>]

On 30/11/21, Fabio Estevam wrote:
> Hi Oliver,
> 
> On Tue, Nov 30, 2021 at 8:29 AM Oliver Graute
> <oliver.graute@kococonnector.com> wrote:
> >
> > Add i.MX8QM DMSSE20 a1 board support
> >
> > U-Boot 2021.10-00029-g4ad997ec17 (Nov 30 2021 - 09:40:26 +0100)
> >
> > Model: Advantech iMX8QM DMSSE20
> > Board: DMS-SE20A1 8GB
> > Build: SCFW 875cfeb5, SECO-FW d63fdb21, ATF 09c5cc9
> > Boot:  USB
> > DRAM:  8 GiB
> > MMC:   FSL_SDHC: 0, FSL_SDHC: 1, FSL_SDHC: 2
> > Loading Environment from MMC... OK
> > In:    serial@5a060000
> > Out:   serial@5a060000
> > Err:   serial@5a060000
> > Net:   eth0: ethernet@5b040000 [PRIME]
> > Warning: ethernet@5b050000 (eth1) using random MAC address - c6:1b:bf:cc:e5:6c
> > , eth1: ethernet@5b050000
> > Hit any key to stop autoboot:  0
> >
> > Signed-off-by: Oliver Graute <oliver.graute@kococonnector.com>
> > ---
> >
> > This is working on 2021.10. Can you give me some hints what I have to do to
> > get it working with current master branch? There are some imx related changes
> > that I don't overlook.
> 
> What are the exact errors you see with the master branch?

It's more or less the problem where which definition belongs to
nowerdays. I get the following errors on configure and compiling:

+ make imx8qm_dmsse20a1_defconfig
arch/../configs/imx8qm_dmsse20a1_defconfig:78:warning: override: reassigning to symbol MII
#
# configuration written to .config
#
+ echo 'start the u-boot build'
start the u-boot build
+ make u-boot.bin
scripts/kconfig/conf  --syncconfig Kconfig
.config:279:warning: symbol value '' invalid for SYS_LOAD_ADDR
*
* Restart config...
*
*
* General setup
*
Local version - append to U-Boot release (LOCALVERSION) []
Automatically append version information to the version string (LOCALVERSION_AUTO) [Y/n/?] y
Optimize for size (CC_OPTIMIZE_FOR_SIZE) [Y/n/?] y
Allow compiler to uninline functions marked 'inline' in full U-Boot (OPTIMIZE_INLINING) [N/y/?] n
Allow compiler to uninline functions marked 'inline' in SPL (SPL_OPTIMIZE_INLINING) [N/y/?] n
Enable Link Time Optimizations (LTO) [N/y/?] n
Select U-Boot be run as a bootloader for XEN Virtual Machine (XEN) [N/y/?] n
Select defaults suitable for booting general purpose Linux distributions (DISTRO_DEFAULTS) [N/y/?] n
Add arch, board, vendor and soc variables to default environment (ENV_VARS_UBOOT_CONFIG) [N/y/?] n
Number of DRAM banks (NR_DRAM_BANKS) [4] 4
Enable kernel command line setup (SYS_BOOT_GET_CMDLINE) [N/y/?] n
Enable kernel board information setup (SYS_BOOT_GET_KBD) [N/y/?] n
Enable malloc() pool before relocation (SYS_MALLOC_F) [Y/?] y
  Size of malloc() pool before relocation (SYS_MALLOC_F_LEN) [0x4000] 0x4000
Define memory for Dynamic allocation (SYS_MALLOC_LEN) [0x400000] 0x400000
Size of malloc() pool in SPL (SPL_SYS_MALLOC_F_LEN) [0x4000] 0x4000
64bit physical address support (PHYS_64BIT) [Y/?] y
SPL image used in the combined SPL+U-Boot image (SPL_IMAGE) [spl/u-boot-spl.bin] spl/u-boot-spl.bin
Build target special images (BUILD_TARGET) []
Use a custom location for the U-Boot linker script (SYS_CUSTOM_LDSCRIPT) [N/y/?] n
Address in memory to use by default (SYS_LOAD_ADDR) [] (NEW)

Where is the place to set SYS_LOAD_ADDR to 0x80280000?


Address in memory to use by default (SYS_LOAD_ADDR) [] (NEW) 0x80280000
Define max stack size that can be used by U-Boot (STACK_SIZE) [0x1000000] 0x1000000
Compile API examples (EXAMPLES) [N/y/?] n
  CFG     u-boot.cfg
  GEN     include/autoconf.mk
  GEN     include/autoconf.mk.dep
  CFG     spl/u-boot.cfg
  GEN     spl/include/autoconf.mk
  UPD     include/generated/timestamp_autogenerated.h
./arch/arm/mach-imx/spl_sd.cfg:7:10: fatal error: config.h: Datei oder Verzeichnis nicht gefunden
 #include <config.h>
          ^~~~~~~~~~
compilation terminated.
  CC      cmd/version.o
In file included from include/common.h:16:0,
                 from cmd/version.c:7:
include/config.h:2:0: warning: "CONFIG_IMX_CONFIG" redefined
 #define CONFIG_IMX_CONFIG board/advantech/imx8qm_dmsse20_a1/imximage.cfg

In file included from ././include/linux/kconfig.h:4:0,
                 from <command-line>:0:
include/generated/autoconf.h:315:0: note: this is the location of the previous definition
 #define CONFIG_IMX_CONFIG "arch/arm/mach-imx/spl_sd.cfg"

In file included from include/config.h:5:0,
                 from include/common.h:16,
                 from cmd/version.c:7:
include/configs/imx8qm_dmsse20.h:120:0: warning: "CONFIG_SYS_LOAD_ADDR" redefined
 #define CONFIG_SYS_LOAD_ADDR  CONFIG_LOADADDR

In file included from ././include/linux/kconfig.h:4:0,
                 from <command-line>:0:
include/generated/autoconf.h:128:0: note: this is the location of the previous definition
 #define CONFIG_SYS_LOAD_ADDR 0x80280000

In file included from include/config.h:5:0,
                 from include/common.h:16,
                 from cmd/version.c:7:
include/configs/imx8qm_dmsse20.h:138:0: warning: "CONFIG_SYS_MMC_ENV_DEV" redefined
 #define CONFIG_SYS_MMC_ENV_DEV  2   /* USDHC3 */

In file included from ././include/linux/kconfig.h:4:0,
                 from <command-line>:0:
include/generated/autoconf.h:115:0: note: this is the location of the previous definition
 #define CONFIG_SYS_MMC_ENV_DEV 0

In file included from include/config.h:5:0,
                 from include/common.h:16,
                 from cmd/version.c:7:
include/configs/imx8qm_dmsse20.h:143:0: warning: "CONFIG_SYS_MALLOC_LEN" redefined
 #define CONFIG_SYS_MALLOC_LEN  ((CONFIG_ENV_SIZE + (32 * 1024)) * 1024)

In file included from ././include/linux/kconfig.h:4:0,
                 from <command-line>:0:
include/generated/autoconf.h:16:0: note: this is the location of the previous definition
 #define CONFIG_SYS_MALLOC_LEN 0x400000

snippet

Best regards,

Oliver