From: Chris Dimich <Chris.Dimich@boundarydevices.com>
Date: Tue, 18 May 2021 18:32:16 +0000 (-0700)
Subject: weston-init: uncomment use-g2d=1 based on imxgpu2d MACHINEOVERRIDE
X-Git-Url: https://code.ossystems.io/gitweb?a=commitdiff_plain;h=4a61fdbd50e6200239dcb0b936bb37acaab2b9e0;p=meta-freescale.git

weston-init: uncomment use-g2d=1 based on imxgpu2d MACHINEOVERRIDE

affects the following SoCs:
-mx6q
-mx6dl
-mx6sx
-mx6sl
-mx7ulp
-mx8qm
-mx8mm
-mx8mp
-mx8qxp

Signed-off-by: Chris Dimich <Chris.Dimich@boundarydevices.com>
---

diff --git a/recipes-graphics/wayland/weston-init.bbappend b/recipes-graphics/wayland/weston-init.bbappend
index 66e148c1..ebd98268 100644
--- a/recipes-graphics/wayland/weston-init.bbappend
+++ b/recipes-graphics/wayland/weston-init.bbappend
@@ -15,30 +15,25 @@ SRC_URI_append_mx6sl = " file://weston.config"
 INI_UNCOMMENT_ASSIGNMENTS_append_imx = " \
     ${@bb.utils.contains('DISTRO_FEATURES', 'x11 wayland', 'xwayland=true', '', d)} \
 "
-INI_UNCOMMENT_ASSIGNMENTS_append_mx6dl = " \
-    use-g2d=1 \
-"
-INI_UNCOMMENT_ASSIGNMENTS_append_mx6q = " \
-    use-g2d=1 \
-"
-INI_UNCOMMENT_ASSIGNMENTS_append_mx7ulp = " \
-    use-g2d=1 \
-"
 INI_UNCOMMENT_ASSIGNMENTS_append_mx8 = " \
     repaint-window=16 \
 "
-INI_UNCOMMENT_ASSIGNMENTS_append_mx8mm = " \
-    use-g2d=1 \
-"
-INI_UNCOMMENT_ASSIGNMENTS_append_mx8mp = " \
-    use-g2d=1 \
-"
 INI_UNCOMMENT_ASSIGNMENTS_append_mx8mq = " \
     gbm-format=argb8888 \
     \\[shell\\] \
     size=1920x1080 \
 "
 
+# FIXME: The 8QM and 8QXP SoCs have better performance without G2D so don't enable it
+# Ideally, this should be seamless and Vivante ought to handle it internally and take the fastest
+# rendering code.
+INI_UNCOMMENT_USE_G2D_imxgpu2d ?= "use-g2d=1"
+INI_UNCOMMENT_USE_G2D_mx8qm = ""
+INI_UNCOMMENT_USE_G2D_mx8qxp = ""
+INI_UNCOMMENT_ASSIGNMENTS_append_imxgpu2d = " \
+    ${INI_UNCOMMENT_USE_G2D} \
+"
+
 uncomment() {
     if ! grep -q "^#$1" $2 && ! grep -q "^$1" $2; then
         bbwarn "Commented setting '#$1' not found in file $2"