All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v5 1/1] sunxi-mali-driver-mainline: new package
@ 2017-11-21 22:07 Giulio Benetti
  2017-11-25 15:50 ` Maxime Ripard
  2017-11-25 17:59 ` Arnout Vandecappelle
  0 siblings, 2 replies; 7+ messages in thread
From: Giulio Benetti @ 2017-11-21 22:07 UTC (permalink / raw)
  To: buildroot

Added kernel allwinner mali driver package.
Used combined with userspace Allwinner openGL blob,
it gives possibility to use 3D openGL SoC acceleration.

Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
---
Changes v4 -> v5:
* prevent build twice

 DEVELOPERS                                         |  3 +++
 package/Config.in                                  |  1 +
 package/sunxi-mali-driver-mainline/Config.in       | 22 ++++++++++++++++++
 .../sunxi-mali-driver-mainline.mk                  | 26 ++++++++++++++++++++++
 4 files changed, 52 insertions(+)
 create mode 100644 package/sunxi-mali-driver-mainline/Config.in
 create mode 100644 package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk

diff --git a/DEVELOPERS b/DEVELOPERS
index bb81a19..3fdb28b 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -695,6 +695,9 @@ F:	package/ocrad/
 F:	package/tesseract-ocr/
 F:	package/webp/
 
+N:	Giulio Benetti <giulio.benetti@micronovasrl.com>
+F:	package/sunxi-mali-driver-mainline/
+
 N:	Gregory Dymarek <gregd72002@gmail.com>
 F:	package/ding-libs/
 F:	package/gengetopt/
diff --git a/package/Config.in b/package/Config.in
index fe5ccc4..8de2227 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -496,6 +496,7 @@ endmenu
 	source "package/statserial/Config.in"
 	source "package/stm32flash/Config.in"
 	source "package/sunxi-cedarx/Config.in"
+	source "package/sunxi-mali-driver-mainline/Config.in"
 	source "package/sunxi-mali/Config.in"
 	source "package/sysstat/Config.in"
 	source "package/targetcli-fb/Config.in"
diff --git a/package/sunxi-mali-driver-mainline/Config.in b/package/sunxi-mali-driver-mainline/Config.in
new file mode 100644
index 0000000..5b7b2ba
--- /dev/null
+++ b/package/sunxi-mali-driver-mainline/Config.in
@@ -0,0 +1,22 @@
+comment "sunxi-mali-driver-mainline needs a Linux kernel to be built"
+	depends on !BR2_LINUX_KERNEL
+
+config BR2_PACKAGE_SUNXI_MALI_DRIVER_MAINLINE
+	bool "sunxi-mali-driver-mainline"
+	depends on BR2_LINUX_KERNEL
+	select BR2_LINUX_NEEDS_MODULES # not using kernel-module infra
+	help
+	  This package builds and installs the Linux kernel driver for
+	  the Mali GPU on Allwinner platforms, compatible with recent
+	  Linux kernels (>= 4.4). Note that it should be installed
+	  together with the corresponding userspace binary blobs to
+	  provide complete OpenGL functionality.
+
+	  sunxi-mali-driver-mainline requires a Linux kernel >= 4.4 with the
+	  following options enabled:
+
+	  - CONFIG_CMA
+	  - CONFIG_DMA_CMA
+	  - CONFIG_DRM_FBDEV_OVERALLOC >= 200 for double buffer or more
+
+	  https://github.com/mripard/sunxi-mali/blob/master/README.md
diff --git a/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
new file mode 100644
index 0000000..1692646
--- /dev/null
+++ b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
@@ -0,0 +1,26 @@
+################################################################################
+#
+# sunxi-mali-driver-mainline
+#
+################################################################################
+
+SUNXI_MALI_DRIVER_MAINLINE_VERSION = 5b7b3c0b6e1736676c90928816d271debbb09d95
+SUNXI_MALI_DRIVER_MAINLINE_SITE = $(call github,mripard,sunxi-mali,$(SUNXI_MALI_DRIVER_MAINLINE_VERSION))
+SUNXI_MALI_DRIVER_MAINLINE_DEPENDENCIES = linux
+
+SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS = \
+	KDIR=$(LINUX_DIR) \
+	CROSS_COMPILE=$(TARGET_CROSS) \
+	INSTALL_MOD_PATH=$(TARGET_DIR)
+
+define SUNXI_MALI_DRIVER_MAINLINE_BUILD_CMDS
+	cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
+		$(SHELL) ./build.sh -r r6p2 -a
+endef
+
+define SUNXI_MALI_DRIVER_MAINLINE_INSTALL_TARGET_CMDS
+	cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
+		$(SHELL) ./build.sh -r r6p2 -i
+endef
+
+$(eval $(generic-package))
-- 
2.7.4

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

* [Buildroot] [PATCH v5 1/1] sunxi-mali-driver-mainline: new package
  2017-11-21 22:07 [Buildroot] [PATCH v5 1/1] sunxi-mali-driver-mainline: new package Giulio Benetti
