On 12/6/22 11:45 AM, Randolph Sapp via lists.yoctoproject.org wrote:
-PREFERRED_PROVIDER_virtual/egl ?= "ti-img-rogue-umlibs"
-PREFERRED_PROVIDER_virtual/libgles1 ?= "ti-img-rogue-umlibs"
-PREFERRED_PROVIDER_virtual/libgles2 ?= "ti-img-rogue-umlibs"
-PREFERRED_PROVIDER_virtual/libgbm ?= "ti-img-rogue-umlibs"
  PREFERRED_PROVIDER_virtual/gpudriver ?= "ti-img-rogue-driver"

Since the UM driver now RDEPENDS on the KM, could we also drop this
last line everywhere?

diff --git 
a/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_1.18.6276027.bb 
b/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_1.18.6276027.bb
new file mode 100644
index 00000000..c9110053
--- /dev/null
+++ b/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_1.18.6276027.bb
@@ -0,0 +1,55 @@
+DESCRIPTION = "Userspace libraries for PowerVR Rogue GPU on TI SoCs"
+HOMEPAGE = "http://git.ti.com/graphics/ti-img-rogue-umlibs";
+LICENSE = "TI-TFL"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=7232b98c1c58f99e3baa03de5207e76f"
+
+inherit features_check
+
+REQUIRED_MACHINE_FEATURES = "gpu"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+COMPATIBLE_MACHINE = "j721e|j721s2|j784s4|am62xx"
+
+PR = "r2"
+
+BRANCH = "linuxws/kirkstone/k5.10/${PV}"
+
+SRC_URI = 
"git://git.ti.com/git/graphics/ti-img-rogue-umlibs.git;protocol=https;branch=${BRANCH}"
+SRCREV = "540919e26f2a3674ba6a3044a113905298125894"
+
+TARGET_PRODUCT:j721e = "j721e_linux"
+TARGET_PRODUCT:j721s2 = "j721s2_linux"
+TARGET_PRODUCT:j784s4 = "j784s4_linux"
+TARGET_PRODUCT:am62xx = "am62_linux"
+PVR_BUILD ?= "release"
+PVR_WS = "lws-generic"
+
+INITSCRIPT_NAME = "rc.pvr"
+INITSCRIPT_PARAMS = "defaults 8"
+
+inherit update-rc.d
+
+DEPENDS += "libdrm wayland expat"
+RDEPENDS:${PN} += "bash mesa-megadriver ti-img-rogue-driver"

Bash? Are we sure we need that?

+RDEPENDS:${PN} += "wayland expat"

These are already in the "DEPENDS", if any of the shipped shared objects link
to them, they will be auto-added to the RDEPENDS.

+
+S = "${WORKDIR}/git"
+
+do_install () {
+    oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT} 
BUILD=${PVR_BUILD} WINDOW_SYSTEM=${PVR_WS}

Could we add these to EXTRA_OEMAKE instead of overriding do_install()?

EXTRA_OEMAKE += 'DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT} 
BUILD=${PVR_BUILD} WINDOW_SYSTEM=${PVR_WS}'


+    chown -R root:root ${D}
+}

This still needed? I'd think the default do_install() should handle file 
ownership.

+
+PACKAGES = "${PN}-tools ${PN}"
+FILES:${PN}-tools += " ${bindir}/"
+RDEPENDS:${PN}-tools += "python3-core"
+RRECOMMENDS:${PN} += "${PN}-tools"
+
+FILES:${PN} += " ${nonarch_base_libdir}/firmware/"
+FILES:${PN} += " ${datadir}/"
+FILES:${PN} += " ${libdir}/"
+FILES:${PN} += " ${etcdir}/OpenCL ${etcdir}/vulkan"
+
+INSANE_SKIP:${PN} += "ldflags arch already-stripped dev-so"

"ldflags already-stripped" should be the only skips we still need.

Andrew
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#15420): 
https://lists.yoctoproject.org/g/meta-ti/message/15420
Mute This Topic: https://lists.yoctoproject.org/mt/95498459/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to