All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in
@ 2017-11-13 18:41 Giulio Benetti
  2017-11-13 18:41 ` [Buildroot] [PATCH v3 2/2] qt5base: add sunxi-mali-mainline eglfs backend Giulio Benetti
  2018-04-04 21:45 ` [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in Arnout Vandecappelle
  0 siblings, 2 replies; 7+ messages in thread
From: Giulio Benetti @ 2017-11-13 18:41 UTC (permalink / raw)
  To: buildroot

qt5base: correct eglfs support in qmake.conf.in

Setting EGLFS_DEVICE_INTEGRATION at the end of qmake.conf like is done by commit
0c219ddb8a doesn't work correctly: it has to be set before the
include(../common/linux_device_post.conf)

Instead of appending to the file, change it into a qmake.conf.in template file
that contains an EGLFS_DEVICE_INTEGRATION assignment and update it with sed.

Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
---
Changes v1 -> v2:
* write directly to package target qmake.conf
* simplified eglfs backend choice
 package/qt5/qt5base/{qmake.conf => qmake.conf.in} |  2 ++
 package/qt5/qt5base/qt5base.mk                    | 22 +++++++++++++++++-----
 2 files changed, 19 insertions(+), 5 deletions(-)
 rename package/qt5/qt5base/{qmake.conf => qmake.conf.in} (95%)

diff --git a/package/qt5/qt5base/qmake.conf b/package/qt5/qt5base/qmake.conf.in
similarity index 95%
rename from package/qt5/qt5base/qmake.conf
rename to package/qt5/qt5base/qmake.conf.in
index 8b6debe..fb698b9 100644
--- a/package/qt5/qt5base/qmake.conf
+++ b/package/qt5/qt5base/qmake.conf.in
@@ -24,5 +24,7 @@ QMAKE_CFLAGS_ISYSTEM   =
 # Architecturespecific configuration
 include(arch.conf)
 
+EGLFS_DEVICE_INTEGRATION = @EGLFS_DEVICE@
+
 include(../common/linux_device_post.conf)
 load(qt_config)
diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
index b3f9c79..ef99131 100644
--- a/package/qt5/qt5base/qt5base.mk
+++ b/package/qt5/qt5base/qt5base.mk
@@ -240,10 +240,23 @@ QT5BASE_INSTALL_LIBS_$(BR2_PACKAGE_QT5BASE_PRINTSUPPORT) += Qt5PrintSupport
 
 QT5BASE_INSTALL_LIBS_$(BR2_PACKAGE_QT5BASE_DBUS) += Qt5DBus
 
-ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST)$(BR2_PACKAGE_IMX_GPU_VIV),yy)
-# use vivante backend
+ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST),y)
+ifeq ($(BR2_PACKAGE_IMX_GPU_VIV),y)
+QT5BASE_EGLFS_DEVICE = eglfs_viv
+else ifeq ($(BR2_PACKAGE_SUNXI_MALI),y)
+QT5BASE_EGLFS_DEVICE = eglfs_mali
+endif
+endif
+
+ifeq ($(QT5BASE_EGLFS_DEVICE),)
 define QT5BASE_CONFIGURE_QMAKE_CONFIG
-	echo "EGLFS_DEVICE_INTEGRATION = eglfs_viv" >> \
+	sed '/@EGLFS_DEVICE@/,+1d' $(QT5BASE_PKGDIR)/qmake.conf.in > \
+		$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
+endef
+else
+define QT5BASE_CONFIGURE_QMAKE_CONFIG
+	sed 's/@EGLFS_DEVICE@/$(QT5BASE_EGLFS_DEVICE)/g' \
+		$(QT5BASE_PKGDIR)/qmake.conf.in > \
 		$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
 endef
 endif
@@ -263,8 +276,7 @@ endef
 endif
 
 define QT5BASE_CONFIGURE_CMDS