@ 2017-11-25 15:50 ` Maxime Ripard
  2017-11-25 16:42   ` Giulio Benetti
  2017-11-25 17:59 ` Arnout Vandecappelle
  1 sibling, 1 reply; 7+ messages in thread
From: Maxime Ripard @ 2017-11-25 15:50 UTC (permalink / raw)
  To: buildroot

On Tue, Nov 21, 2017 at 11:07:52PM +0100, Giulio Benetti wrote:
> Added kernel allwinner mali driver package.
> Used combined with userspace Allwinner openGL blob,
> it gives possibility to use 3D openGL SoC acceleration.
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
> ---
> Changes v4 -> v5:
> * prevent build twice
> 
>  DEVELOPERS                                         |  3 +++
>  package/Config.in                                  |  1 +
>  package/sunxi-mali-driver-mainline/Config.in       | 22 ++++++++++++++++++
>  .../sunxi-mali-driver-mainline.mk                  | 26 ++++++++++++++++++++++
>  4 files changed, 52 insertions(+)
>  create mode 100644 package/sunxi-mali-driver-mainline/Config.in
>  create mode 100644 package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
> 
> diff --git a/DEVELOPERS b/DEVELOPERS
> index bb81a19..3fdb28b 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -695,6 +695,9 @@ F:	package/ocrad/
>  F:	package/tesseract-ocr/
>  F:	package/webp/
>  
> +N:	Giulio Benetti <giulio.benetti@micronovasrl.com>
> +F:	package/sunxi-mali-driver-mainline/
> +
>  N:	Gregory Dymarek <gregd72002@gmail.com>
>  F:	package/ding-libs/
>  F:	package/gengetopt/
> diff --git a/package/Config.in b/package/Config.in
> index fe5ccc4..8de2227 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -496,6 +496,7 @@ endmenu
>  	source "package/statserial/Config.in"
>  	source "package/stm32flash/Config.in"
>  	source "package/sunxi-cedarx/Config.in"
> +	source "package/sunxi-mali-driver-mainline/Config.in"
>  	source "package/sunxi-mali/Config.in"
>  	source "package/sysstat/Config.in"
>  	source "package/targetcli-fb/Config.in"
> diff --git a/package/sunxi-mali-driver-mainline/Config.in b/package/sunxi-mali-driver-mainline/Config.in
> new file mode 100644
> index 0000000..5b7b2ba
> --- /dev/null
> +++ b/package/sunxi-mali-driver-mainline/Config.in
> @@ -0,0 +1,22 @@
> +comment "sunxi-mali-driver-mainline needs a Linux kernel to be built"
> +	depends on !BR2_LINUX_KERNEL
> +
> +config BR2_PACKAGE_SUNXI_MALI_DRIVER_MAINLINE
> +	bool "sunxi-mali-driver-mainline"
> +	depends on BR2_LINUX_KERNEL
> +	select BR2_LINUX_NEEDS_MODULES # not using kernel-module infra
> +	help
> +	  This package builds and installs the Linux kernel driver for
> +	  the Mali GPU on Allwinner platforms, compatible with recent
> +	  Linux kernels (>= 4.4). Note that it should be installed
> +	  together with the corresponding userspace binary blobs to
> +	  provide complete OpenGL functionality.
> +
> +	  sunxi-mali-driver-mainline requires a Linux kernel >= 4.4 with the
> +	  following options enabled:
> +
> +	  - CONFIG_CMA
> +	  - CONFIG_DMA_CMA
> +	  - CONFIG_DRM_FBDEV_OVERALLOC >= 200 for double buffer or more
> +
> +	  https://github.com/mripard/sunxi-mali/blob/master/README.md
> diff --git a/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
> new file mode 100644
> index 0000000..1692646
> --- /dev/null
> +++ b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
> @@ -0,0 +1,26 @@
> +################################################################################
> +#
> +# sunxi-mali-driver-mainline
> +#
> +################################################################################
> +
> +SUNXI_MALI_DRIVER_MAINLINE_VERSION = 5b7b3c0b6e1736676c90928816d271debbb09d95
> +SUNXI_MALI_DRIVER_MAINLINE_SITE = $(call github,mripard,sunxi-mali,$(SUNXI_MALI_DRIVER_MAINLINE_VERSION))
> +SUNXI_MALI_DRIVER_MAINLINE_DEPENDENCIES = linux
> +
> +SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS = \
> +	KDIR=$(LINUX_DIR) \
> +	CROSS_COMPILE=$(TARGET_CROSS) \
> +	INSTALL_MOD_PATH=$(TARGET_DIR)
> +
> +define SUNXI_MALI_DRIVER_MAINLINE_BUILD_CMDS
> +	cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
> +		$(SHELL) ./build.sh -r r6p2 -a
> +endef

This doesn't build anything, this will just apply the patches, you
should use -b.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20171125/21ffe270/attachment.asc>

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

* [Buildroot] [PATCH v5 1/1] sunxi-mali-driver-mainline: new package
  2017-11-25 15:50 ` Maxime Ripard
