[U-Boot] [PATCH] drivers: SPI: sunxi SPL: fix warning

[Andre Przywara]

On 14/11/16 20:12, Tom Rini wrote:
> On Mon, Nov 14, 2016 at 07:43:40PM +0000, André Przywara wrote:
> 
> [snip]
>> On a related matter, I ran buildman on HEAD for armv8 today and GCC 6.2
>> found quite some issues (will send out the fixes ASAP).
>> So is there some buildbot somewhere that runs buildman? If yes, with
>> what compilers?
> 
> What gcc-6.2 compilers are you using?

I just built my own, in fact for Trusty. I have some easy build scripts
here [1], though the README needs some update. With this I create .deb
packages for binutils and gcc-stage1 of (almost) any vanilla GCC version
you throw at it.

I don't know of any newer GCC in some proper repo, historically not many
people cared and either built their own or used a Linaro drop.

> The travis-ci setup is using the
> Ubuntu 'Trusty' compilers, which are not that recent.  I also have been
> running debian/unstable in a chroot, but it's old enough of a copy to be
> using gcc-5.3.1 currently.  I'm going to kick off a new debootstrap
> now'ish to see what toolchain versions I can get there.  Do you know if
> these gcc-6.2 problems you see would also show up for rpi3 in 64bit
> mode?

I didn't observe any runtime issues (though at least someone else did in
one occasion).
The compiler warnings are mostly due to the new GCC6 option
-Wmisleading-indentation, which seems to be enabled by default.
Some of them are really just wrong indentation, so cosmetic, but others
are genuine bugs. I will send patches later today ($work here atm).

> That's my current easiest test platform for aarch64.  Thanks!

As said this was observed only by running buildman, so you shouldn't
need any actual hardware to see this.

Cheers,
Andre.

[1] https://github.com/apritzel/cross