Build failure -- powerpc/boot: Add OPAL console to epapr wrappers

[Daniel Walker]

On 03/24/2017 07:16 PM, Oliver O'Halloran wrote:
> On Sat, Mar 25, 2017 at 4:00 AM, Daniel Walker <danielwa at cisco.com> wrote:
>> I get this build failure,
>>
>>
>> In file included from arch/powerpc/boot/fdt.c:51:
>> ../arch/powerpc/boot/libfdt_env.h:9: error: redefinition of typedef
>> 'uint32_t'
>> ../arch/powerpc/boot/types.h:20: note: previous declaration of 'uint32_t'
>> was here
>> ../arch/powerpc/boot/libfdt_env.h:10: error: redefinition of typedef
>> 'uint64_t'
>> ../arch/powerpc/boot/types.h:21: note: previous declaration of 'uint64_t'
>> was here
>> make[2]: *** [arch/powerpc/boot/fdt.o] Error 1
>> make[1]: *** [uImage] Error 2
>> make[1]: Leaving directory `/nobackup/danielwa/linux/t1040'
>> make: *** [sub-make] Error 2
>>
>>
>> and it bisects to ,
>>
>>
>> commit 656ad58ef19e2a763fa5c938b20ae0f6b8d67242
>> Author: Oliver O'Halloran <oohall at gmail.com>
>> Date:   Fri Jul 1 00:34:37 2016 +1000
>>
>>      powerpc/boot: Add OPAL console to epapr wrappers
>>
>>      This patch adds an OPAL console backend to the powerpc boot wrapper so
>>      that decompression failures inside the wrapper can be reported to the
>>      user. This is important since it typically indicates data corruption in
>>      the firmware and other nasty things.
>>
>>      Currently this only works when building a little endian kernel. When
>>      compiling a 64 bit BE kernel the wrapper is always build 32 bit to be
>>      compatible with some 32 bit firmwares. BE support will be added at a
>>      later date. Another limitation of this is that only the "raw" type of
>>      OPAL console is supported, however machines that provide a hvsi console
>>      also provide a raw console so this is not an issue in practice.
>>
>>      Actually-written-by: Benjamin Herrenschmidt <benh at kernel.crashing.org>
>>      Signed-off-by: Oliver O'Halloran <oohall at gmail.com>
>>      [mpe: Move #ifdef __powerpc64__ to avoid warnings on 32-bit]
>>      Signed-off-by: Michael Ellerman <mpe at ellerman.id.au>
>>
>>
>> I can provide a config file if needed. My apologies if this was already
>> reported.
> Thanks for the report, I don't think this is a known bug. mpe's build
> testing is pretty thorough so I'm surprised this wasn't caught sooner.
>
> A config file and the version of gcc that you're using would be useful.
>
> Oliver


Config attached , it's for a Fresecale t1042 machine. The GCC is custom 
based on 4.4.1 .


Daniel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: t1042.config.gz
Type: application/gzip
Size: 20934 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20170327/e3639d00/attachment-0001.gz>