@ 2017-11-25 16:42   ` Giulio Benetti
  0 siblings, 0 replies; 7+ messages in thread
From: Giulio Benetti @ 2017-11-25 16:42 UTC (permalink / raw)
  To: buildroot

Hi Maxime,

Il 25/11/2017 16:50, Maxime Ripard ha scritto:
> On Tue, Nov 21, 2017 at 11:07:52PM +0100, Giulio Benetti wrote:
>> Added kernel allwinner mali driver package.
>> Used combined with userspace Allwinner openGL blob,
>> it gives possibility to use 3D openGL SoC acceleration.
>>
>> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
>> ---
>> Changes v4 -> v5:
>> * prevent build twice
>>
>>   DEVELOPERS                                         |  3 +++
>>   package/Config.in                                  |  1 +
>>   package/sunxi-mali-driver-mainline/Config.in       | 22 ++++++++++++++++++
>>   .../sunxi-mali-driver-mainline.mk                  | 26 ++++++++++++++++++++++
>>   4 files changed, 52 insertions(+)
>>   create mode 100644 package/sunxi-mali-driver-mainline/Config.in
>>   create mode 100644 package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>>
>> diff --git a/DEVELOPERS b/DEVELOPERS
>> index bb81a19..3fdb28b 100644
>> --- a/DEVELOPERS
>> +++ b/DEVELOPERS
>> @@ -695,6 +695,9 @@ F:	package/ocrad/
>>   F:	package/tesseract-ocr/
>>   F:	package/webp/
>>   
>> +N:	Giulio Benetti <giulio.benetti@micronovasrl.com>
>> +F:	package/sunxi-mali-driver-mainline/
>> +
>>   N:	Gregory Dymarek <gregd72002@gmail.com>
>>   F:	package/ding-libs/
>>   F:	package/gengetopt/
>> diff --git a/package/Config.in b/package/Config.in
>> index fe5ccc4..8de2227 100644
>> --- a/package/Config.in
>> +++ b/package/Config.in
>> @@ -496,6 +496,7 @@ endmenu
>>   	source "package/statserial/Config.in"
>>   	source "package/stm32flash/Config.in"
>>   	source "package/sunxi-cedarx/Config.in"
>> +	source "package/sunxi-mali-driver-mainline/Config.in"
>>   	source "package/sunxi-mali/Config.in"
>>   	source "package/sysstat/Config.in"
>>   	source "package/targetcli-fb/Config.in"
>> diff --git a/package/sunxi-mali-driver-mainline/Config.in b/package/sunxi-mali-driver-mainline/Config.in
>> new file mode 100644
>> index 0000000..5b7b2ba
>> --- /dev/null
>> +++ b/package/sunxi-mali-driver-mainline/Config.in
>> @@ -0,0 +1,22 @@
>> +comment "sunxi-mali-driver-mainline needs a Linux kernel to be built"
>> +	depends on !BR2_LINUX_KERNEL
>> +
>> +config BR2_PACKAGE_SUNXI_MALI_DRIVER_MAINLINE
>> +	bool "sunxi-mali-driver-mainline"
>> +	depends on BR2_LINUX_KERNEL
>> +	select BR2_LINUX_NEEDS_MODULES # not using kernel-module infra
>> +	help
>> +	  This package builds and installs the Linux kernel driver for
>> +	  the Mali GPU on Allwinner platforms, compatible with recent
>> +	  Linux kernels (>= 4.4). Note that it should be installed
>> +	  together with the corresponding userspace binary blobs to
>> +	  provide complete OpenGL functionality.
>> +
>> +	  sunxi-mali-driver-mainline requires a Linux kernel >= 4.4 with the
>> +	  following options enabled:
>> +
>> +	  - CONFIG_CMA
>> +	  - CONFIG_DMA_CMA
>> +	  - CONFIG_DRM_FBDEV_OVERALLOC >= 200 for double buffer or more
>> +
>> +	  https://github.com/mripard/sunxi-mali/blob/master/README.md
>> diff --git a/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>> new file mode 100644
>> index 0000000..1692646
>> --- /dev/null
>> +++ b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>> @@ -0,0 +1,26 @@
>> +################################################################################
>> +#
>> +# sunxi-mali-driver-mainline
>> +#
>> +################################################################################
>> +
>> +SUNXI_MALI_DRIVER_MAINLINE_VERSION = 5b7b3c0b6e1736676c90928816d271debbb09d95
>> +SUNXI_MALI_DRIVER_MAINLINE_SITE = $(call github,mripard,sunxi-mali,$(SUNXI_MALI_DRIVER_MAINLINE_VERSION))
>> +SUNXI_MALI_DRIVER_MAINLINE_DEPENDENCIES = linux
>> +
>> +SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS = \
>> +	KDIR=$(LINUX_DIR) \
>> +	CROSS_COMPILE=$(TARGET_CROSS) \
>> +	INSTALL_MOD_PATH=$(TARGET_DIR)
>> +
>> +define SUNXI_MALI_DRIVER_MAINLINE_BUILD_CMDS
>> +	cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
>> +		$(SHELL) ./build.sh -r r6p2 -a
>> +endef
> 
> This doesn't build anything, this will just apply the patches, you
> should use -b.

You're right, I've tried it as a workaround to avoid compiling twice.
But it's not here the problem.
I've just sent you a PR on github:
https://github.com/mripard/sunxi-mali/pull/6
This will fix the problem.

On "[PATCH v4 1/1] sunxi-mali-driver-mainline: new package" I did it right:
+define SUNXI_MALI_DRIVER_MAINLINE_BUILD_CMDS
+	cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
+	$(SHELL) ./build.sh -r r6p2 -b
+endef
+
+define SUNXI_MALI_DRIVER_MAINLINE_INSTALL_TARGET_CMDS
+	cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
+	$(SHELL) ./build.sh -r r6p2 -i
+endef

So v5 patch doesn't make sense if you accept my Pull Request on Github.
How can I proceed?
Do I submit v6 equal to v4 with changes respect v5?

Thank you
Regards

> 
> Maxime
> 


-- 
Giulio Benetti
R&D Manager &
Advanced Research

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 v5 1/1] sunxi-mali-driver-mainline: new package
  2017-11-21 22:07 [Buildroot] [PATCH v5 1/1] sunxi-mali-driver-mainline: new package Giulio Benetti
  2017-11-25 15:50 ` Maxime Ripard