-	$(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qmake.conf \
-		$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
+	mkdir -p $(@D)/mkspecs/devices/linux-buildroot-g++/
 	$(QT5BASE_CONFIGURE_QMAKE_CONFIG)
 	$(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qplatformdefs.h \
 		$(@D)/mkspecs/devices/linux-buildroot-g++/qplatformdefs.h
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v3 2/2] qt5base: add sunxi-mali-mainline eglfs backend
  2017-11-13 18:41 [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in Giulio Benetti
@ 2017-11-13 18:41 ` Giulio Benetti
  2018-04-04 21:47   ` Arnout Vandecappelle
  2018-04-04 21:45 ` [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in Arnout Vandecappelle
  1 sibling, 1 reply; 7+ messages in thread
From: Giulio Benetti @ 2017-11-13 18:41 UTC (permalink / raw)
  To: buildroot

qt5base: add support for sunxi-mali-mainline eglfs backend

Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
---
 package/qt5/qt5base/qt5base.mk | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
index ef99131..6ea42ec 100644
--- a/package/qt5/qt5base/qt5base.mk
+++ b/package/qt5/qt5base/qt5base.mk
@@ -245,6 +245,8 @@ ifeq ($(BR2_PACKAGE_IMX_GPU_VIV),y)
 QT5BASE_EGLFS_DEVICE = eglfs_viv
 else ifeq ($(BR2_PACKAGE_SUNXI_MALI),y)
 QT5BASE_EGLFS_DEVICE = eglfs_mali
+else ifeq ($(BR2_PACKAGE_SUNXI_MALI_MAINLINE),y)
+QT5BASE_EGLFS_DEVICE = eglfs_mali
 endif
 endif
 
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in
  2017-11-13 18:41 [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in Giulio Benetti
  2017-11-13 18:41 ` [Buildroot] [PATCH v3 2/2] qt5base: add sunxi-mali-mainline eglfs backend Giulio Benetti
@ 2018-04-04 21:45 ` Arnout Vandecappelle
  2018-04-06 19:54   ` Giulio Benetti
  1 sibling, 1 reply; 7+ messages in thread
From: Arnout Vandecappelle @ 2018-04-04 21:45 UTC (permalink / raw)
  To: buildroot

 Hi Giulio,

On 13-11-17 19:41, Giulio Benetti wrote:
> qt5base: correct eglfs support in qmake.conf.in
> 
> Setting EGLFS_DEVICE_INTEGRATION at the end of qmake.conf like is done by commit
> 0c219ddb8a doesn't work correctly: it has to be set before the
> include(../common/linux_device_post.conf)
> 
> Instead of appending to the file, change it into a qmake.conf.in template file
> that contains an EGLFS_DEVICE_INTEGRATION assignment and update it with sed.
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
> ---
> Changes v1 -> v2:
> * write directly to package target qmake.conf
> * simplified eglfs backend choice
>  package/qt5/qt5base/{qmake.conf => qmake.conf.in} |  2 ++
>  package/qt5/qt5base/qt5base.mk                    | 22 +++++++++++++++++-----
>  2 files changed, 19 insertions(+), 5 deletions(-)
>  rename package/qt5/qt5base/{qmake.conf => qmake.conf.in} (95%)
> 
> diff --git a/package/qt5/qt5base/qmake.conf b/package/qt5/qt5base/qmake.conf.in
> similarity index 95%
> rename from package/qt5/qt5base/qmake.conf
> rename to package/qt5/qt5base/qmake.conf.in
> index 8b6debe..fb698b9 100644
> --- a/package/qt5/qt5base/qmake.conf
> +++ b/package/qt5/qt5base/qmake.conf.in
> @@ -24,5 +24,7 @@ QMAKE_CFLAGS_ISYSTEM   =
>  # Architecturespecific configuration
>  include(arch.conf)
>  
> +EGLFS_DEVICE_INTEGRATION = @EGLFS_DEVICE@

 I've changed this into a single

@EGFLS_DEVICE@

> +
>  include(../common/linux_device_post.conf)
>  load(qt_config)
> diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
> index b3f9c79..ef99131 100644
> --- a/package/qt5/qt5base/qt5base.mk
> +++ b/package/qt5/qt5base/qt5base.mk
> @@ -240,10 +240,23 @@ QT5BASE_INSTALL_LIBS_$(BR2_PACKAGE_QT5BASE_PRINTSUPPORT) += Qt5PrintSupport
>  
>  QT5BASE_INSTALL_LIBS_$(BR2_PACKAGE_QT5BASE_DBUS) += Qt5DBus
>  
> -ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST)$(BR2_PACKAGE_IMX_GPU_VIV),yy)
> -# use vivante backend
> +ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST),y)
> +ifeq ($(BR2_PACKAGE_IMX_GPU_VIV),y)
> +QT5BASE_EGLFS_DEVICE = eglfs_viv

... and added the "EGLFS_DEVICE_INTEGRATION = " part here. That way, we don't
need a separate sed expression for the case when no device integration is set,
and the whole QT5BASE_CONFIGURE_QMAKE_CONFIG can be put directly into
QT5BASE_CONFIGURE_CMDS.

> +else ifeq ($(BR2_PACKAGE_SUNXI_MALI),y)
> +QT5BASE_EGLFS_DEVICE = eglfs_mali

 This one should move to the patch adding support for sunxi-mali.

 I've committed to master with those changes, thanks.

 Could you retest with current master? That way, you can also test the
sunxi-mali support in Qt 5.10.

 Regards,
 Arnout

> +endif
> +endif
> +
> +ifeq ($(QT5BASE_EGLFS_DEVICE),)
>  define QT5BASE_CONFIGURE_QMAKE_CONFIG
> -	echo "EGLFS_DEVICE_INTEGRATION = eglfs_viv" >> \
> +	sed '/@EGLFS_DEVICE@/,+1d' $(QT5BASE_PKGDIR)/qmake.conf.in > \
> +		$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
> +endef
> +else
> +define QT5BASE_CONFIGURE_QMAKE_CONFIG
> +	sed 's/@EGLFS_DEVICE@/$(QT5BASE_EGLFS_DEVICE)/g' \
> +		$(QT5BASE_PKGDIR)/qmake.conf.in > \
>  		$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
>  endef
>  endif
> @@ -263,8 +276,7 @@ endef
>  endif
>  
>  define QT5BASE_CONFIGURE_CMDS
> -	$(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qmake.conf \
> -		$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
> +	mkdir -p $(@D)/mkspecs/devices/linux-buildroot-g++/
>  	$(QT5BASE_CONFIGURE_QMAKE_CONFIG)
>  	$(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qplatformdefs.h \
>  		$(@D)/mkspecs/devices/linux-buildroot-g++/qplatformdefs.h
> 

-- 
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] 7+ messages in thread
* [Buildroot] [PATCH v3 2/2] qt5base: add sunxi-mali-mainline eglfs backend
  2017-11-13 18:41 ` [Buildroot] [PATCH v3 2/2] qt5base: add sunxi-mali-mainline eglfs backend Giulio Benetti
@ 2018-04-04 21:47   ` Arnout Vandecappelle
  2018-04-06 19:51     ` Giulio Benetti
  0 siblings, 1 reply; 7+ messages in thread
From: Arnout Vandecappelle @ 2018-04-04 21:47 UTC (permalink / raw)
  To: buildroot



On 13-11-17 19:41, Giulio Benetti wrote:
> qt5base: add support for sunxi-mali-mainline eglfs backend
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
> ---
>  package/qt5/qt5base/qt5base.mk | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
> index ef99131..6ea42ec 100644
> --- a/package/qt5/qt5base/qt5base.mk
> +++ b/package/qt5/qt5base/qt5base.mk
> @@ -245,6 +245,8 @@ ifeq ($(BR2_PACKAGE_IMX_GPU_VIV),y)
>  QT5BASE_EGLFS_DEVICE = eglfs_viv
>  else ifeq ($(BR2_PACKAGE_SUNXI_MALI),y)
>  QT5BASE_EGLFS_DEVICE = eglfs_mali
> +else ifeq ($(BR2_PACKAGE_SUNXI_MALI_MAINLINE),y)

 I've merged the two sunxi-mali cases into a single condition and applied to
master, thanks.

 Regards,
 Arnout

> +QT5BASE_EGLFS_DEVICE = eglfs_mali
>  endif
>  endif
>  
> 

-- 
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] 7+ messages in thread
* [Buildroot] [PATCH v3 2/2] qt5base: add sunxi-mali-mainline eglfs backend
  2018-04-04 21:47   ` Arnout Vandecappelle
@ 2018-04-06 19:51     ` Giulio Benetti
  0 siblings, 0 replies; 7+ messages in thread
From: Giulio Benetti @ 2018-04-06 19:51 UTC (permalink / raw)
  To: buildroot

Hi,

Il 04/04/2018 23:47, Arnout Vandecappelle ha scritto:
> 
> 
> On 13-11-17 19:41, Giulio Benetti wrote:
>> qt5base: add support for sunxi-mali-mainline eglfs backend
>>
>> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
>> ---
>>   package/qt5/qt5base/qt5base.mk | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
>> index ef99131..6ea42ec 100644
>> --- a/package/qt5/qt5base/qt5base.mk
>> +++ b/package/qt5/qt5base/qt5base.mk
>> @@ -245,6 +245,8 @@ ifeq ($(BR2_PACKAGE_IMX_GPU_VIV),y)
>>   QT5BASE_EGLFS_DEVICE = eglfs_viv
>>   else ifeq ($(BR2_PACKAGE_SUNXI_MALI),y)
>>   QT5BASE_EGLFS_DEVICE = eglfs_mali
>> +else ifeq ($(BR2_PACKAGE_SUNXI_MALI_MAINLINE),y)
> 
>   I've merged the two sunxi-mali cases into a single condition and applied to
> master, thanks.

Good idea, thank you.

Regards,
Giulio

> 
>   Regards,
>   Arnout
> 
>> +QT5BASE_EGLFS_DEVICE = eglfs_mali
>>   endif
>>   endif
>>   
>>
> 


-- 
Giulio Benetti
CTO

MICRONOVA SRL
Sede: Via A. Niedda 3 - 35010 Vigonza (PD)
Tel. 049/8931563 - Fax 049/8931346
Cod.Fiscale - P.IVA 02663420285
Capitale Sociale ? 26.000 i.v.
Iscritta al Reg. Imprese di Padova N. 02663420285
Numero R.E.A. 258642

^ permalink raw reply	[flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in
  2018-04-04 21:45 ` [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in Arnout Vandecappelle
@ 2018-04-06 19:54   ` Giulio Benetti
  2018-04-16  9:17     ` Giulio Benetti
  0 siblings, 1 reply; 7+ messages in thread
From: Giulio Benetti @ 2018-04-06 19:54 UTC (permalink / raw)
  To: buildroot

Hi Arnout,

Il 04/04/2018 23:45, Arnout Vandecappelle ha scritto:
>   Hi Giulio,
> 
> On 13-11-17 19:41, Giulio Benetti wrote:
>> qt5base: correct eglfs support in qmake.conf.in
>>
>> Setting EGLFS_DEVICE_INTEGRATION at the end of qmake.conf like is done by commit
>> 0c219ddb8a doesn't work correctly: it has to be set before the
>> include(../common/linux_device_post.conf)
>>
>> Instead of appending to the file, change it into a qmake.conf.in template file
>> that contains an EGLFS_DEVICE_INTEGRATION assignment and update it with sed.
>>
>> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
>> ---
>> Changes v1 -> v2:
>> * write directly to package target qmake.conf
>> * simplified eglfs backend choice
>>   package/qt5/qt5base/{qmake.conf => qmake.conf.in} |  2 ++
>>   package/qt5/qt5base/qt5base.mk                    | 22 +++++++++++++++++-----
>>   2 files changed, 19 insertions(+), 5 deletions(-)
>>   rename package/qt5/qt5base/{qmake.conf => qmake.conf.in} (95%)
>>
>> diff --git a/package/qt5/qt5base/qmake.conf b/package/qt5/qt5base/qmake.conf.in
>> similarity index 95%
>> rename from package/qt5/qt5base/qmake.conf
>> rename to package/qt5/qt5base/qmake.conf.in
>> index 8b6debe..fb698b9 100644
>> --- a/package/qt5/qt5base/qmake.conf
>> +++ b/package/qt5/qt5base/qmake.conf.in
>> @@ -24,5 +24,7 @@ QMAKE_CFLAGS_ISYSTEM   =
>>   # Architecturespecific configuration
>>   include(arch.conf)
>>   
>> +EGLFS_DEVICE_INTEGRATION = @EGLFS_DEVICE@
> 
>   I've changed this into a single
> 
> @EGFLS_DEVICE@
> 
>> +
>>   include(../common/linux_device_post.conf)
>>   load(qt_config)
>> diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
>> index b3f9c79..ef99131 100644
>> --- a/package/qt5/qt5base/qt5base.mk
>> +++ b/package/qt5/qt5base/qt5base.mk
>> @@ -240,10 +240,23 @@ QT5BASE_INSTALL_LIBS_$(BR2_PACKAGE_QT5BASE_PRINTSUPPORT) += Qt5PrintSupport
>>   
>>   QT5BASE_INSTALL_LIBS_$(BR2_PACKAGE_QT5BASE_DBUS) += Qt5DBus
>>   
>> -ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST)$(BR2_PACKAGE_IMX_GPU_VIV),yy)
>> -# use vivante backend
>> +ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST),y)
>> +ifeq ($(BR2_PACKAGE_IMX_GPU_VIV),y)
>> +QT5BASE_EGLFS_DEVICE = eglfs_viv
> 
> ... and added the "EGLFS_DEVICE_INTEGRATION = " part here. That way, we don't
> need a separate sed expression for the case when no device integration is set,
> and the whole QT5BASE_CONFIGURE_QMAKE_CONFIG can be put directly into
> QT5BASE_CONFIGURE_CMDS.

