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