Re: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2

[Denys Dmytriyenko <denis@denix.org>]

On Mon, Nov 17, 2025 at 05:22:51PM -0600, Andrew Davis via lists.yoctoproject.org wrote:
> On 11/17/25 3:15 PM, Antonios Christidis via lists.yoctoproject.org wrote:
> >From: Antonios Christidis <a-christidis@ti.com>
> >
> >New recipes for GPU DDK 25.2. Update BSP configuration files with new
> >preferred recipe versions.
> >
> >For the driver recipe, specifically unset the CC variable due to yocto
> >appending CROSS_COMPILE and CC, resulting in a warning for the
> >do_compile task:
> >
> >make[1]: aarch64-oe-linux-aarch64-oe-linux-gcc: No such file or directory
> >
> >Signed-off-by: Antonios Christidis <a-christidis@ti.com>
> >---
> >  meta-beagle/conf/machine/include/beagle-bsp.inc        |  4 ++--
> >  meta-ti-bsp/conf/machine/include/ti-bsp.inc            |  4 ++--
> >  ....6643903.bb => ti-img-rogue-driver_25.2.6850647.bb} | 10 ++++++++--
> >  ....6643903.bb => ti-img-rogue-umlibs_25.2.6850647.bb} |  4 +---
> >  4 files changed, 13 insertions(+), 9 deletions(-)
> >  rename meta-ti-bsp/recipes-bsp/powervr-drivers/{ti-img-rogue-driver_24.2.6643903.bb => ti-img-rogue-driver_25.2.6850647.bb} (83%)
> >  rename meta-ti-bsp/recipes-graphics/powervr-umlibs/{ti-img-rogue-umlibs_24.2.6643903.bb => ti-img-rogue-umlibs_25.2.6850647.bb} (98%)
> >
> >diff --git a/meta-beagle/conf/machine/include/beagle-bsp.inc b/meta-beagle/conf/machine/include/beagle-bsp.inc
> >index 448e2086..5060e286 100644
> >--- a/meta-beagle/conf/machine/include/beagle-bsp.inc
> >+++ b/meta-beagle/conf/machine/include/beagle-bsp.inc
> >@@ -19,7 +19,7 @@ KERNEL_DEVICETREE_PREFIX:bsp-bb_org-6_12 = ""
> >  BSP_SGX_DRIVER_PROVIDER:bsp-bb_org-6_12 = "ti-sgx-ddk-km"
> >  BSP_SGX_DRIVER_VERSION:bsp-bb_org-6_12 = "1.17%"
> >  BSP_ROGUE_DRIVER_PROVIDER:bsp-bb_org-6_12 = "ti-img-rogue-driver"
> >-BSP_ROGUE_DRIVER_VERSION:bsp-bb_org-6_12 = "24%"
> >+BSP_ROGUE_DRIVER_VERSION:bsp-bb_org-6_12 = "25%"
> >  BSP_MESA_PVR_VERSION:bsp-bb_org-6_12 = "24%"
> >  # GC320 support requires out-of-tree drivers not yet available in bb.org 6.12
> >@@ -40,7 +40,7 @@ KERNEL_DEVICETREE_PREFIX:bsp-bb_org-6_6 = ""
> >  BSP_SGX_DRIVER_PROVIDER:bsp-bb_org-6_6 = "ti-sgx-ddk-km"
> >  BSP_SGX_DRIVER_VERSION:bsp-bb_org-6_6 = "1.17%"
> >  BSP_ROGUE_DRIVER_PROVIDER:bsp-bb_org-6_6 = "ti-img-rogue-driver"
> >-BSP_ROGUE_DRIVER_VERSION:bsp-bb_org-6_6 = "24%"
> >+BSP_ROGUE_DRIVER_VERSION:bsp-bb_org-6_6 = "25%"
> >  BSP_MESA_PVR_VERSION:bsp-bb_org-6_6 = "24%"
> >  # GC320 support requires out-of-tree drivers not yet available in bb.org 6.6
> >diff --git a/meta-ti-bsp/conf/machine/include/ti-bsp.inc b/meta-ti-bsp/conf/machine/include/ti-bsp.inc
> >index 0ea7bd7b..eaa501c3 100644
> >--- a/meta-ti-bsp/conf/machine/include/ti-bsp.inc
> >+++ b/meta-ti-bsp/conf/machine/include/ti-bsp.inc
> >@@ -45,7 +45,7 @@ BSP_BOOTLOADER_VERSION:bsp-ti-6_12 = "2025.01%"
> >  BSP_SGX_DRIVER_PROVIDER:bsp-ti-6_12 = "ti-sgx-ddk-km"
> >  BSP_SGX_DRIVER_VERSION:bsp-ti-6_12 = "1.17%"
> >  BSP_ROGUE_DRIVER_PROVIDER:bsp-ti-6_12 = "ti-img-rogue-driver"
> >-BSP_ROGUE_DRIVER_VERSION:bsp-ti-6_12 = "24%"
> >+BSP_ROGUE_DRIVER_VERSION:bsp-ti-6_12 = "25%"
> >  BSP_MESA_PVR_VERSION:bsp-ti-6_12 = "24%"
> >  # ==========
> >@@ -60,7 +60,7 @@ BSP_BOOTLOADER_VERSION:bsp-ti-6_6 = "2024.04%"
> >  BSP_SGX_DRIVER_PROVIDER:bsp-ti-6_6 = "ti-sgx-ddk-km"
> >  BSP_SGX_DRIVER_VERSION:bsp-ti-6_6 = "1.17%"
> >  BSP_ROGUE_DRIVER_PROVIDER:bsp-ti-6_6 = "ti-img-rogue-driver"
> >-BSP_ROGUE_DRIVER_VERSION:bsp-ti-6_6 = "24%"
> >+BSP_ROGUE_DRIVER_VERSION:bsp-ti-6_6 = "25%"
> >  BSP_MESA_PVR_VERSION:bsp-ti-6_6 = "24%"
> >  # GC320 support requires out-of-tree drivers not yet available in 6.6
> >diff --git a/meta-ti-bsp/recipes-bsp/powervr-drivers/ti-img-rogue-driver_24.2.6643903.bb b/meta-ti-bsp/recipes-bsp/powervr-drivers/ti-img-rogue-driver_25.2.6850647.bb
> >similarity index 83%
> >rename from meta-ti-bsp/recipes-bsp/powervr-drivers/ti-img-rogue-driver_24.2.6643903.bb
> >rename to meta-ti-bsp/recipes-bsp/powervr-drivers/ti-img-rogue-driver_25.2.6850647.bb
> >index ddb401a5..b2f5e4ea 100644
> >--- a/meta-ti-bsp/recipes-bsp/powervr-drivers/ti-img-rogue-driver_24.2.6643903.bb
> >+++ b/meta-ti-bsp/recipes-bsp/powervr-drivers/ti-img-rogue-driver_25.2.6850647.bb
> >@@ -17,7 +17,8 @@ BRANCH = "linuxws/scarthgap/k6.12/${PV}"
> >  SRC_URI = "git://git.ti.com/git/graphics/ti-img-rogue-driver.git;protocol=https;branch=${BRANCH}"
> >-SRCREV = "8eaff654a8871118c08cfafe53795f57e3b6b396"
> >+SRCREV = "72723eea0cab47c20e5734c1c7b9a441ed2d74c2"
> >+
> 
> Extra newline?
> 
> >  TARGET_PRODUCT:j721e = "j721e_linux"
> >  TARGET_PRODUCT:j721s2 = "j721s2_linux"
> >@@ -29,7 +30,12 @@ TARGET_PRODUCT:j722s = "j722s_linux"
> >  PVR_BUILD = "release"
> >  PVR_WS = "lws-generic"
> >-EXTRA_OEMAKE += 'KERNELDIR="${STAGING_KERNEL_DIR}" BUILD=${PVR_BUILD} PVR_BUILD_DIR=${TARGET_PRODUCT} WINDOW_SYSTEM=${PVR_WS}'
> >+EXTRA_OEMAKE += 'KERNELDIR="${KBUILD_OUTPUT}" BUILD=${PVR_BUILD} PVR_BUILD_DIR=${TARGET_PRODUCT} WINDOW_SYSTEM=${PVR_WS}'
> >+
> >+do_compile(){
> >+    unset CC
> 
> This feels like something that could be fixed in the driver's makefile..

I agree with Andrew - this seems like a bug in the corresponding Makefile, 
where CC is already defined (with all the necessary flags), but it gets 
composed again directly from CROSS_COMPILE, hence causing double-prefix.


> >+    oe_runmake
> >+}
> >  do_install() {
> >      make -C ${STAGING_KERNEL_DIR} M=${B}/binary_${TARGET_PRODUCT}_${PVR_WS}_${PVR_BUILD}/target_aarch64/kbuild INSTALL_MOD_PATH=${D}${root_prefix} PREFIX=${STAGING_DIR_HOST} modules_install
> >diff --git a/meta-ti-bsp/recipes-graphics/powervr-umlibs/ti-img-rogue-umlibs_24.2.6643903.bb b/meta-ti-bsp/recipes-graphics/powervr-umlibs/ti-img-rogue-umlibs_25.2.6850647.bb
> >similarity index 98%
> >rename from meta-ti-bsp/recipes-graphics/powervr-umlibs/ti-img-rogue-umlibs_24.2.6643903.bb
> >rename to meta-ti-bsp/recipes-graphics/powervr-umlibs/ti-img-rogue-umlibs_25.2.6850647.bb
> >index 949c2d5d..e4b95f8e 100644
> >--- a/meta-ti-bsp/recipes-graphics/powervr-umlibs/ti-img-rogue-umlibs_24.2.6643903.bb
> >+++ b/meta-ti-bsp/recipes-graphics/powervr-umlibs/ti-img-rogue-umlibs_25.2.6850647.bb
> >@@ -6,11 +6,9 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=7232b98c1c58f99e3baa03de5207e76f"
> >  PACKAGE_ARCH = "${MACHINE_ARCH}"
> >  COMPATIBLE_MACHINE = "j721e|j721s2|j784s4|j742s2|am62xx|am62pxx|j722s"
> >-PR = "r4"
> >-
> >  BRANCH = "linuxws/scarthgap/k6.12/${PV}"
> >  SRC_URI = "git://git.ti.com/git/graphics/ti-img-rogue-umlibs.git;protocol=https;branch=${BRANCH}"
> >-SRCREV = "1ed9ee185cd876200e6747192854015b8e94a7b0"
> >+SRCREV = "adcbb5c620ff172da4152c02a2fee8f42dc4c472"
> >  TARGET_PRODUCT:j721e = "j721e_linux"
> >  TARGET_PRODUCT:j721s2 = "j721s2_linux"