Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4
@ 2016-02-24 12:20 Gustavo Zacarias
  2016-02-24 12:20 ` [Buildroot] [PATCH 2/2] qt: properly enable script aarch64 support Gustavo Zacarias
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Gustavo Zacarias @ 2016-02-24 12:20 UTC (permalink / raw)
  To: buildroot

They use __sync_add_and_fetch_4() and other 32-bit atomics, fixes:
http://autobuild.buildroot.net/results/7ac/7acdbc54e26ad2ba4756512614192be0a086f563/

Also remove sparc from supported architectures since it won't grow
atomics support magically.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/qt/Config.in | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/package/qt/Config.in b/package/qt/Config.in
index bf14724..1579b61 100644
--- a/package/qt/Config.in
+++ b/package/qt/Config.in
@@ -407,12 +407,13 @@ config BR2_PACKAGE_QT_NETWORK
 
 config BR2_PACKAGE_QT_ARCH_SUPPORTS_WEBKIT
 	bool
+	depends on BR2_TOOLCHAIN_HAS_SYNC_4
 	# see src/3rdparty/webkit/Source/JavaScriptCore/wtf/Platform.h
 	# see http://lists.busybox.net/pipermail/buildroot/2014-November/112605.html
 	default y if BR2_arc || BR2_arm || BR2_armeb || BR2_i386 || BR2_microblazeel || \
 		BR2_microblazebe || BR2_mips || BR2_mipsel || \
 		(BR2_mips64 || BR2_mips64el) && !BR2_MIPS_NABI32 || BR2_powerpc || \
-		BR2_powerpc64 || BR2_powerpc64le || BR2_sparc || BR2_x86_64
+		BR2_powerpc64 || BR2_powerpc64le || BR2_x86_64
 	# The CodeSourcery SuperH toolchain fails to build Webkit,
 	# with an assertion failure in binutils.
 	default y if (BR2_sh4 || BR2_sh4eb || BR2_sh4a || BR2_sh4aeb) && \
@@ -454,12 +455,13 @@ config BR2_PACKAGE_QT_OPENSSL
 
 config BR2_PACKAGE_QT_ARCH_SUPPORTS_SCRIPT
 	bool
+	depends on BR2_TOOLCHAIN_HAS_SYNC_4
 	# see http://lists.busybox.net/pipermail/buildroot/2014-November/112605.html
 	default y if BR2_arc || BR2_arm || BR2_armeb || aarch64 || BR2_i386 || \
 		BR2_microblazeel || BR2_microblazebe || BR2_mips || BR2_mipsel || \
 		BR2_mips64 || BR2_mips64el || BR2_nios2 || BR2_powerpc || \
 		BR2_powerpc64 || BR2_powerpc64le || BR2_sh4 || BR2_sh4eb || \
-		BR2_sh4a || BR2_sh4aeb || BR2_sparc || BR2_x86_64
+		BR2_sh4a || BR2_sh4aeb || BR2_x86_64
 
 config BR2_PACKAGE_QT_SCRIPT
 	bool "Script Module"
-- 
2.4.10

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH 2/2] qt: properly enable script aarch64 support
  2016-02-24 12:20 [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4 Gustavo Zacarias
@ 2016-02-24 12:20 ` Gustavo Zacarias
  2016-02-24 22:29   ` Peter Korsgaard
  2016-02-24 21:09 ` [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4 Arnout Vandecappelle
  2016-02-24 22:29 ` Peter Korsgaard
  2 siblings, 1 reply; 6+ messages in thread
From: Gustavo Zacarias @ 2016-02-24 12:20 UTC (permalink / raw)
  To: buildroot

It's BR2_aarch64, not aarch64 alone.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/qt/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/qt/Config.in b/package/qt/Config.in
index 1579b61..248af00 100644
--- a/package/qt/Config.in
+++ b/package/qt/Config.in
@@ -457,7 +457,7 @@ config BR2_PACKAGE_QT_ARCH_SUPPORTS_SCRIPT
 	bool
 	depends on BR2_TOOLCHAIN_HAS_SYNC_4
 	# see http://lists.busybox.net/pipermail/buildroot/2014-November/112605.html
-	default y if BR2_arc || BR2_arm || BR2_armeb || aarch64 || BR2_i386 || \
+	default y if BR2_arc || BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || \
 		BR2_microblazeel || BR2_microblazebe || BR2_mips || BR2_mipsel || \
 		BR2_mips64 || BR2_mips64el || BR2_nios2 || BR2_powerpc || \
 		BR2_powerpc64 || BR2_powerpc64le || BR2_sh4 || BR2_sh4eb || \
-- 
2.4.10

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4
  2016-02-24 12:20 [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4 Gustavo Zacarias
  2016-02-24 12:20 ` [Buildroot] [PATCH 2/2] qt: properly enable script aarch64 support Gustavo Zacarias
@ 2016-02-24 21:09 ` Arnout Vandecappelle
  2016-02-24 21:12   ` Arnout Vandecappelle
  2016-02-24 22:29 ` Peter Korsgaard
  2 siblings, 1 reply; 6+ messages in thread
From: Arnout Vandecappelle @ 2016-02-24 21:09 UTC (permalink / raw)
  To: buildroot

On 02/24/16 13:20, Gustavo Zacarias wrote:
> They use __sync_add_and_fetch_4() and other 32-bit atomics, fixes:
> http://autobuild.buildroot.net/results/7ac/7acdbc54e26ad2ba4756512614192be0a086f563/
> 
> Also remove sparc from supported architectures since it won't grow
> atomics support magically.
> 
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

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


 Regards,
 Arnout


> ---
>  package/qt/Config.in | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/package/qt/Config.in b/package/qt/Config.in
> index bf14724..1579b61 100644
> --- a/package/qt/Config.in
> +++ b/package/qt/Config.in
> @@ -407,12 +407,13 @@ config BR2_PACKAGE_QT_NETWORK
>  
>  config BR2_PACKAGE_QT_ARCH_SUPPORTS_WEBKIT
>  	bool
> +	depends on BR2_TOOLCHAIN_HAS_SYNC_4
>  	# see src/3rdparty/webkit/Source/JavaScriptCore/wtf/Platform.h
>  	# see http://lists.busybox.net/pipermail/buildroot/2014-November/112605.html
>  	default y if BR2_arc || BR2_arm || BR2_armeb || BR2_i386 || BR2_microblazeel || \
>  		BR2_microblazebe || BR2_mips || BR2_mipsel || \
>  		(BR2_mips64 || BR2_mips64el) && !BR2_MIPS_NABI32 || BR2_powerpc || \
> -		BR2_powerpc64 || BR2_powerpc64le || BR2_sparc || BR2_x86_64
> +		BR2_powerpc64 || BR2_powerpc64le || BR2_x86_64
>  	# The CodeSourcery SuperH toolchain fails to build Webkit,
>  	# with an assertion failure in binutils.
>  	default y if (BR2_sh4 || BR2_sh4eb || BR2_sh4a || BR2_sh4aeb) && \
> @@ -454,12 +455,13 @@ config BR2_PACKAGE_QT_OPENSSL
>  
>  config BR2_PACKAGE_QT_ARCH_SUPPORTS_SCRIPT
>  	bool
> +	depends on BR2_TOOLCHAIN_HAS_SYNC_4
>  	# see http://lists.busybox.net/pipermail/buildroot/2014-November/112605.html
>  	default y if BR2_arc || BR2_arm || BR2_armeb || aarch64 || BR2_i386 || \
>  		BR2_microblazeel || BR2_microblazebe || BR2_mips || BR2_mipsel || \
>  		BR2_mips64 || BR2_mips64el || BR2_nios2 || BR2_powerpc || \
>  		BR2_powerpc64 || BR2_powerpc64le || BR2_sh4 || BR2_sh4eb || \
> -		BR2_sh4a || BR2_sh4aeb || BR2_sparc || BR2_x86_64
> +		BR2_sh4a || BR2_sh4aeb || BR2_x86_64
>  
>  config BR2_PACKAGE_QT_SCRIPT
>  	bool "Script Module"
> 


-- 
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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4
  2016-02-24 21:09 ` [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4 Arnout Vandecappelle
@ 2016-02-24 21:12   ` Arnout Vandecappelle
  0 siblings, 0 replies; 6+ messages in thread
From: Arnout Vandecappelle @ 2016-02-24 21:12 UTC (permalink / raw)
  To: buildroot

On 02/24/16 22:09, Arnout Vandecappelle wrote:
> On 02/24/16 13:20, Gustavo Zacarias wrote:
>> They use __sync_add_and_fetch_4() and other 32-bit atomics, fixes:
>> http://autobuild.buildroot.net/results/7ac/7acdbc54e26ad2ba4756512614192be0a086f563/
>>
>> Also remove sparc from supported architectures since it won't grow
>> atomics support magically.
>>
>> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> 
> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

 Just to be clear:

- I checked in the source that only sync4 is used.

- I saw in the source that sparc support is hopeless.

 Regards,
 Arnout

> 
> 
>  Regards,
>  Arnout
> 
> 
>> ---
>>  package/qt/Config.in | 6 ++++--
>>  1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/package/qt/Config.in b/package/qt/Config.in
>> index bf14724..1579b61 100644
>> --- a/package/qt/Config.in
>> +++ b/package/qt/Config.in
>> @@ -407,12 +407,13 @@ config BR2_PACKAGE_QT_NETWORK
>>  
>>  config BR2_PACKAGE_QT_ARCH_SUPPORTS_WEBKIT
>>  	bool
>> +	depends on BR2_TOOLCHAIN_HAS_SYNC_4
>>  	# see src/3rdparty/webkit/Source/JavaScriptCore/wtf/Platform.h
>>  	# see http://lists.busybox.net/pipermail/buildroot/2014-November/112605.html
>>  	default y if BR2_arc || BR2_arm || BR2_armeb || BR2_i386 || BR2_microblazeel || \
>>  		BR2_microblazebe || BR2_mips || BR2_mipsel || \
>>  		(BR2_mips64 || BR2_mips64el) && !BR2_MIPS_NABI32 || BR2_powerpc || \
>> -		BR2_powerpc64 || BR2_powerpc64le || BR2_sparc || BR2_x86_64
>> +		BR2_powerpc64 || BR2_powerpc64le || BR2_x86_64
>>  	# The CodeSourcery SuperH toolchain fails to build Webkit,
>>  	# with an assertion failure in binutils.
>>  	default y if (BR2_sh4 || BR2_sh4eb || BR2_sh4a || BR2_sh4aeb) && \
>> @@ -454,12 +455,13 @@ config BR2_PACKAGE_QT_OPENSSL
>>  
>>  config BR2_PACKAGE_QT_ARCH_SUPPORTS_SCRIPT
>>  	bool
>> +	depends on BR2_TOOLCHAIN_HAS_SYNC_4
>>  	# see http://lists.busybox.net/pipermail/buildroot/2014-November/112605.html
>>  	default y if BR2_arc || BR2_arm || BR2_armeb || aarch64 || BR2_i386 || \
>>  		BR2_microblazeel || BR2_microblazebe || BR2_mips || BR2_mipsel || \
>>  		BR2_mips64 || BR2_mips64el || BR2_nios2 || BR2_powerpc || \
>>  		BR2_powerpc64 || BR2_powerpc64le || BR2_sh4 || BR2_sh4eb || \
>> -		BR2_sh4a || BR2_sh4aeb || BR2_sparc || BR2_x86_64
>> +		BR2_sh4a || BR2_sh4aeb || BR2_x86_64
>>  
>>  config BR2_PACKAGE_QT_SCRIPT
>>  	bool "Script Module"
>>
> 
> 


-- 
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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4
  2016-02-24 12:20 [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4 Gustavo Zacarias
  2016-02-24 12:20 ` [Buildroot] [PATCH 2/2] qt: properly enable script aarch64 support Gustavo Zacarias
  2016-02-24 21:09 ` [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4 Arnout Vandecappelle
@ 2016-02-24 22:29 ` Peter Korsgaard
  2 siblings, 0 replies; 6+ messages in thread
From: Peter Korsgaard @ 2016-02-24 22:29 UTC (permalink / raw)
  To: buildroot

>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

 > They use __sync_add_and_fetch_4() and other 32-bit atomics, fixes:
 > http://autobuild.buildroot.net/results/7ac/7acdbc54e26ad2ba4756512614192be0a086f563/

 > Also remove sparc from supported architectures since it won't grow
 > atomics support magically.

 > Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

Committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH 2/2] qt: properly enable script aarch64 support
  2016-02-24 12:20 ` [Buildroot] [PATCH 2/2] qt: properly enable script aarch64 support Gustavo Zacarias
@ 2016-02-24 22:29   ` Peter Korsgaard
  0 siblings, 0 replies; 6+ messages in thread
From: Peter Korsgaard @ 2016-02-24 22:29 UTC (permalink / raw)
  To: buildroot

>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

 > It's BR2_aarch64, not aarch64 alone.
 > Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

Committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-02-24 22:29 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-24 12:20 [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4 Gustavo Zacarias
2016-02-24 12:20 ` [Buildroot] [PATCH 2/2] qt: properly enable script aarch64 support Gustavo Zacarias
2016-02-24 22:29   ` Peter Korsgaard
2016-02-24 21:09 ` [Buildroot] [PATCH 1/2] qt: script & webkit depend on BR2_TOOLCHAIN_HAS_SYNC_4 Arnout Vandecappelle
2016-02-24 21:12   ` Arnout Vandecappelle
2016-02-24 22:29 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox