[Buildroot] [PATCH] mpd: enable package for external ARC toolchain

[Buildroot] [PATCH] mpd: enable package for external ARC toolchain

[Evgeniy Didin]

Now using prebuilt arc-2017.09-rc1 toolchain it is possible to
build mpd package. The GCC patch solving the problem with pthread
support check now is the part of ARC toolchain.

Note this patch has a prerequisite that updates ARC prebuilt toolchain,
http://patchwork.ozlabs.org/patch/824311/

Signed-off-by: Evgeniy Didin <didin@synopsys.com>
CC: Alexey Brodkin <abrodkin@synopsys.com>
CC: arc-buildroot at synopsys.com
CC: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

---
 package/mpd/Config.in | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/package/mpd/Config.in b/package/mpd/Config.in
index 5991b6f4bb..b7b21672fc 100644
--- a/package/mpd/Config.in
+++ b/package/mpd/Config.in
@@ -7,8 +7,6 @@ menuconfig BR2_PACKAGE_MPD
 	depends on BR2_TOOLCHAIN_HAS_ATOMIC
 	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # C++14
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # exception_ptr
-	# ARC toolchain issue
-	depends on !BR2_TOOLCHAIN_EXTERNAL_SYNOPSYS_ARC
 	select BR2_PACKAGE_BOOST
 	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
 	select BR2_PACKAGE_MPD_TREMOR if !(BR2_PACKAGE_MPD_MAD || BR2_PACKAGE_MPD_MPG123 || BR2_PACKAGE_MPD_VORBIS || BR2_PACKAGE_MPD_WAVPACK || BR2_PACKAGE_MPD_FLAC || BR2_PACKAGE_MPD_MUSEPACK || BR2_PACKAGE_MPD_FFMPEG)
@@ -320,7 +318,6 @@ comment "mpd needs a toolchain w/ C++, threads, wchar, gcc >= 4.9"
 	depends on BR2_TOOLCHAIN_HAS_ATOMIC
 	depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \
 		!BR2_TOOLCHAIN_HAS_THREADS || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
-	depends on !BR2_TOOLCHAIN_EXTERNAL_SYNOPSYS_ARC
 
 comment "mpd needs a toolchain not affected by GCC bug 64735"
 	depends on BR2_TOOLCHAIN_HAS_GCC_BUG_64735
-- 
2.11.0

[Buildroot] [PATCH] mpd: enable package for external ARC toolchain

[Arnout Vandecappelle]

 Hi Evgeny,

On 11-10-17 14:16, Evgeniy Didin wrote:
> Now using prebuilt arc-2017.09-rc1 toolchain it is possible to
> build mpd package. The GCC patch solving the problem with pthread
> support check now is the part of ARC toolchain.
> 
> Note this patch has a prerequisite that updates ARC prebuilt toolchain,
> http://patchwork.ozlabs.org/patch/824311/

 In this case you should send the patches as a series. And if you don't, such a
comment should be below the --- line because it's not something that should stay
in the history.

 Otherwise:
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

 Regards,
 Arnout
> 
> Signed-off-by: Evgeniy Didin <didin@synopsys.com>
> CC: Alexey Brodkin <abrodkin@synopsys.com>
> CC: arc-buildroot at synopsys.com
> CC: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> 
> ---
>  package/mpd/Config.in | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/package/mpd/Config.in b/package/mpd/Config.in
> index 5991b6f4bb..b7b21672fc 100644
> --- a/package/mpd/Config.in
> +++ b/package/mpd/Config.in
> @@ -7,8 +7,6 @@ menuconfig BR2_PACKAGE_MPD
>  	depends on BR2_TOOLCHAIN_HAS_ATOMIC
>  	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 # C++14
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # exception_ptr
> -	# ARC toolchain issue
> -	depends on !BR2_TOOLCHAIN_EXTERNAL_SYNOPSYS_ARC
>  	select BR2_PACKAGE_BOOST
>  	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
>  	select BR2_PACKAGE_MPD_TREMOR if !(BR2_PACKAGE_MPD_MAD || BR2_PACKAGE_MPD_MPG123 || BR2_PACKAGE_MPD_VORBIS || BR2_PACKAGE_MPD_WAVPACK || BR2_PACKAGE_MPD_FLAC || BR2_PACKAGE_MPD_MUSEPACK || BR2_PACKAGE_MPD_FFMPEG)
> @@ -320,7 +318,6 @@ comment "mpd needs a toolchain w/ C++, threads, wchar, gcc >= 4.9"
>  	depends on BR2_TOOLCHAIN_HAS_ATOMIC
>  	depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \
>  		!BR2_TOOLCHAIN_HAS_THREADS || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
> -	depends on !BR2_TOOLCHAIN_EXTERNAL_SYNOPSYS_ARC
>  
>  comment "mpd needs a toolchain not affected by GCC bug 64735"
>  	depends on BR2_TOOLCHAIN_HAS_GCC_BUG_64735
> 

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

