On Tue, Jul 04, 2017 at 01:51:47PM +0200, Alexander Graf wrote: > > > > Am 04.07.2017 um 13:24 schrieb Tom Rini <tr...@konsulko.com>: > > > >> On Tue, Jul 04, 2017 at 07:27:39AM +0200, Alexander Graf wrote: > >> > >> > >>>> Am 03.07.2017 um 23:11 schrieb Tom Rini <tr...@konsulko.com>: > >>>> > >>>> On Mon, Jul 03, 2017 at 04:05:56PM +0200, Alexander Graf wrote: > >>>> > >>>> Hi Tom, > >>>> > >>>> This is my current patch queue for efi. Please pull. > >>>> > >>>> Alex > >>>> > >>>> > >>>> The following changes since commit > >>>> 8d6040c725bfa40fba1bd6b5f356ee713c1ac7d1: > >>>> > >>>> pico-imx7d: Remove bouncing email (2017-06-29 21:30:16 -0400) > >>>> > >>>> are available in the git repository at: > >>>> > >>>> git://github.com/agraf/u-boot.git tags/signed-efi-next > >>>> > >>>> for you to fetch changes up to a88456645ec60570d37c8be5877614a16467b790: > >>>> > >>>> efi_loader: efi_handle_protocol set attributes (2017-07-03 14:41:14 > >>>> +0200) > >>>> > >>> > >>> This gives me: > >>> w+(orangepi_plus) ../lib/efi_loader/efi_image_loader.c: In function > >>> ?efi_load_pe?: > >>> w+(orangepi_plus) ../lib/efi_loader/efi_image_loader.c:177:18: warning: > >>> cast from pointer to integer of different size [-Wpointer-to-int-cast] > >>> > >>> On tons and tons of boards, with gcc-6.3. > >> > >> Do you know why Travis didn't catch this? Do we need to update the gcc > >> version used there maybe? > > > > Are you sure it didn't catch it? It's a warning only and that doesn't > > cause the job to fail. > > Ah, so that's why! Maybe we should start to introduce -Werror by default?
I've been toying with that idea but first we need to fix the warnings that we have ;) And then make it easy enough to disable during local development. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
