[U-Boot] [PATCH v1] Makefile: Don't generate position independent code

[Heinrich Schuchardt]

On 08/09/2018 11:54 AM, Bin Meng wrote:
> Hi Andy,
> 
> On Thu, Aug 9, 2018 at 5:43 PM, Andy Shevchenko
> <andy.shevchenko at gmail.com> wrote:
>> On Mon, Aug 6, 2018 at 10:49 PM, Andy Shevchenko
>> <andy.shevchenko at gmail.com> wrote:
>>> On Mon, Aug 6, 2018 at 10:40 PM, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>>>> On 08/06/2018 08:55 PM, Andy Shevchenko wrote:
>>>>> On Mon, 2018-08-06 at 19:32 +0200, Heinrich Schuchardt wrote:
>>>>>> On 08/06/2018 07:11 PM, Andy Shevchenko wrote:
>>>>>>> On Mon, 2018-08-06 at 18:56 +0200, Heinrich Schuchardt wrote:
>>>>>>>> On 08/06/2018 06:00 PM, Andy Shevchenko wrote:
>>>>>>>>> Fix all these by disabling PIE on Makefile level.
>>>>>>>> With the patch building with gcc-8.1 works on i386.
>>>>>>>
>>>>>>> Does it mean you are actually run it and it works?
>>>>>
>>>>> Can you confirm that binary you got is working for you?
>>>>
>>>> I can start qemu-x86_defconfig and run bootefi selftest with your patch.
>>>>
>>>> odroid-c2_defconfig can boot both with either booti or bootefi with your
>>>> patch.
>>>
>>> Thanks for testing!
>>
>> Bin, can we get this applied? For now (*) it's a bit of burden to run
>> make CC=gcc-7 each time I would like U-Boot to be built.
>>
> 
> This was not assigned to me in patchwork. But yes, I can take this one
> via the x86 tree.
> 
> Heinrich,
> 
> I assume I can add your "Tested-by" tag since from patchwork I don't
> see this patch has any tags associated. Let me know if this is OK.

You can add the tag. But, please, observe that I did not test on the
following architectures: RISC-V, ARM 32bit. I did not test loading of
Linux on x86.

Best regards

Heinrich

> 
>> (*) It seems Debian (-like) distros, one of which we are using as an
>> OS on build machines, switched to gcc-8 by default and this is a
>> problem for native compilation.
>>
>> --
> 
> Regards,
> Bin
>