* [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
@ 2025-11-17 21:15 a-christidis
2025-11-17 21:23 ` PRC Automation
2025-11-17 23:22 ` Andrew Davis
0 siblings, 2 replies; 10+ messages in thread
From: a-christidis @ 2025-11-17 21:15 UTC (permalink / raw)
To: meta-ti, reatmon; +Cc: denys, rs
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"
+
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"
--
2.34.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
2025-11-17 21:15 [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2 a-christidis
@ 2025-11-17 21:23 ` PRC Automation
2025-11-17 23:22 ` Andrew Davis
1 sibling, 0 replies; 10+ messages in thread
From: PRC Automation @ 2025-11-17 21:23 UTC (permalink / raw)
To: Antonios Christidis; +Cc: meta-ti, reatmon, denys, rs
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
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
2025-11-17 21:15 [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2 a-christidis
2025-11-17 21:23 ` PRC Automation
@ 2025-11-17 23:22 ` Andrew Davis
2025-11-18 19:09 ` Denys Dmytriyenko
1 sibling, 1 reply; 10+ messages in thread
From: Andrew Davis @ 2025-11-17 23:22 UTC (permalink / raw)
To: a-christidis, meta-ti, reatmon; +Cc: denys, rs
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]
> -=-=-=-=-=-=-=-=-=-=-=-
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
2025-11-17 23:22 ` Andrew Davis
@ 2025-11-18 19:09 ` Denys Dmytriyenko
2025-11-19 1:00 ` Randolph Sapp
0 siblings, 1 reply; 10+ messages in thread
From: Denys Dmytriyenko @ 2025-11-18 19:09 UTC (permalink / raw)
To: afd; +Cc: a-christidis, meta-ti, reatmon, denys, rs
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"
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
2025-11-18 19:09 ` Denys Dmytriyenko
@ 2025-11-19 1:00 ` Randolph Sapp
2025-11-19 14:52 ` Denys Dmytriyenko
0 siblings, 1 reply; 10+ messages in thread
From: Randolph Sapp @ 2025-11-19 1:00 UTC (permalink / raw)
To: Denys Dmytriyenko, afd; +Cc: a-christidis, meta-ti, reatmon, denys, rs
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"
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
2025-11-19 1:00 ` Randolph Sapp
@ 2025-11-19 14:52 ` Denys Dmytriyenko
2025-11-19 17:19 ` Ryan Eatmon
0 siblings, 1 reply; 10+ messages in thread
From: Denys Dmytriyenko @ 2025-11-19 14:52 UTC (permalink / raw)
To: rs; +Cc: afd, a-christidis, meta-ti, reatmon, denys
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"
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
2025-11-19 14:52 ` Denys Dmytriyenko
@ 2025-11-19 17:19 ` Ryan Eatmon
2025-11-21 20:04 ` Denys Dmytriyenko
2025-11-21 22:15 ` Denys Dmytriyenko
0 siblings, 2 replies; 10+ messages in thread
From: Ryan Eatmon @ 2025-11-19 17:19 UTC (permalink / raw)
To: Denys Dmytriyenko, rs; +Cc: afd, a-christidis, meta-ti, denys
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"
--
Ryan Eatmon reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc. - LCPD - MGTS
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
2025-11-19 17:19 ` Ryan Eatmon
@ 2025-11-21 20:04 ` Denys Dmytriyenko
2025-11-21 22:15 ` Denys Dmytriyenko
1 sibling, 0 replies; 10+ messages in thread
From: Denys Dmytriyenko @ 2025-11-21 20:04 UTC (permalink / raw)
To: Ryan Eatmon; +Cc: rs, afd, a-christidis, meta-ti, denys
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"
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
2025-11-19 17:19 ` Ryan Eatmon
2025-11-21 20:04 ` Denys Dmytriyenko
@ 2025-11-21 22:15 ` Denys Dmytriyenko
2025-12-11 21:54 ` Randolph Sapp
1 sibling, 1 reply; 10+ messages in thread
From: Denys Dmytriyenko @ 2025-11-21 22:15 UTC (permalink / raw)
To: Ryan Eatmon; +Cc: rs, afd, a-christidis, meta-ti, denys
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"
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2
2025-11-21 22:15 ` Denys Dmytriyenko
@ 2025-12-11 21:54 ` Randolph Sapp
0 siblings, 0 replies; 10+ messages in thread
From: Randolph Sapp @ 2025-12-11 21:54 UTC (permalink / raw)
To: Denys Dmytriyenko, Ryan Eatmon; +Cc: rs, afd, a-christidis, meta-ti, denys
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"
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2025-12-11 21:54 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-17 21:15 [meta-ti][master][PATCH] ti-img-rogue-*: Migrate GPU driver version 24.2 -> 25.2 a-christidis
2025-11-17 21:23 ` PRC Automation
2025-11-17 23:22 ` Andrew Davis
2025-11-18 19:09 ` Denys Dmytriyenko
2025-11-19 1:00 ` Randolph Sapp
2025-11-19 14:52 ` Denys Dmytriyenko
2025-11-19 17:19 ` Ryan Eatmon
2025-11-21 20:04 ` Denys Dmytriyenko
2025-11-21 22:15 ` Denys Dmytriyenko
2025-12-11 21:54 ` Randolph Sapp
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.