@ 2017-11-25 17:59 ` Arnout Vandecappelle
  2017-11-25 20:33   ` Giulio Benetti
  1 sibling, 1 reply; 7+ messages in thread
From: Arnout Vandecappelle @ 2017-11-25 17:59 UTC (permalink / raw)
  To: buildroot

 Hi Giulio,

On 21-11-17 23:07, Giulio Benetti wrote:
> Added kernel allwinner mali driver package.
> Used combined with userspace Allwinner openGL blob,
> it gives possibility to use 3D openGL SoC acceleration.

 I should have asked before, but: where do those binaries come from? Are they
from the mali-sunxi package? Or not yet packaged in Buildroot?


[snip]
> index fe5ccc4..8de2227 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -496,6 +496,7 @@ endmenu
>  	source "package/statserial/Config.in"
>  	source "package/stm32flash/Config.in"
>  	source "package/sunxi-cedarx/Config.in"
> +	source "package/sunxi-mali-driver-mainline/Config.in"
>  	source "package/sunxi-mali/Config.in"

 foo-bar sorts after foo, so swap these two lines.

>  	source "package/sysstat/Config.in"
>  	source "package/targetcli-fb/Config.in"
> diff --git a/package/sunxi-mali-driver-mainline/Config.in b/package/sunxi-mali-driver-mainline/Config.in
> new file mode 100644
> index 0000000..5b7b2ba
> --- /dev/null
> +++ b/package/sunxi-mali-driver-mainline/Config.in
> @@ -0,0 +1,22 @@
> +comment "sunxi-mali-driver-mainline needs a Linux kernel to be built"
> +	depends on !BR2_LINUX_KERNEL
> +
> +config BR2_PACKAGE_SUNXI_MALI_DRIVER_MAINLINE
> +	bool "sunxi-mali-driver-mainline"
> +	depends on BR2_LINUX_KERNEL
> +	select BR2_LINUX_NEEDS_MODULES # not using kernel-module infra
> +	help
> +	  This package builds and installs the Linux kernel driver for
> +	  the Mali GPU on Allwinner platforms, compatible with recent
> +	  Linux kernels (>= 4.4). Note that it should be installed
> +	  together with the corresponding userspace binary blobs to

 I guess with 'binary blobs' you mean the OpenGL libraries, which are only
provided as binaries? 'binary blobs' to me sounds like something like firmware
blobs: is put in your rootfs but doesn't run on the target processor. So I'd
rephrase as: ... together with the corresponding userspace OpenGL libraries,
which are provided as binaries only.

 And preferably also explain where you can get those.

> +	  provide complete OpenGL functionality.
> +
> +	  sunxi-mali-driver-mainline requires a Linux kernel >= 4.4 with the
> +	  following options enabled:
> +
> +	  - CONFIG_CMA
> +	  - CONFIG_DMA_CMA
> +	  - CONFIG_DRM_FBDEV_OVERALLOC >= 200 for double buffer or more
> +
> +	  https://github.com/mripard/sunxi-mali/blob/master/README.md
> diff --git a/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
> new file mode 100644
> index 0000000..1692646
> --- /dev/null
> +++ b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
> @@ -0,0 +1,26 @@
> +################################################################################
> +#
> +# sunxi-mali-driver-mainline
> +#
> +################################################################################
> +
> +SUNXI_MALI_DRIVER_MAINLINE_VERSION = 5b7b3c0b6e1736676c90928816d271debbb09d95
> +SUNXI_MALI_DRIVER_MAINLINE_SITE = $(call github,mripard,sunxi-mali,$(SUNXI_MALI_DRIVER_MAINLINE_VERSION))
> +SUNXI_MALI_DRIVER_MAINLINE_DEPENDENCIES = linux
> +
> +SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS = \
> +	KDIR=$(LINUX_DIR) \
> +	CROSS_COMPILE=$(TARGET_CROSS) \
> +	INSTALL_MOD_PATH=$(TARGET_DIR)
> +
> +define SUNXI_MALI_DRIVER_MAINLINE_BUILD_CMDS
> +	cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
> +		$(SHELL) ./build.sh -r r6p2 -a

 Shouldn't the r6p2 be configurable? That can be done in a later patch, though.

 Regards,
 Arnout

