Re: [PATCH] scripts: dtc: Start building fdtoverlay and fdtdump

From: kernel test robot
Date: Wed Jan 06 2021 - 07:25:53 EST


Hi Viresh,

I love your patch! Yet something to improve:

[auto build test ERROR on robh/for-next]
[also build test ERROR on v5.11-rc2 next-20210104]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url: https://github.com/0day-ci/linux/commits/Viresh-Kumar/scripts-dtc-Start-building-fdtoverlay-and-fdtdump/20210106-181140
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: x86_64-randconfig-a013-20210106 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 5c951623bc8965fa1e89660f2f5f4a2944e4981a)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://github.com/0day-ci/linux/commit/5ee2a6ba3d6de17f6e9b9dc259081630b6deffb8
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Viresh-Kumar/scripts-dtc-Start-building-fdtoverlay-and-fdtdump/20210106-181140
git checkout 5ee2a6ba3d6de17f6e9b9dc259081630b6deffb8
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All errors (new ones prefixed by >>):

>> make[2]: *** No rule to make target 'scripts/dtc/fdtoverlay.c', needed by 'scripts/dtc/fdtoverlay.o'.
In file included from scripts/dtc/fdtdump.c:12:
>> scripts/dtc/libfdt/fdt.h:13:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t magic; /* magic word FDT_MAGIC */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:14:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t totalsize; /* total size of DT block */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:15:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t off_dt_struct; /* offset to structure */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:16:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t off_dt_strings; /* offset to strings */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:17:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t off_mem_rsvmap; /* offset to memory reserve map */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:18:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t version; /* format version */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:19:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t last_comp_version; /* last compatible version */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:22:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t boot_cpuid_phys; /* Which physical CPU id we're
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:25:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t size_dt_strings; /* size of the strings block */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:28:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t size_dt_struct; /* size of the structure block */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
>> scripts/dtc/libfdt/fdt.h:32:2: error: unknown type name 'fdt64_t'; did you mean 'int64_t'?
fdt64_t address;
^~~~~~~
int64_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:27:19: note: 'int64_t' declared here
typedef __int64_t int64_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:33:2: error: unknown type name 'fdt64_t'; did you mean 'int64_t'?
fdt64_t size;
^~~~~~~
int64_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:27:19: note: 'int64_t' declared here
typedef __int64_t int64_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:37:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t tag;
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:42:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t tag;
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:43:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t len;
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:44:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t nameoff;
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
>> scripts/dtc/fdtdump.c:156:28: error: too few arguments to function call, expected 2, have 1
buf = utilfdt_read(argv[1]);
~~~~~~~~~~~~ ^
scripts/dtc/util.h:97:7: note: 'utilfdt_read' declared here
char *utilfdt_read(const char *filename, size_t *len);
^
17 errors generated.
make[2]: *** [scripts/Makefile.host:112: scripts/dtc/fdtdump.o] Error 1
make[2]: Target '__build' not remade because of errors.
make[1]: *** [Makefile:1356: scripts_dtc] Error 2
make[1]: Target 'modules_prepare' not remade because of errors.
make: *** [Makefile:185: __sub-make] Error 2
make: Target 'modules_prepare' not remade because of errors.
--
>> make[2]: *** No rule to make target 'scripts/dtc/fdtoverlay.c', needed by 'scripts/dtc/fdtoverlay.o'.
In file included from scripts/dtc/fdtdump.c:12:
>> scripts/dtc/libfdt/fdt.h:13:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t magic; /* magic word FDT_MAGIC */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:14:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t totalsize; /* total size of DT block */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:15:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t off_dt_struct; /* offset to structure */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:16:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t off_dt_strings; /* offset to strings */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:17:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t off_mem_rsvmap; /* offset to memory reserve map */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:18:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t version; /* format version */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:19:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t last_comp_version; /* last compatible version */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:22:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t boot_cpuid_phys; /* Which physical CPU id we're
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:25:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t size_dt_strings; /* size of the strings block */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:28:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t size_dt_struct; /* size of the structure block */
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
>> scripts/dtc/libfdt/fdt.h:32:2: error: unknown type name 'fdt64_t'; did you mean 'int64_t'?
fdt64_t address;
^~~~~~~
int64_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:27:19: note: 'int64_t' declared here
typedef __int64_t int64_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:33:2: error: unknown type name 'fdt64_t'; did you mean 'int64_t'?
fdt64_t size;
^~~~~~~
int64_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:27:19: note: 'int64_t' declared here
typedef __int64_t int64_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:37:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t tag;
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:42:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t tag;
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:43:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t len;
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
In file included from scripts/dtc/fdtdump.c:12:
scripts/dtc/libfdt/fdt.h:44:2: error: unknown type name 'fdt32_t'; did you mean 'int32_t'?
fdt32_t nameoff;
^~~~~~~
int32_t
/usr/include/x86_64-linux-gnu/bits/stdint-intn.h:26:19: note: 'int32_t' declared here
typedef __int32_t int32_t;
^
>> scripts/dtc/fdtdump.c:156:28: error: too few arguments to function call, expected 2, have 1
buf = utilfdt_read(argv[1]);
~~~~~~~~~~~~ ^
scripts/dtc/util.h:97:7: note: 'utilfdt_read' declared here
char *utilfdt_read(const char *filename, size_t *len);
^
17 errors generated.
make[2]: *** [scripts/Makefile.host:112: scripts/dtc/fdtdump.o] Error 1
make[2]: Target '__build' not remade because of errors.
make[1]: *** [Makefile:1356: scripts_dtc] Error 2
make[1]: Target 'prepare' not remade because of errors.
make: *** [Makefile:185: __sub-make] Error 2
make: Target 'prepare' not remade because of errors.


vim +13 scripts/dtc/libfdt/fdt.h

1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 11
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 12 struct fdt_header {
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 @13 fdt32_t magic; /* magic word FDT_MAGIC */
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 14 fdt32_t totalsize; /* total size of DT block */
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 15 fdt32_t off_dt_struct; /* offset to structure */
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 16 fdt32_t off_dt_strings; /* offset to strings */
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 17 fdt32_t off_mem_rsvmap; /* offset to memory reserve map */
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 18 fdt32_t version; /* format version */
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 19 fdt32_t last_comp_version; /* last compatible version */
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 20
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 21 /* version 2 fields below */
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 22 fdt32_t boot_cpuid_phys; /* Which physical CPU id we're
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 23 booting on */
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 24 /* version 3 fields below */
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 25 fdt32_t size_dt_strings; /* size of the strings block */
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 26
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 27 /* version 17 fields below */
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 28 fdt32_t size_dt_struct; /* size of the structure block */
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 29 };
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 30
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 31 struct fdt_reserve_entry {
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 @32 fdt64_t address;
4760597116e34bd scripts/dtc/libfdt/fdt.h Rob Herring 2015-04-29 33 fdt64_t size;
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 34 };
1cade99497c881a arch/powerpc/boot/libfdt/fdt.h David Gibson 2007-12-10 35

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip