diff mbox series

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

Message ID 20251117211517.785176-1-a-christidis@ti.com
State Accepted
Delegated to: Ryan Eatmon
Headers show
Series [meta-ti,master] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2 | expand

Commit Message

Antonios Christidis Nov. 17, 2025, 9:15 p.m. UTC 
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%)

Comments

PRC Automation Nov. 17, 2025, 9:23 p.m. UTC | #1 
meta-ti / na / 20251117211517.785176-1-a-christidis

PRC Results: PASS

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
WARN - [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
    WARN: Commit message does not include file/recipe name: ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2. (COMMIT-MESSAGE-2)
        patch



=========================================================
  apply-yocto-patch: PASS
=========================================================
master
=====================
Summary:
- Patch Series: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
- Submitter: From: <a-christidis@ti.com>
From: Antonios Christidis <a-christidis@ti.com>
- Date: Date: Mon, 17 Nov 2025 15:15:17 -0600
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: a6ba27e5e27e865365291b1d1b9a005dd449cb33

Applied to:
- Repository: lcpd-prc-meta-ti
- Base Branch: master-wip
- Commit Author: Ryan Eatmon <reatmon@ti.com>
- Commit Subject: conf: j784s4: Add k3_bist tie-test-builds config
- Commit SHA: 886e6a0587b1964665138f2173144f694d92c5c6

Patches
----------------------------------------
All patches applied



=========================================================
  check-yocto-repo: PASS
=========================================================
master
=====================
PASS



=========================================================
  yocto-check-layers: PASS
=========================================================
master - 
=====================
All checks passed
Andrew Davis Nov. 17, 2025, 11:22 p.m. UTC | #2 
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..

Andrew

> +    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"
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#19282): https://lists.yoctoproject.org/g/meta-ti/message/19282
> Mute This Topic: https://lists.yoctoproject.org/mt/116346378/3619733
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [afd@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Denys Dmytriyenko Nov. 18, 2025, 7:09 p.m. UTC | #3 
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"
Randolph Sapp Nov. 19, 2025, 1 a.m. UTC | #4 
On Tue Nov 18, 2025 at 1:09 PM CST, Denys Dmytriyenko wrote:
> 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.

I disagree. The CC variable in their build system is supposed to either be a
full host toolchain, gcc, or clang. It was never intended to be a cross
compiler. If it is a cross compiler, then it's not to be used in conjunction
with the CROSS_COMPILE variable. In either case any extra arguments provided to
the compiler in the CC variable will be stripped away.

The build system used on the km and um are the same, and the user mode
components produce intermediaries that are executed on the host during
compilation. This requirement dictated the above variable interaction.

Yocto in particular is mixing CC and CROSS_COMPILE in ways I haven't seen on
other build systems. It's also only doing this for kernel modules. Seems a
little too niche.

- Randolph

>> >+    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"
Denys Dmytriyenko Nov. 19, 2025, 2:52 p.m. UTC | #5 
On Tue, Nov 18, 2025 at 07:00:52PM -0600, Randolph Sapp via lists.yoctoproject.org wrote:
> On Tue Nov 18, 2025 at 1:09 PM CST, Denys Dmytriyenko wrote:
> > 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.
> 
> I disagree. The CC variable in their build system is supposed to either be a
> full host toolchain, gcc, or clang. It was never intended to be a cross
> compiler. If it is a cross compiler, then it's not to be used in conjunction
> with the CROSS_COMPILE variable.

> In either case any extra arguments provided to
> the compiler in the CC variable will be stripped away.

Not correct - from the make manual, see the clarification in parenthesis:

The variables used in implicit rules fall into two classes: those that are 
names of programs (like CC) and those that contain arguments for the programs 
(like CFLAGS). (The "name of a program" may also contain some command 
arguments, but it must start with an actual executable program name.) If a 
variable value contains more than one argument, separate them with spaces. 

https://www.gnu.org/software/make/manual/html_node/Implicit-Variables.html

As make itself doesn't care about host vs. cross compilation, it's a common 
practice to set CC (or CXX, etc.) accordingly:

$ make CC=$(PREFIX)-gcc


> The build system used on the km and um are the same, and the user mode
> components produce intermediaries that are executed on the host during
> compilation. This requirement dictated the above variable interaction.
> 
> Yocto in particular is mixing CC and CROSS_COMPILE in ways I haven't seen on
> other build systems. It's also only doing this for kernel modules. Seems a
> little too niche.
> 
> - Randolph
> 
> >> >+    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"
Ryan Eatmon Nov. 19, 2025, 5:19 p.m. UTC | #6 
On 11/19/2025 8:52 AM, Denys Dmytriyenko wrote:
> On Tue, Nov 18, 2025 at 07:00:52PM -0600, Randolph Sapp via lists.yoctoproject.org wrote:
>> On Tue Nov 18, 2025 at 1:09 PM CST, Denys Dmytriyenko wrote:
>>> 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.
>>
>> I disagree. The CC variable in their build system is supposed to either be a
>> full host toolchain, gcc, or clang. It was never intended to be a cross
>> compiler. If it is a cross compiler, then it's not to be used in conjunction
>> with the CROSS_COMPILE variable.
> 
>> In either case any extra arguments provided to
>> the compiler in the CC variable will be stripped away.
> 
> Not correct - from the make manual, see the clarification in parenthesis:
> 
> The variables used in implicit rules fall into two classes: those that are
> names of programs (like CC) and those that contain arguments for the programs
> (like CFLAGS). (The "name of a program" may also contain some command
> arguments, but it must start with an actual executable program name.) If a
> variable value contains more than one argument, separate them with spaces.
> 
> https://www.gnu.org/software/make/manual/html_node/Implicit-Variables.html
> 
> As make itself doesn't care about host vs. cross compilation, it's a common
> practice to set CC (or CXX, etc.) accordingly:
> 
> $ make CC=$(PREFIX)-gcc


I am all in support of doing this the "right way", but I'm going to take 
this specific patch because we have the same patch on scarthgap already. 
  So we need to iterate on a second patch that cleans up this one, please.


> 
>> The build system used on the km and um are the same, and the user mode
>> components produce intermediaries that are executed on the host during
>> compilation. This requirement dictated the above variable interaction.
>>
>> Yocto in particular is mixing CC and CROSS_COMPILE in ways I haven't seen on
>> other build systems. It's also only doing this for kernel modules. Seems a
>> little too niche.
>>
>> - Randolph
>>
>>>>> +    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"
Denys Dmytriyenko Nov. 21, 2025, 8:04 p.m. UTC | #7 
On Wed, Nov 19, 2025 at 11:19:59AM -0600, Ryan Eatmon wrote:
> 
> 
> On 11/19/2025 8:52 AM, Denys Dmytriyenko wrote:
> >On Tue, Nov 18, 2025 at 07:00:52PM -0600, Randolph Sapp via lists.yoctoproject.org wrote:
> >>On Tue Nov 18, 2025 at 1:09 PM CST, Denys Dmytriyenko wrote:
> >>>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

If we are going with this, at least we should not override the entire 
do_compile(), but simply inject "unset" at the beginning:

do_compile:prepend() {
	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.
> >>
> >>I disagree. The CC variable in their build system is supposed to either be a
> >>full host toolchain, gcc, or clang. It was never intended to be a cross
> >>compiler. If it is a cross compiler, then it's not to be used in conjunction
> >>with the CROSS_COMPILE variable.
> >
> >>In either case any extra arguments provided to
> >>the compiler in the CC variable will be stripped away.
> >
> >Not correct - from the make manual, see the clarification in parenthesis:
> >
> >The variables used in implicit rules fall into two classes: those that are
> >names of programs (like CC) and those that contain arguments for the programs
> >(like CFLAGS). (The "name of a program" may also contain some command
> >arguments, but it must start with an actual executable program name.) If a
> >variable value contains more than one argument, separate them with spaces.
> >
> >https://www.gnu.org/software/make/manual/html_node/Implicit-Variables.html
> >
> >As make itself doesn't care about host vs. cross compilation, it's a common
> >practice to set CC (or CXX, etc.) accordingly:
> >
> >$ make CC=$(PREFIX)-gcc
> 
> 
> I am all in support of doing this the "right way", but I'm going to
> take this specific patch because we have the same patch on scarthgap
> already.  So we need to iterate on a second patch that cleans up
> this one, please.
> 
> 
> >
> >>The build system used on the km and um are the same, and the user mode
> >>components produce intermediaries that are executed on the host during
> >>compilation. This requirement dictated the above variable interaction.
> >>
> >>Yocto in particular is mixing CC and CROSS_COMPILE in ways I haven't seen on
> >>other build systems. It's also only doing this for kernel modules. Seems a
> >>little too niche.
> >>
> >>- Randolph
> >>
> >>>>>+    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"
Denys Dmytriyenko Nov. 21, 2025, 10:15 p.m. UTC | #8 
On Wed, Nov 19, 2025 at 11:19:59AM -0600, Ryan Eatmon wrote:
> 
> 
> On 11/19/2025 8:52 AM, Denys Dmytriyenko wrote:
> >On Tue, Nov 18, 2025 at 07:00:52PM -0600, Randolph Sapp via lists.yoctoproject.org wrote:
> >>On Tue Nov 18, 2025 at 1:09 PM CST, Denys Dmytriyenko wrote:
> >>>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.
> >>
> >>I disagree. The CC variable in their build system is supposed to either be a
> >>full host toolchain, gcc, or clang. It was never intended to be a cross
> >>compiler. If it is a cross compiler, then it's not to be used in conjunction
> >>with the CROSS_COMPILE variable.
> >
> >>In either case any extra arguments provided to
> >>the compiler in the CC variable will be stripped away.
> >
> >Not correct - from the make manual, see the clarification in parenthesis:
> >
> >The variables used in implicit rules fall into two classes: those that are
> >names of programs (like CC) and those that contain arguments for the programs
> >(like CFLAGS). (The "name of a program" may also contain some command
> >arguments, but it must start with an actual executable program name.) If a
> >variable value contains more than one argument, separate them with spaces.
> >
> >https://www.gnu.org/software/make/manual/html_node/Implicit-Variables.html
> >
> >As make itself doesn't care about host vs. cross compilation, it's a common
> >practice to set CC (or CXX, etc.) accordingly:
> >
> >$ make CC=$(PREFIX)-gcc
> 
> 
> I am all in support of doing this the "right way", but I'm going to
> take this specific patch because we have the same patch on scarthgap
> already.

Sorry for being a PITA, but scarthgap version of this patch didn't have any 
issues with CC. Just saying...


>  So we need to iterate on a second patch that cleans up
> this one, please.
> 
> 
> >
> >>The build system used on the km and um are the same, and the user mode
> >>components produce intermediaries that are executed on the host during
> >>compilation. This requirement dictated the above variable interaction.
> >>
> >>Yocto in particular is mixing CC and CROSS_COMPILE in ways I haven't seen on
> >>other build systems. It's also only doing this for kernel modules. Seems a
> >>little too niche.
> >>
> >>- Randolph
> >>
> >>>>>+    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"
Randolph Sapp Dec. 11, 2025, 9:54 p.m. UTC | #9 
On Fri Nov 21, 2025 at 4:15 PM CST, Denys Dmytriyenko wrote:
> On Wed, Nov 19, 2025 at 11:19:59AM -0600, Ryan Eatmon wrote:
>> 
>> 
>> On 11/19/2025 8:52 AM, Denys Dmytriyenko wrote:
>> >On Tue, Nov 18, 2025 at 07:00:52PM -0600, Randolph Sapp via lists.yoctoproject.org wrote:
>> >>On Tue Nov 18, 2025 at 1:09 PM CST, Denys Dmytriyenko wrote:
>> >>>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.
>> >>
>> >>I disagree. The CC variable in their build system is supposed to either be a
>> >>full host toolchain, gcc, or clang. It was never intended to be a cross
>> >>compiler. If it is a cross compiler, then it's not to be used in conjunction
>> >>with the CROSS_COMPILE variable.
>> >
>> >>In either case any extra arguments provided to
>> >>the compiler in the CC variable will be stripped away.
>> >
>> >Not correct - from the make manual, see the clarification in parenthesis:
>> >
>> >The variables used in implicit rules fall into two classes: those that are
>> >names of programs (like CC) and those that contain arguments for the programs
>> >(like CFLAGS). (The "name of a program" may also contain some command
>> >arguments, but it must start with an actual executable program name.) If a
>> >variable value contains more than one argument, separate them with spaces.
>> >
>> >https://www.gnu.org/software/make/manual/html_node/Implicit-Variables.html
>> >
>> >As make itself doesn't care about host vs. cross compilation, it's a common
>> >practice to set CC (or CXX, etc.) accordingly:
>> >
>> >$ make CC=$(PREFIX)-gcc
>> 
>> 
>> I am all in support of doing this the "right way", but I'm going to
>> take this specific patch because we have the same patch on scarthgap
>> already.
>
> Sorry for being a PITA, but scarthgap version of this patch didn't have any 
> issues with CC. Just saying...

It actually was. The log of do_compile used to report an unusual compiler
warning, but internally resolved it. Even 25.2 does this, so this change isn't
actually necessary. I just figured specifically unsetting CC made it clear that
the variable was being ignored in the build process, for anyone who was
following along.

>>  So we need to iterate on a second patch that cleans up
>> this one, please.
>> 
>> 
>> >
>> >>The build system used on the km and um are the same, and the user mode
>> >>components produce intermediaries that are executed on the host during
>> >>compilation. This requirement dictated the above variable interaction.
>> >>
>> >>Yocto in particular is mixing CC and CROSS_COMPILE in ways I haven't seen on
>> >>other build systems. It's also only doing this for kernel modules. Seems a
>> >>little too niche.
>> >>
>> >>- Randolph
>> >>
>> >>>>>+    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"
diff mbox series

Patch

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"
+
 
 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
+    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"