> +endef
> +
> +define SUNXI_MALI_DRIVER_MAINLINE_INSTALL_TARGET_CMDS
> +	cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
> +		$(SHELL) ./build.sh -r r6p2 -i
> +endef
> +
> +$(eval $(generic-package))
> 

-- 
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 v5 1/1] sunxi-mali-driver-mainline: new package
  2017-11-25 17:59 ` Arnout Vandecappelle
@ 2017-11-25 20:33   ` Giulio Benetti
  2017-11-25 21:45     ` Giulio Benetti
  0 siblings, 1 reply; 7+ messages in thread
From: Giulio Benetti @ 2017-11-25 20:33 UTC (permalink / raw)
  To: buildroot

Hi Arnout,

Il 25/11/2017 18:59, Arnout Vandecappelle ha scritto:
>   Hi Giulio,
> 
> On 21-11-17 23:07, Giulio Benetti wrote:
>> Added kernel allwinner mali driver package.
>> Used combined with userspace Allwinner openGL blob,
>> it gives possibility to use 3D openGL SoC acceleration.
> 
>   I should have asked before, but: where do those binaries come from? Are they
> from the mali-sunxi package? Or not yet packaged in Buildroot?

It will be combined with sunxi-mali-mainline,
I've submitted a patch still pending:
[PATCH v4 1/1] sunxi-mali-mainline: new package
http://buildroot-busybox.2317881.n4.nabble.com/PATCH-v4-1-1-sunxi-mali-mainline-new-package-td180182.html

> 
> 
> [snip]
>> index fe5ccc4..8de2227 100644
>> --- a/package/Config.in
>> +++ b/package/Config.in
>> @@ -496,6 +496,7 @@ endmenu
>>   	source "package/statserial/Config.in"
>>   	source "package/stm32flash/Config.in"
>>   	source "package/sunxi-cedarx/Config.in"
>> +	source "package/sunxi-mali-driver-mainline/Config.in"
>>   	source "package/sunxi-mali/Config.in"
> 
>   foo-bar sorts after foo, so swap these two lines.

Ok, on next patch I will correct it

> 
>>   	source "package/sysstat/Config.in"
>>   	source "package/targetcli-fb/Config.in"
>> diff --git a/package/sunxi-mali-driver-mainline/Config.in b/package/sunxi-mali-driver-mainline/Config.in
>> new file mode 100644
>> index 0000000..5b7b2ba
>> --- /dev/null
>> +++ b/package/sunxi-mali-driver-mainline/Config.in
>> @@ -0,0 +1,22 @@
>> +comment "sunxi-mali-driver-mainline needs a Linux kernel to be built"
>> +	depends on !BR2_LINUX_KERNEL
>> +
>> +config BR2_PACKAGE_SUNXI_MALI_DRIVER_MAINLINE
>> +	bool "sunxi-mali-driver-mainline"
>> +	depends on BR2_LINUX_KERNEL
>> +	select BR2_LINUX_NEEDS_MODULES # not using kernel-module infra
>> +	help
>> +	  This package builds and installs the Linux kernel driver for
>> +	  the Mali GPU on Allwinner platforms, compatible with recent
>> +	  Linux kernels (>= 4.4). Note that it should be installed
>> +	  together with the corresponding userspace binary blobs to
> 
>   I guess with 'binary blobs' you mean the OpenGL libraries, which are only
> provided as binaries? 'binary blobs' to me sounds like something like firmware
> blobs: is put in your rootfs but doesn't run on the target processor. So I'd
> rephrase as: ... together with the corresponding userspace OpenGL libraries,
> which are provided as binaries only.

Ok

> 
>   And preferably also explain where you can get those.

Ok, I will add github sunxi-mali by mripard

> 
>> +	  provide complete OpenGL functionality.
>> +
>> +	  sunxi-mali-driver-mainline requires a Linux kernel >= 4.4 with the
>> +	  following options enabled:
>> +
>> +	  - CONFIG_CMA
>> +	  - CONFIG_DMA_CMA
>> +	  - CONFIG_DRM_FBDEV_OVERALLOC >= 200 for double buffer or more
>> +
>> +	  https://github.com/mripard/sunxi-mali/blob/master/README.md
>> diff --git a/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>> new file mode 100644
>> index 0000000..1692646
>> --- /dev/null
>> +++ b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>> @@ -0,0 +1,26 @@
>> +################################################################################
>> +#
>> +# sunxi-mali-driver-mainline
>> +#
>> +################################################################################
>> +
>> +SUNXI_MALI_DRIVER_MAINLINE_VERSION = 5b7b3c0b6e1736676c90928816d271debbb09d95
>> +SUNXI_MALI_DRIVER_MAINLINE_SITE = $(call github,mripard,sunxi-mali,$(SUNXI_MALI_DRIVER_MAINLINE_VERSION))
>> +SUNXI_MALI_DRIVER_MAINLINE_DEPENDENCIES = linux
>> +
>> +SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS = \
>> +	KDIR=$(LINUX_DIR) \
>> +	CROSS_COMPILE=$(TARGET_CROSS) \
>> +	INSTALL_MOD_PATH=$(TARGET_DIR)
>> +
>> +define SUNXI_MALI_DRIVER_MAINLINE_BUILD_CMDS
>> +	cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
>> +		$(SHELL) ./build.sh -r r6p2 -a
> 
>   Shouldn't the r6p2 be configurable? That can be done in a later patch, though.

You're right, but at this time there's only r6p2 available.
Anyway I can add it parametric, so it will be ready for any next versions.

> 
>   Regards,
>   Arnout
> 
>> +endef
>> +
>> +define SUNXI_MALI_DRIVER_MAINLINE_INSTALL_TARGET_CMDS
>> +	cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
>> +		$(SHELL) ./build.sh -r r6p2 -i
>> +endef
>> +
>> +$(eval $(generic-package))
>>
> 


-- 
Giulio Benetti
R&D Manager &
Advanced Research

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 v5 1/1] sunxi-mali-driver-mainline: new package
  2017-11-25 20:33   ` Giulio Benetti