Yes, it is cleaner this way, thanks!

> 
>> +else ifeq ($(BR2_PACKAGE_SUNXI_MALI),y)
>> +QT5BASE_EGLFS_DEVICE = eglfs_mali
> 
>   This one should move to the patch adding support for sunxi-mali.
> 
>   I've committed to master with those changes, thanks.
> 
>   Could you retest with current master? That way, you can also test the
> sunxi-mali support in Qt 5.10.

Sure I will. I'll be on holydays until 11 april,
I've already built everything here.
Asap I give it a try and let you know.

Regards,
Giulio

> 
>   Regards,
>   Arnout
> 
>> +endif
>> +endif
>> +
>> +ifeq ($(QT5BASE_EGLFS_DEVICE),)
>>   define QT5BASE_CONFIGURE_QMAKE_CONFIG
>> -	echo "EGLFS_DEVICE_INTEGRATION = eglfs_viv" >> \
>> +	sed '/@EGLFS_DEVICE@/,+1d' $(QT5BASE_PKGDIR)/qmake.conf.in > \
>> +		$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
>> +endef
>> +else
>> +define QT5BASE_CONFIGURE_QMAKE_CONFIG
>> +	sed 's/@EGLFS_DEVICE@/$(QT5BASE_EGLFS_DEVICE)/g' \
>> +		$(QT5BASE_PKGDIR)/qmake.conf.in > \
>>   		$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
>>   endef
>>   endif
>> @@ -263,8 +276,7 @@ endef
>>   endif
>>   
>>   define QT5BASE_CONFIGURE_CMDS
>> -	$(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qmake.conf \
>> -		$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
>> +	mkdir -p $(@D)/mkspecs/devices/linux-buildroot-g++/
>>   	$(QT5BASE_CONFIGURE_QMAKE_CONFIG)
>>   	$(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qplatformdefs.h \
>>   		$(@D)/mkspecs/devices/linux-buildroot-g++/qplatformdefs.h
>>
> 


-- 
Giulio Benetti
CTO

MICRONOVA SRL
Sede: Via A. Niedda 3 - 35010 Vigonza (PD)
Tel. 049/8931563 - Fax 049/8931346
Cod.Fiscale - P.IVA 02663420285
Capitale Sociale ? 26.000 i.v.
Iscritta al Reg. Imprese di Padova N. 02663420285
Numero R.E.A. 258642

^ permalink raw reply	[flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in
  2018-04-06 19:54   ` Giulio Benetti
@ 2018-04-16  9:17     ` Giulio Benetti
  0 siblings, 0 replies; 7+ messages in thread
From: Giulio Benetti @ 2018-04-16  9:17 UTC (permalink / raw)
  To: buildroot

Hi Arnout,

Il 06/04/2018 21:54, Giulio Benetti ha scritto:
> Hi Arnout,
> 
> Il 04/04/2018 23:45, Arnout Vandecappelle ha scritto:
>> ? Hi Giulio,
>>
>> On 13-11-17 19:41, Giulio Benetti wrote:
>>> qt5base: correct eglfs support in qmake.conf.in
>>>
>>> Setting EGLFS_DEVICE_INTEGRATION at the end of qmake.conf like is 
>>> done by commit
>>> 0c219ddb8a doesn't work correctly: it has to be set before the
>>> include(../common/linux_device_post.conf)
>>>
>>> Instead of appending to the file, change it into a qmake.conf.in 
>>> template file
>>> that contains an EGLFS_DEVICE_INTEGRATION assignment and update it 
>>> with sed.
>>>
>>> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
>>> ---
>>> Changes v1 -> v2:
>>> * write directly to package target qmake.conf
>>> * simplified eglfs backend choice
>>> ? package/qt5/qt5base/{qmake.conf => qmake.conf.in} |? 2 ++
>>> ? package/qt5/qt5base/qt5base.mk??????????????????? | 22 
>>> +++++++++++++++++-----
>>> ? 2 files changed, 19 insertions(+), 5 deletions(-)
>>> ? rename package/qt5/qt5base/{qmake.conf => qmake.conf.in} (95%)
>>>
>>> diff --git a/package/qt5/qt5base/qmake.conf 
>>> b/package/qt5/qt5base/qmake.conf.in
>>> similarity index 95%
>>> rename from package/qt5/qt5base/qmake.conf
>>> rename to package/qt5/qt5base/qmake.conf.in
>>> index 8b6debe..fb698b9 100644
>>> --- a/package/qt5/qt5base/qmake.conf
>>> +++ b/package/qt5/qt5base/qmake.conf.in
>>> @@ -24,5 +24,7 @@ QMAKE_CFLAGS_ISYSTEM?? =
>>> ? # Architecturespecific configuration
>>> ? include(arch.conf)
>>> +EGLFS_DEVICE_INTEGRATION = @EGLFS_DEVICE@
>>
>> ? I've changed this into a single
>>
>> @EGFLS_DEVICE@
>>
>>> +
>>> ? include(../common/linux_device_post.conf)
>>> ? load(qt_config)
>>> diff --git a/package/qt5/qt5base/qt5base.mk 
>>> b/package/qt5/qt5base/qt5base.mk
>>> index b3f9c79..ef99131 100644
>>> --- a/package/qt5/qt5base/qt5base.mk
>>> +++ b/package/qt5/qt5base/qt5base.mk
>>> @@ -240,10 +240,23 @@ 
>>> QT5BASE_INSTALL_LIBS_$(BR2_PACKAGE_QT5BASE_PRINTSUPPORT) += 
>>> Qt5PrintSupport
>>> ? QT5BASE_INSTALL_LIBS_$(BR2_PACKAGE_QT5BASE_DBUS) += Qt5DBus
>>> -ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST)$(BR2_PACKAGE_IMX_GPU_VIV),yy)
>>> -# use vivante backend
>>> +ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST),y)
>>> +ifeq ($(BR2_PACKAGE_IMX_GPU_VIV),y)
>>> +QT5BASE_EGLFS_DEVICE = eglfs_viv
>>
>> ... and added the "EGLFS_DEVICE_INTEGRATION = " part here. That way, 
>> we don't
>> need a separate sed expression for the case when no device integration 
>> is set,
>> and the whole QT5BASE_CONFIGURE_QMAKE_CONFIG can be put directly into
>> QT5BASE_CONFIGURE_CMDS.
> 
> Yes, it is cleaner this way, thanks!
> 
>>
>>> +else ifeq ($(BR2_PACKAGE_SUNXI_MALI),y)
>>> +QT5BASE_EGLFS_DEVICE = eglfs_mali
>>
>> ? This one should move to the patch adding support for sunxi-mali.
>>
>> ? I've committed to master with those changes, thanks.
>>
>> ? Could you retest with current master? That way, you can also test the
>> sunxi-mali support in Qt 5.10.
> 
> Sure I will. I'll be on holydays until 11 april,
> I've already built everything here.
> Asap I give it a try and let you know.
> 
> Regards,
> Giulio

