diff mbox

[v2,2/2] mesa3d: Add support for Etnaviv gallium driver

Message ID 1486999762-4118-2-git-send-email-festevam@gmail.com
State Superseded
Headers show

Commit Message

Fabio Estevam Feb. 13, 2017, 3:29 p.m. UTC
Add support for Etnaviv gallium driver.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
Changes since v1:
- Also select BR2_PACKAGE_MESA3D_OPENGL_EGL

 package/mesa3d/Config.in | 10 ++++++++++
 package/mesa3d/mesa3d.mk |  4 ++++
 2 files changed, 14 insertions(+)

Comments

Gary Bisson Feb. 14, 2017, 1:29 p.m. UTC | #1
Fabio, All,

On Mon, Feb 13, 2017 at 4:29 PM, Fabio Estevam <festevam@gmail.com> wrote:
> Add support for Etnaviv gallium driver.
>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>

Tested-by: Gary Bisson <gary.bisson@boundarydevices.com>

On i.MX6Q Nitrogen6x with latest stable kernel (4.9.9).

Regards,
Gary
Peter Seiderer Feb. 14, 2017, 7:39 p.m. UTC | #2
Hello Fabio,

On Mon, 13 Feb 2017 13:29:22 -0200, Fabio Estevam <festevam@gmail.com> wrote:

> Add support for Etnaviv gallium driver.
> 
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
> Changes since v1:
> - Also select BR2_PACKAGE_MESA3D_OPENGL_EGL
> 
>  package/mesa3d/Config.in | 10 ++++++++++
>  package/mesa3d/mesa3d.mk |  4 ++++
>  2 files changed, 14 insertions(+)
> 
> diff --git a/package/mesa3d/Config.in b/package/mesa3d/Config.in
> index bbab953..aba17fb 100644
> --- a/package/mesa3d/Config.in
> +++ b/package/mesa3d/Config.in
> @@ -45,6 +45,16 @@ config BR2_PACKAGE_MESA3D_NEEDS_XA
>  
>  comment "Gallium drivers"
>  
> +config BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_ETNAVIV
> +	bool "Etnaviv driver"

I would prefer "Gallium Etnaviv driver" (as all the other gallium driver are named)...
 
> +	depends on BR2_arm
> +	select BR2_PACKAGE_MESA3D_GALLIUM_DRIVER
> +	select BR2_PACKAGE_LIBDRM_ETNAVIV
> +	select BR2_PACKAGE_MESA3D_OPENGL_EGL
> +	select BR2_PACKAGE_MESA3D_NEEDS_XA

Out of interest, for what is XA needed?

> +	help
> +	  Mesa driver for Vivante GPUs.
> +
>  config BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_NOUVEAU
>  	bool "Gallium nouveau driver"
>  	depends on BR2_i386 || BR2_x86_64
> diff --git a/package/mesa3d/mesa3d.mk b/package/mesa3d/mesa3d.mk
> index 3bfb005..eca1f53 100644
> --- a/package/mesa3d/mesa3d.mk
> +++ b/package/mesa3d/mesa3d.mk
> @@ -67,6 +67,8 @@ endif
>  # Drivers
>  
>  #Gallium Drivers
> +MESA3D_GALLIUM_DRIVERS-$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_ETNAVIV)  += etnaviv
> +MESA3D_GALLIUM_DRIVERS-$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_ETNAVIV)  += imx

Why not the one-liner '... += etnaviv imx'? Line length?

With the nits fixed you can add my

Reviewed-by: Peter Seiderer <ps.report@gmx.net>

Regards,
Peter


>  MESA3D_GALLIUM_DRIVERS-$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_NOUVEAU)  += nouveau
>  MESA3D_GALLIUM_DRIVERS-$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_R600)     += r600
>  MESA3D_GALLIUM_DRIVERS-$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_SVGA)     += svga
> @@ -151,6 +153,8 @@ ifeq ($(BR2_PACKAGE_MESA3D_DRI_DRIVER),y)
>  MESA3D_EGL_PLATFORMS = drm
>  else ifeq ($(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_VC4),y)
>  MESA3D_EGL_PLATFORMS = drm
> +else ifeq ($(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_ETNAVIV),y)
> +MESA3D_EGL_PLATFORMS = drm
>  else ifeq ($(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_VIRGL),y)
>  MESA3D_EGL_PLATFORMS = drm
>  endif
diff mbox

Patch

diff --git a/package/mesa3d/Config.in b/package/mesa3d/Config.in
index bbab953..aba17fb 100644
--- a/package/mesa3d/Config.in
+++ b/package/mesa3d/Config.in
@@ -45,6 +45,16 @@  config BR2_PACKAGE_MESA3D_NEEDS_XA
 
 comment "Gallium drivers"
 
+config BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_ETNAVIV
+	bool "Etnaviv driver"
+	depends on BR2_arm
+	select BR2_PACKAGE_MESA3D_GALLIUM_DRIVER
+	select BR2_PACKAGE_LIBDRM_ETNAVIV
+	select BR2_PACKAGE_MESA3D_OPENGL_EGL
+	select BR2_PACKAGE_MESA3D_NEEDS_XA
+	help
+	  Mesa driver for Vivante GPUs.
+
 config BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_NOUVEAU
 	bool "Gallium nouveau driver"
 	depends on BR2_i386 || BR2_x86_64
diff --git a/package/mesa3d/mesa3d.mk b/package/mesa3d/mesa3d.mk
index 3bfb005..eca1f53 100644
--- a/package/mesa3d/mesa3d.mk
+++ b/package/mesa3d/mesa3d.mk
@@ -67,6 +67,8 @@  endif
 # Drivers
 
 #Gallium Drivers
+MESA3D_GALLIUM_DRIVERS-$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_ETNAVIV)  += etnaviv
+MESA3D_GALLIUM_DRIVERS-$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_ETNAVIV)  += imx
 MESA3D_GALLIUM_DRIVERS-$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_NOUVEAU)  += nouveau
 MESA3D_GALLIUM_DRIVERS-$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_R600)     += r600
 MESA3D_GALLIUM_DRIVERS-$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_SVGA)     += svga
@@ -151,6 +153,8 @@  ifeq ($(BR2_PACKAGE_MESA3D_DRI_DRIVER),y)
 MESA3D_EGL_PLATFORMS = drm
 else ifeq ($(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_VC4),y)
 MESA3D_EGL_PLATFORMS = drm
+else ifeq ($(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_ETNAVIV),y)
+MESA3D_EGL_PLATFORMS = drm
 else ifeq ($(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_VIRGL),y)
 MESA3D_EGL_PLATFORMS = drm
 endif