@ 2017-11-25 21:45     ` Giulio Benetti
  2017-11-25 22:25       ` Giulio Benetti
  0 siblings, 1 reply; 7+ messages in thread
From: Giulio Benetti @ 2017-11-25 21:45 UTC (permalink / raw)
  To: buildroot

Hello Everybody,

I was thinking about doing a unique patchset for:
sunxi-mali-driver-mainline
and
sunxi-mali-mainline

They're very connected each other.
What do you think?

Il 25/11/2017 21:33, Giulio Benetti ha scritto:
> Hi Arnout,
> 
> Il 25/11/2017 18:59, Arnout Vandecappelle ha scritto:
>> ? Hi Giulio,
>>
>> On 21-11-17 23:07, Giulio Benetti wrote:
>>> Added kernel allwinner mali driver package.
>>> Used combined with userspace Allwinner openGL blob,
>>> it gives possibility to use 3D openGL SoC acceleration.
>>
>> ? I should have asked before, but: where do those binaries come from? 
>> Are they
>> from the mali-sunxi package? Or not yet packaged in Buildroot?
> 
> It will be combined with sunxi-mali-mainline,
> I've submitted a patch still pending:
> [PATCH v4 1/1] sunxi-mali-mainline: new package
> http://buildroot-busybox.2317881.n4.nabble.com/PATCH-v4-1-1-sunxi-mali-mainline-new-package-td180182.html 
> 
> 
>>
>>
>> [snip]
>>> index fe5ccc4..8de2227 100644
>>> --- a/package/Config.in
>>> +++ b/package/Config.in
>>> @@ -496,6 +496,7 @@ endmenu
>>> ????? source "package/statserial/Config.in"
>>> ????? source "package/stm32flash/Config.in"
>>> ????? source "package/sunxi-cedarx/Config.in"
>>> +??? source "package/sunxi-mali-driver-mainline/Config.in"
>>> ????? source "package/sunxi-mali/Config.in"
>>
>> ? foo-bar sorts after foo, so swap these two lines.
> 
> Ok, on next patch I will correct it
> 
>>
>>> ????? source "package/sysstat/Config.in"
>>> ????? source "package/targetcli-fb/Config.in"
>>> diff --git a/package/sunxi-mali-driver-mainline/Config.in 
>>> b/package/sunxi-mali-driver-mainline/Config.in
>>> new file mode 100644
>>> index 0000000..5b7b2ba
>>> --- /dev/null
>>> +++ b/package/sunxi-mali-driver-mainline/Config.in
>>> @@ -0,0 +1,22 @@
>>> +comment "sunxi-mali-driver-mainline needs a Linux kernel to be built"
>>> +??? depends on !BR2_LINUX_KERNEL
>>> +
>>> +config BR2_PACKAGE_SUNXI_MALI_DRIVER_MAINLINE
>>> +??? bool "sunxi-mali-driver-mainline"
>>> +??? depends on BR2_LINUX_KERNEL
>>> +??? select BR2_LINUX_NEEDS_MODULES # not using kernel-module infra
>>> +??? help
>>> +????? This package builds and installs the Linux kernel driver for
>>> +????? the Mali GPU on Allwinner platforms, compatible with recent
>>> +????? Linux kernels (>= 4.4). Note that it should be installed
>>> +????? together with the corresponding userspace binary blobs to
>>
>> ? I guess with 'binary blobs' you mean the OpenGL libraries, which are 
>> only
>> provided as binaries? 'binary blobs' to me sounds like something like 
>> firmware
>> blobs: is put in your rootfs but doesn't run on the target processor. 
>> So I'd
>> rephrase as: ... together with the corresponding userspace OpenGL 
>> libraries,
>> which are provided as binaries only.
> 
> Ok
> 
>>
>> ? And preferably also explain where you can get those.
> 
> Ok, I will add github sunxi-mali by mripard
> 
>>
>>> +????? provide complete OpenGL functionality.
>>> +
>>> +????? sunxi-mali-driver-mainline requires a Linux kernel >= 4.4 with 
>>> the
>>> +????? following options enabled:
>>> +
>>> +????? - CONFIG_CMA
>>> +????? - CONFIG_DMA_CMA
>>> +????? - CONFIG_DRM_FBDEV_OVERALLOC >= 200 for double buffer or more
>>> +
>>> +????? https://github.com/mripard/sunxi-mali/blob/master/README.md
>>> diff --git 
>>> a/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk 
>>> b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>>> new file mode 100644
>>> index 0000000..1692646
>>> --- /dev/null
>>> +++ b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>>> @@ -0,0 +1,26 @@
>>> +################################################################################ 
>>>
>>> +#
>>> +# sunxi-mali-driver-mainline
>>> +#
>>> +################################################################################ 
>>>
>>> +
>>> +SUNXI_MALI_DRIVER_MAINLINE_VERSION = 
>>> 5b7b3c0b6e1736676c90928816d271debbb09d95
>>> +SUNXI_MALI_DRIVER_MAINLINE_SITE = $(call 
>>> github,mripard,sunxi-mali,$(SUNXI_MALI_DRIVER_MAINLINE_VERSION))
>>> +SUNXI_MALI_DRIVER_MAINLINE_DEPENDENCIES = linux
>>> +
>>> +SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS = \
>>> +??? KDIR=$(LINUX_DIR) \
>>> +??? CROSS_COMPILE=$(TARGET_CROSS) \
>>> +??? INSTALL_MOD_PATH=$(TARGET_DIR)
>>> +
>>> +define SUNXI_MALI_DRIVER_MAINLINE_BUILD_CMDS
>>> +??? cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
>>> +??????? $(SHELL) ./build.sh -r r6p2 -a
>>
>> ? Shouldn't the r6p2 be configurable? That can be done in a later 
>> patch, though.
> 
> You're right, but at this time there's only r6p2 available.
> Anyway I can add it parametric, so it will be ready for any next versions.
> 
>>
>> ? Regards,
>> ? Arnout
>>
>>> +endef
>>> +
>>> +define SUNXI_MALI_DRIVER_MAINLINE_INSTALL_TARGET_CMDS
>>> +??? cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
>>> +??????? $(SHELL) ./build.sh -r r6p2 -i
>>> +endef
>>> +
>>> +$(eval $(generic-package))
>>>
>>
> 
> 


