Re: [U-Boot] [PATCH v3 0/8] add clang support for some ARM boards

2014-09-11 Thread Jeroen Hofstee
Hello Albert, On 11-09-14 15:31, Albert ARIBAUD wrote: Thanks, tested building rpi_b, it works now. The, tested on versatileqemu out of curiosity and got the following results: 1. clang warns about Unused static functions in common/console.c, namely console_printdevs and console_doenv (1). Wh

Re: [U-Boot] [PATCH v3 0/8] add clang support for some ARM boards

2014-09-11 Thread Jeroen Hofstee
Hello Albert, On 11-09-14 17:43, Albert ARIBAUD wrote: Hi Jeroen, Correction on the asm stuff: On Thu, 11 Sep 2014 13:17:20 +0200, Jeroen Hofstee wrote: clang errors on arch/arm/lib/cache.c:28 for this: asm("0: mrc p15, 0, r15, c7, c10, 3\n\t" "bne 0b\n" : : : "memory"); and that is a clang

Re: [U-Boot] [PATCH v3 0/8] add clang support for some ARM boards

2014-09-11 Thread Albert ARIBAUD
Hi Jeroen, Correction on the asm stuff: On Thu, 11 Sep 2014 13:17:20 +0200, Jeroen Hofstee wrote: > > clang errors on arch/arm/lib/cache.c:28 for this: > > asm("0: mrc p15, 0, r15, c7, c10, 3\n\t" "bne 0b\n" : : : "memory"); > > and that is a clang mistake, as for ARM926EJS r15 is a valid (albe

Re: [U-Boot] [PATCH v3 0/8] add clang support for some ARM boards

2014-09-11 Thread Albert ARIBAUD
Hi Jeroen, On Wed, 10 Sep 2014 20:08:50 +0200, Jeroen Hofstee wrote: > Changes since v2: >- As Albert pointed out the clang instructions don't work with > Debian based binary packages. While I was aware of that it is > for a different reason then I thought, it is not that ARM is no

Re: [U-Boot] [PATCH v3 0/8] add clang support for some ARM boards

2014-09-11 Thread Albert ARIBAUD
Hi Jeroen, On Thu, 11 Sep 2014 13:17:20 +0200, Jeroen Hofstee wrote: > Hello Albert, > > On do, 2014-09-11 at 10:32 +0200, Albert ARIBAUD wrote: > > > > > On Wed, 10 Sep 2014 20:08:50 +0200, Jeroen Hofstee > > wrote: > > > > > Changes since v2: > > >- As Albert pointed out the clang ins

Re: [U-Boot] [PATCH v3 0/8] add clang support for some ARM boards

2014-09-11 Thread Jeroen Hofstee
Hello Albert, On do, 2014-09-11 at 10:32 +0200, Albert ARIBAUD wrote: > > On Wed, 10 Sep 2014 20:08:50 +0200, Jeroen Hofstee > wrote: > > > Changes since v2: > >- As Albert pointed out the clang instructions don't work with > > Debian based binary packages. While I was aware of that i

Re: [U-Boot] [PATCH v3 0/8] add clang support for some ARM boards

2014-09-11 Thread Albert ARIBAUD
Hi Jeroen, On Wed, 10 Sep 2014 20:08:50 +0200, Jeroen Hofstee wrote: > Changes since v2: >- As Albert pointed out the clang instructions don't work with > Debian based binary packages. While I was aware of that it is > for a different reason then I thought, it is not that ARM is no

[U-Boot] [PATCH v3 0/8] add clang support for some ARM boards

2014-09-10 Thread Jeroen Hofstee
Changes since v2: - As Albert pointed out the clang instructions don't work with Debian based binary packages. While I was aware of that it is for a different reason then I thought, it is not that ARM is not enabled as a backend but older versions are a bit more picky on the