[Buildroot] [PATCH] mpd: enable package for external ARC toolchain

[Thomas Petazzoni]

Hello,

On Wed, 11 Oct 2017 22:14:38 +0200, Arnout Vandecappelle wrote:
>  Hi Evgeny,
> 
> On 11-10-17 14:16, Evgeniy Didin wrote:
> > Now using prebuilt arc-2017.09-rc1 toolchain it is possible to
> > build mpd package. The GCC patch solving the problem with pthread
> > support check now is the part of ARC toolchain.
> > 
> > Note this patch has a prerequisite that updates ARC prebuilt toolchain,
> > http://patchwork.ozlabs.org/patch/824311/  

The ARC prebuilt toolchain is not tested by our autobuilders. The two
ARC configurations that we have are:

support/config-fragments/autobuild/br-arc-full-internal.config
support/config-fragments/autobuild/br-arcle-hs38.config

The first is a Buildroot internal toolchain, the second is an external
toolchain, built with Buildroot.

So we are not testing the Synopsys-provided pre-built toolchain, and
therefore the build failures that were detected in mpd and vlc were not
caused by Synopsys pre-built toolchain.

Best regards,

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

[Buildroot] [PATCH] mpd: enable package for external ARC toolchain

[Alexey Brodkin]

Hi Thomas,

On Wed, 2017-10-11 at 23:19 +0200, Thomas Petazzoni wrote:
> Hello,
> 
> On Wed, 11 Oct 2017 22:14:38 +0200, Arnout Vandecappelle wrote:
> > 
> > ?Hi Evgeny,
> > 
> > On 11-10-17 14:16, Evgeniy Didin wrote:
> > > 
> > > Now using prebuilt arc-2017.09-rc1 toolchain it is possible to
> > > build mpd package. The GCC patch solving the problem with pthread
> > > support check now is the part of ARC toolchain.
> > > 
> > > Note this patch has a prerequisite that updates ARC prebuilt toolchain,
> > > https://urldefense.proofpoint.com/v2/url?u=http-3A__patchwork.ozlabs.org_patch_824311_&d=DwICAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=lqdeeSSEes0GFDDl656eV
> > > iXO7breS55ytWkhpk5R81I&m=g9SyhkcD7kkLjdVnxy_0PIxm6vf8vG9VuhheaCLHUOM&s=iTUcPjYUEl_yRBzAeiF08HNRskfS5hohJtPn80zE064&e=??
> 
> The ARC prebuilt toolchain is not tested by our autobuilders. The two
> ARC configurations that we have are:
> 
> support/config-fragments/autobuild/br-arc-full-internal.config
> support/config-fragments/autobuild/br-arcle-hs38.config
> 
> The first is a Buildroot internal toolchain, the second is an external
> toolchain, built with Buildroot.
> 
> So we are not testing the Synopsys-provided pre-built toolchain, and
> therefore the build failures that were detected in mpd and vlc were not
> caused by Synopsys pre-built toolchain.

Indeed, so then that note about prerequisite makes no sense.
Evgeniy will resend this patch with fixed commit message shortly.

-Alexey

[Buildroot] [PATCH] mpd: enable package for external ARC toolchain

[Thomas Petazzoni]

Hello,

On Wed, 11 Oct 2017 15:16:17 +0300, Evgeniy Didin wrote:
> Now using prebuilt arc-2017.09-rc1 toolchain it is possible to
> build mpd package. The GCC patch solving the problem with pthread
> support check now is the part of ARC toolchain.
> 
> Note this patch has a prerequisite that updates ARC prebuilt toolchain,
> http://patchwork.ozlabs.org/patch/824311/
> 
> Signed-off-by: Evgeniy Didin <didin@synopsys.com>
> CC: Alexey Brodkin <abrodkin@synopsys.com>
> CC: arc-buildroot at synopsys.com
> CC: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> 
> ---
>  package/mpd/Config.in | 3 ---
>  1 file changed, 3 deletions(-)

As was pointed out in the comments, this is only applicable once the
ARC external toolchain has been updated. So could you please send both
patches as a patch series, once the final version of the 2017.09 ARC
external toolchain is available?

In the mean time I'll mark the patch as Changes Requested in patchwork.

Thanks!

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