[U-Boot] Please pull u-boot-dm

Simon Glass sjg at chromium.org
Tue Jan 8 00:38:13 UTC 2019


Hi Tom,

On Fri, 4 Jan 2019 at 22:00, Tom Rini <trini at 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


More information about the U-Boot mailing list