Just tested on A20-LiNova1 with Qt 5.10.1 and CinematicExperience.
It's alive and kicking.
No problem with QT_QPA_EGLFS_INTEGRATION,
it works as expected.

Cheers!

-- 
Giulio Benetti
CTO

MICRONOVA SRL
Sede: Via A. Niedda 3 - 35010 Vigonza (PD)
Tel. 049/8931563 - Fax 049/8931346
Cod.Fiscale - P.IVA 02663420285
Capitale Sociale ? 26.000 i.v.
Iscritta al Reg. Imprese di Padova N. 02663420285
Numero R.E.A. 258642

> 
>>
>> ? Regards,
>> ? Arnout
>>
>>> +endif
>>> +endif
>>> +
>>> +ifeq ($(QT5BASE_EGLFS_DEVICE),)
>>> ? define QT5BASE_CONFIGURE_QMAKE_CONFIG
>>> -??? echo "EGLFS_DEVICE_INTEGRATION = eglfs_viv" >> \
>>> +??? sed '/@EGLFS_DEVICE@/,+1d' $(QT5BASE_PKGDIR)/qmake.conf.in > \
>>> +??????? $(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
>>> +endef
>>> +else
>>> +define QT5BASE_CONFIGURE_QMAKE_CONFIG
>>> +??? sed 's/@EGLFS_DEVICE@/$(QT5BASE_EGLFS_DEVICE)/g' \
>>> +??????? $(QT5BASE_PKGDIR)/qmake.conf.in > \
>>> ????????? $(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
>>> ? endef
>>> ? endif
>>> @@ -263,8 +276,7 @@ endef
>>> ? endif
>>> ? define QT5BASE_CONFIGURE_CMDS
>>> -??? $(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qmake.conf \
>>> -??????? $(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
>>> +??? mkdir -p $(@D)/mkspecs/devices/linux-buildroot-g++/
>>> ????? $(QT5BASE_CONFIGURE_QMAKE_CONFIG)
>>> ????? $(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qplatformdefs.h \
>>> ????????? $(@D)/mkspecs/devices/linux-buildroot-g++/qplatformdefs.h
>>>
>>
> 
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-04-16  9:17 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-13 18:41 [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in Giulio Benetti
2017-11-13 18:41 ` [Buildroot] [PATCH v3 2/2] qt5base: add sunxi-mali-mainline eglfs backend Giulio Benetti
2018-04-04 21:47   ` Arnout Vandecappelle
2018-04-06 19:51     ` Giulio Benetti
2018-04-04 21:45 ` [Buildroot] [PATCH v3 1/2] qt5base: correct eglfs support in qmake.conf.in Arnout Vandecappelle
2018-04-06 19:54   ` Giulio Benetti
2018-04-16  9:17     ` Giulio Benetti

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.