-- 
Giulio Benetti
R&D Manager &
Advanced Research

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 v5 1/1] sunxi-mali-driver-mainline: new package
  2017-11-25 21:45     ` Giulio Benetti
@ 2017-11-25 22:25       ` Giulio Benetti
  0 siblings, 0 replies; 7+ messages in thread
From: Giulio Benetti @ 2017-11-25 22:25 UTC (permalink / raw)
  To: buildroot

I've sent a patchset as described below.
Those packaged are bound together.
In this way there's no problem on applying patches.
Hope I did it well.
I was not able to keep version for each patch,
but as I know it's not possible,
because version refer to other patches.

Let me know if I have to change something.

Kind regards

Il 25/11/2017 22:45, Giulio Benetti ha scritto:
> Hello Everybody,
> 
> I was thinking about doing a unique patchset for:
> sunxi-mali-driver-mainline
> and
> sunxi-mali-mainline
> 
> They're very connected each other.
> What do you think?
> 
> Il 25/11/2017 21:33, Giulio Benetti ha scritto:
>> Hi Arnout,
>>
>> Il 25/11/2017 18:59, Arnout Vandecappelle ha scritto:
>>> ? Hi Giulio,
>>>
>>> On 21-11-17 23:07, Giulio Benetti wrote:
>>>> Added kernel allwinner mali driver package.
>>>> Used combined with userspace Allwinner openGL blob,
>>>> it gives possibility to use 3D openGL SoC acceleration.
>>>
>>> ? I should have asked before, but: where do those binaries come from? 
>>> Are they
>>> from the mali-sunxi package? Or not yet packaged in Buildroot?
>>
>> It will be combined with sunxi-mali-mainline,
>> I've submitted a patch still pending:
>> [PATCH v4 1/1] sunxi-mali-mainline: new package
>> http://buildroot-busybox.2317881.n4.nabble.com/PATCH-v4-1-1-sunxi-mali-mainline-new-package-td180182.html 
>>
>>
>>>
>>>
>>> [snip]
>>>> index fe5ccc4..8de2227 100644
>>>> --- a/package/Config.in
>>>> +++ b/package/Config.in
>>>> @@ -496,6 +496,7 @@ endmenu
>>>> ????? source "package/statserial/Config.in"
>>>> ????? source "package/stm32flash/Config.in"
>>>> ????? source "package/sunxi-cedarx/Config.in"
>>>> +??? source "package/sunxi-mali-driver-mainline/Config.in"
>>>> ????? source "package/sunxi-mali/Config.in"
>>>
>>> ? foo-bar sorts after foo, so swap these two lines.
>>
>> Ok, on next patch I will correct it
>>
>>>
>>>> ????? source "package/sysstat/Config.in"
>>>> ????? source "package/targetcli-fb/Config.in"
>>>> diff --git a/package/sunxi-mali-driver-mainline/Config.in 
>>>> b/package/sunxi-mali-driver-mainline/Config.in
>>>> new file mode 100644
>>>> index 0000000..5b7b2ba
>>>> --- /dev/null
>>>> +++ b/package/sunxi-mali-driver-mainline/Config.in
>>>> @@ -0,0 +1,22 @@
>>>> +comment "sunxi-mali-driver-mainline needs a Linux kernel to be built"
>>>> +??? depends on !BR2_LINUX_KERNEL
>>>> +
>>>> +config BR2_PACKAGE_SUNXI_MALI_DRIVER_MAINLINE
>>>> +??? bool "sunxi-mali-driver-mainline"
>>>> +??? depends on BR2_LINUX_KERNEL
>>>> +??? select BR2_LINUX_NEEDS_MODULES # not using kernel-module infra
>>>> +??? help
>>>> +????? This package builds and installs the Linux kernel driver for
>>>> +????? the Mali GPU on Allwinner platforms, compatible with recent
>>>> +????? Linux kernels (>= 4.4). Note that it should be installed
>>>> +????? together with the corresponding userspace binary blobs to
>>>
>>> ? I guess with 'binary blobs' you mean the OpenGL libraries, which 
>>> are only
>>> provided as binaries? 'binary blobs' to me sounds like something like 
>>> firmware
>>> blobs: is put in your rootfs but doesn't run on the target processor. 
>>> So I'd
>>> rephrase as: ... together with the corresponding userspace OpenGL 
>>> libraries,
>>> which are provided as binaries only.
>>
>> Ok
>>
>>>
>>> ? And preferably also explain where you can get those.
>>
>> Ok, I will add github sunxi-mali by mripard
>>
>>>
>>>> +????? provide complete OpenGL functionality.
>>>> +
>>>> +????? sunxi-mali-driver-mainline requires a Linux kernel >= 4.4 
>>>> with the
>>>> +????? following options enabled:
>>>> +
>>>> +????? - CONFIG_CMA
>>>> +????? - CONFIG_DMA_CMA
>>>> +????? - CONFIG_DRM_FBDEV_OVERALLOC >= 200 for double buffer or more
>>>> +
>>>> +????? https://github.com/mripard/sunxi-mali/blob/master/README.md
>>>> diff --git 
>>>> a/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk 
>>>> b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>>>> new file mode 100644
>>>> index 0000000..1692646
>>>> --- /dev/null
>>>> +++ b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>>>> @@ -0,0 +1,26 @@
>>>> +################################################################################ 
>>>>
>>>> +#
>>>> +# sunxi-mali-driver-mainline
>>>> +#
>>>> +################################################################################ 
>>>>
>>>> +
>>>> +SUNXI_MALI_DRIVER_MAINLINE_VERSION = 
>>>> 5b7b3c0b6e1736676c90928816d271debbb09d95
>>>> +SUNXI_MALI_DRIVER_MAINLINE_SITE = $(call 
>>>> github,mripard,sunxi-mali,$(SUNXI_MALI_DRIVER_MAINLINE_VERSION))
>>>> +SUNXI_MALI_DRIVER_MAINLINE_DEPENDENCIES = linux
>>>> +
>>>> +SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS = \
>>>> +??? KDIR=$(LINUX_DIR) \
>>>> +??? CROSS_COMPILE=$(TARGET_CROSS) \
>>>> +??? INSTALL_MOD_PATH=$(TARGET_DIR)
>>>> +
>>>> +define SUNXI_MALI_DRIVER_MAINLINE_BUILD_CMDS
>>>> +??? cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
>>>> +??????? $(SHELL) ./build.sh -r r6p2 -a
>>>
>>> ? Shouldn't the r6p2 be configurable? That can be done in a later 
>>> patch, though.
>>
>> You're right, but at this time there's only r6p2 available.
>> Anyway I can add it parametric, so it will be ready for any next 
>> versions.
>>
>>>
>>> ? Regards,
>>> ? Arnout
>>>
>>>> +endef
>>>> +
>>>> +define SUNXI_MALI_DRIVER_MAINLINE_INSTALL_TARGET_CMDS
>>>> +??? cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
>>>> +??????? $(SHELL) ./build.sh -r r6p2 -i
>>>> +endef
>>>> +
>>>> +$(eval $(generic-package))
>>>>
>>>
>>
>>
> 
> 


-- 
Giulio Benetti
R&D Manager &
Advanced Research

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

end of thread, other threads:[~2017-11-25 22:25 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-21 22:07 [Buildroot] [PATCH v5 1/1] sunxi-mali-driver-mainline: new package Giulio Benetti
2017-11-25 15:50 ` Maxime Ripard
2017-11-25 16:42   ` Giulio Benetti
2017-11-25 17:59 ` Arnout Vandecappelle
2017-11-25 20:33   ` Giulio Benetti
2017-11-25 21:45     ` Giulio Benetti
2017-11-25 22:25       ` 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.