[Buildroot] [PATCH] microblaze: fix musl support

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Fri Oct 28 13:15:38 UTC 2016


Hello,

On Sun, 23 Oct 2016 21:26:16 +0200, Waldemar Brodkorb wrote:
> For musl we need patches for bintils 2.25.1 and 2.26.1.
> Binutils 2.27 and gcc 6.2.x does not work for microblaze,
> even not for uClibc-ng or glibc.
> For gcc 5.4.x the existing patch need reworking so that
> musl and uClibc-ng is supported.
> 
> Signed-off-by: Waldemar Brodkorb <wbx at openadk.org>

I've applied after making a few changes, and I also have a few
questions, see below.

> diff --git a/package/binutils/2.25.1/800-microblaze-musl.patch b/package/binutils/2.25.1/800-microblaze-musl.patch
> new file mode 100644
> index 0000000..84b91a2
> --- /dev/null
> +++ b/package/binutils/2.25.1/800-microblaze-musl.patch
> @@ -0,0 +1,18 @@
> +From here:
> +https://sourceware.org/bugzilla/show_bug.cgi?id=19516

Please include a real description in patches. I took the one from the
Git commit.

>  	config BR2_BINUTILS_VERSION_2_27_X
>  		bool "binutils 2.27"
> +		# supported but broken on Microblaze
> +		depends on !BR2_microblaze

Once you have reported the issue upstream, please send a patch to add a
link to the upstream issue. I really don't like when we exclude newer
version of tools, as it means that in a few months time, when we'll
want to remove support for binutils 2.26, Microblaze will no longer be
usable at all.


> @@ -73,6 +75,8 @@ choice
>  		bool "gcc 6.x"
>  		# Broken or unsupported architectures
>  		depends on !BR2_arc
> +		depends on !BR2_microblazeel
> +		depends on !BR2_microblazebe

I've changed that to use BR2_microblaze instead

Same comment here: please send a patch adding a link to the upstream
issue.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com



More information about the buildroot mailing list