From: kernel test robot <lkp@intel.com>
To: Viresh Kumar <viresh.kumar@linaro.org>,
Pantelis Antoniou <pantelis.antoniou@konsulko.com>,
Frank Rowand <frowand.list@gmail.com>,
Rob Herring <robh+dt@kernel.org>
Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com,
Viresh Kumar <viresh.kumar@linaro.org>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-kbuild@vger.kernel.org,
Vincent Guittot <vincent.guittot@linaro.org>,
Bill Mills <bill.mills@linaro.org>,
anmar.oueja@linaro.org
Subject: Re: [PATCH] scripts: dtc: Start building fdtoverlay and fdtdump
Date: Wed, 6 Jan 2021 20:24:15 +0800 [thread overview]
Message-ID: <202101062025.AHg58mjK-lkp@intel.com> (raw)
In-Reply-To: <5a416b44155409e45b99c5624d66992a2138cd4c.1609927722.git.viresh.kumar@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 18181 bytes --]
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@intel.com>
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@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 35235 bytes --]
next prev parent reply other threads:[~2021-01-06 12:25 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-05 11:24 [RFC 0/2] kbuild: Add support to build overlays (%.dtbo) Viresh Kumar
2021-01-05 11:24 ` [RFC 1/2] " Viresh Kumar
2021-01-05 11:24 ` [RFC 2/2] scripts: dtc: Handle outform dtbo Viresh Kumar
2021-01-05 15:37 ` Rob Herring
2021-01-12 0:18 ` Frank Rowand
2021-01-05 15:21 ` [RFC 0/2] kbuild: Add support to build overlays (%.dtbo) Rob Herring
2021-01-06 10:09 ` [PATCH] scripts: dtc: Start building fdtoverlay and fdtdump Viresh Kumar
2021-01-06 12:24 ` kernel test robot [this message]
2021-01-06 15:52 ` Rob Herring
2021-01-07 5:28 ` [RFC 0/2] kbuild: Add support to build overlays (%.dtbo) Masahiro Yamada
2021-01-07 7:27 ` Viresh Kumar
2021-01-07 19:02 ` Rob Herring
2021-01-07 19:42 ` Bill Mills
2021-01-11 17:26 ` Masahiro Yamada
2021-01-11 11:17 ` Viresh Kumar
2021-01-11 15:40 ` Masahiro Yamada
2021-01-11 16:13 ` Rob Herring
2021-01-11 17:02 ` Masahiro Yamada
2021-01-12 9:41 ` Viresh Kumar
2021-01-12 0:25 ` Frank Rowand
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202101062025.AHg58mjK-lkp@intel.com \
--to=lkp@intel.com \
--cc=anmar.oueja@linaro.org \
--cc=bill.mills@linaro.org \
--cc=clang-built-linux@googlegroups.com \
--cc=devicetree@vger.kernel.org \
--cc=frowand.list@gmail.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pantelis.antoniou@konsulko.com \
--cc=robh+dt@kernel.org \
--cc=vincent.guittot@linaro.org \
--cc=viresh.kumar@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox