[U-Boot] Please pull u-boot-dm
Hi Tom,
https://travis-ci.org/sglass68/u-boot/builds/474968097
The following changes since commit f97c49d6a2f504d0f0a8aab37c67aa314e006250:
Merge branch 'master' of git://git.denx.de/u-boot-sunxi (2019-01-01 19:55:05 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-pull-4jan19
for you to fetch changes up to 1b4770dfe03b5dd1dfd2a6076c4c85ee566e360a:
sandbox: Correct SDL build flags (2019-01-03 11:06:07 -0700)
---------------------------------------------------------------- Buildman clang support and a few fixes Small fixes to 'dm tree' and regmap test Improve sandbox build compatibility
---------------------------------------------------------------- Christian GMEINER (1): sandbox: add memset_io(..), memcpy_fromio(..) and memcpy_toio(..)
Sekhar Nori (1): common: fdt_support: print hexadecimal numbers in debug
Simon Glass (8): log: Check printf() arguments buildman: Drop comment about Ctrl-C problem buildman: Add support for building with clang travis: Use buildman for building with clang dm: Tidy up 'dm tree' output when there are many devices test: dm: regmap: Fix the long test delay buildman: Deal nicely with invalid build-status file sandbox: Correct SDL build flags
.travis.yml | 13 +++---------- arch/sandbox/config.mk | 2 -- arch/sandbox/cpu/sdl.c | 2 +- arch/sandbox/include/asm/io.h | 12 ++++++++++++ common/fdt_support.c | 4 ++-- drivers/core/dump.c | 4 ++-- include/log.h | 3 ++- include/regmap.h | 14 +++++++++++++- test/dm/regmap.c | 7 ++++--- test/py/tests/test_bind.py | 3 ++- tools/buildman/README | 12 ++++++++++-- tools/buildman/builderthread.py | 7 ++++++- tools/buildman/cmdline.py | 2 ++ tools/buildman/control.py | 2 +- tools/buildman/toolchain.py | 18 ++++++++++++++---- 15 files changed, 74 insertions(+), 31 deletions(-)
Regards, Simon
On Fri, Jan 04, 2019 at 01:47:08PM -0700, Simon Glass wrote:
Hi Tom,
https://travis-ci.org/sglass68/u-boot/builds/474968097
The following changes since commit f97c49d6a2f504d0f0a8aab37c67aa314e006250:
Merge branch 'master' of git://git.denx.de/u-boot-sunxi (2019-01-01 19:55:05 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-pull-4jan19
for you to fetch changes up to 1b4770dfe03b5dd1dfd2a6076c4c85ee566e360a:
sandbox: Correct SDL build flags (2019-01-03 11:06:07 -0700)
So, the clang switch doesn't catch warnings anymore. While travis is clean, my local hand-build shows: /home/trini/u-boot/u-boot/lib/efi_loader/efi_file.c:224:61: warning: format specifies type 'wchar_t *' (aka 'unsigned short *') but the argument has type 's16 *' (aka 'short *') [-Wformat] EFI_ENTRY("%p, %p, "%ls", %llx, %llu", file, new_handle, file_name, ~~~ ^~~~~~~~~ /home/trini/u-boot/u-boot/include/efi_loader.h:40:15: note: expanded from macro 'EFI_ENTRY' __func__, ##__VA_ARGS__); \ ^~~~~~~~~~~ /home/trini/u-boot/u-boot/include/log.h:164:28: note: expanded from macro 'debug' debug_cond(_DEBUG, fmt, ##args) ^~~~ /home/trini/u-boot/u-boot/include/log.h:144:41: note: expanded from macro 'debug_cond' log(LOG_CATEGORY, LOGL_DEBUG, fmt, ##args); \ ^~~~ /home/trini/u-boot/u-boot/include/log.h:121:25: note: expanded from macro 'log' pr_fmt(_fmt), ##_args); \ ^~~~~ 1 warning generated.
So the log changes have a problem, and the buildman integration doesn't promote warnings right :(
Hi Tom,
On Fri, 4 Jan 2019 at 22:00, Tom Rini trini@konsulko.com wrote:
On Fri, Jan 04, 2019 at 01:47:08PM -0700, Simon Glass wrote:
Hi Tom,
https://travis-ci.org/sglass68/u-boot/builds/474968097
The following changes since commit f97c49d6a2f504d0f0a8aab37c67aa314e006250:
Merge branch 'master' of git://git.denx.de/u-boot-sunxi (2019-01-01 19:55:05 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git tags/dm-pull-4jan19
for you to fetch changes up to 1b4770dfe03b5dd1dfd2a6076c4c85ee566e360a:
sandbox: Correct SDL build flags (2019-01-03 11:06:07 -0700)
So, the clang switch doesn't catch warnings anymore. While travis is clean, my local hand-build shows: /home/trini/u-boot/u-boot/lib/efi_loader/efi_file.c:224:61: warning: format specifies type 'wchar_t *' (aka 'unsigned short *') but the argument has type 's16 *' (aka 'short *') [-Wformat] EFI_ENTRY("%p, %p, "%ls", %llx, %llu", file, new_handle, file_name, ~~~ ^~~~~~~~~ /home/trini/u-boot/u-boot/include/efi_loader.h:40:15: note: expanded from macro 'EFI_ENTRY' __func__, ##__VA_ARGS__); \ ^~~~~~~~~~~ /home/trini/u-boot/u-boot/include/log.h:164:28: note: expanded from macro 'debug' debug_cond(_DEBUG, fmt, ##args) ^~~~ /home/trini/u-boot/u-boot/include/log.h:144:41: note: expanded from macro 'debug_cond' log(LOG_CATEGORY, LOGL_DEBUG, fmt, ##args); \ ^~~~ /home/trini/u-boot/u-boot/include/log.h:121:25: note: expanded from macro 'log' pr_fmt(_fmt), ##_args); \ ^~~~~ 1 warning generated.
So the log changes have a problem, and the buildman integration doesn't promote warnings right :(
Actually this uncovered a problem that I had not noticed before. I sent a new series to take care of it. But given that today is release day, I am unfortunately a little late!
Regards, Simon
participants (2)
-
Simon Glass -
Tom Rini