On 1/10/2025 6:45 PM, Denys Dmytriyenko wrote:
On Fri, Jan 10, 2025 at 02:00:26PM -0600, Ryan Eatmon wrote:


On 1/10/2025 1:30 PM, Ryan Eatmon via lists.yoctoproject.org wrote:


On 1/10/2025 12:06 PM, Denys Dmytriyenko wrote:
On Wed, Jan 08, 2025 at 02:51:45PM -0600, Andrew Davis via
lists.yoctoproject.org wrote:
Like we do with GPU support, GC320 support is conditional on the
kernel we are building and so should be disabled here for kernels
on which it is not functional yet.

We can then remove these disabling lines as the driver is updated
for each supported kernel, instead of blanket disabling in the
distro layer as done currently.

Signed-off-by: Andrew Davis <[email protected]>
---

Changes for v2:
   - Rebased on latest with new meta-beagle layer

   meta-beagle/conf/machine/include/beagle-bsp.inc |  6 ++++++
   meta-ti-bsp/conf/machine/include/ti-bsp.inc     | 15 +++++++++++++++
   2 files changed, 21 insertions(+)

diff --git a/meta-beagle/conf/machine/include/beagle-bsp.inc
b/meta-beagle/conf/machine/include/beagle-bsp.inc
index 34889b41..06483ba9 100644
--- a/meta-beagle/conf/machine/include/beagle-bsp.inc
+++ b/meta-beagle/conf/machine/include/beagle-bsp.inc
@@ -23,6 +23,9 @@ BSP_ROGUE_DRIVER_VERSION:bsp-bb_org-6_1 = "23%"
   BSP_ROGUE_UMLIBS_VERSION:bsp-bb_org-6_1 = "23%"
   BSP_MESA_PVR_VERSION:bsp-bb_org-6_1 = "22%"
+# GC320 support requires out-of-tree drivers not yet
available in bb.org 6.1
+MACHINE_FEATURES:remove:bsp-bb_org-6_1 = "gc320"
+
   # ==========
   # bb_org-6_6
   # BeagleBoard.org kernel 6.6, Upstream u-boot 2024.07
@@ -41,3 +44,6 @@ 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_UMLIBS_VERSION:bsp-bb_org-6_6 = "24%"
   BSP_MESA_PVR_VERSION:bsp-bb_org-6_6 = "23%"
+
+# GC320 support requires out-of-tree drivers not yet
available in bb.org 6.6
+MACHINE_FEATURES:remove:bsp-bb_org-6_6 = "gc320"
diff --git a/meta-ti-bsp/conf/machine/include/ti-bsp.inc
b/meta-ti-bsp/conf/machine/include/ti-bsp.inc
index 4bbe7668..6773bb7a 100644
--- a/meta-ti-bsp/conf/machine/include/ti-bsp.inc
+++ b/meta-ti-bsp/conf/machine/include/ti-bsp.inc
@@ -22,6 +22,9 @@ BSP_BOOTLOADER_VERSION:bsp-mainline = "%"
   # in mainline, usually present in TI staging or derivative like BB.org
   MACHINE_FEATURES:remove:bsp-mainline = "gpu"
+# GC320 support requires out-of-tree drivers not available in mainline
+MACHINE_FEATURES:remove:bsp-mainline = "gc320"
+
   # ==========
   # next
   # upcoming upstream/mainline kernel, u-boot
@@ -35,6 +38,9 @@ BSP_BOOTLOADER_VERSION:bsp-next = "%"
   # in next, usually present in TI staging or derivative like BB.org
   MACHINE_FEATURES:remove:bsp-next = "gpu"
+# GC320 support requires out-of-tree drivers not available in next
+MACHINE_FEATURES:remove:bsp-next = "gc320"

Andrew, Ryan,

Something went wrong here ^^^
Instead of removing "gc320" for bsp-next, like in the patch here, it got
committed removing it for bsp-ti-6_12 twice:

https://git.yoctoproject.org/meta-ti/commit/?h=master-wip&id=fe0b35f5f135b98d1ee0f8fe2488b10fbe9a7bb3

Was there any manual massaging involved?

Yes.  I did.  Let me look at it.

Fixed on master-wip, scarthgap-wip was correct.

bsp-ti-6_next ?

Sigh...  Fixed for real this time.



+
   # ==========
   # ti-6_12
   # TI staging kernel 6.12, u-boot 2024.10
@@ -48,6 +54,9 @@ BSP_BOOTLOADER_VERSION:bsp-ti-6_12 = "2024.10%"
   # in 6.12 yet
   MACHINE_FEATURES:remove:bsp-ti-6_12 = "gpu"
+# GC320 support requires out-of-tree drivers not yet available in 6.12
+MACHINE_FEATURES:remove:bsp-ti-6_12 = "gc320"
+
   # ==========
   # ti-6_6
   # TI staging kernel 6.6, u-boot 2024.04
@@ -65,6 +74,9 @@ BSP_ROGUE_DRIVER_VERSION:bsp-ti-6_6 = "24%"
   BSP_ROGUE_UMLIBS_VERSION:bsp-ti-6_6 = "24%"
   BSP_MESA_PVR_VERSION:bsp-ti-6_6 = "23%"
+# GC320 support requires out-of-tree drivers not yet available in 6.6
+MACHINE_FEATURES:remove:bsp-ti-6_6 = "gc320"
+
   # ==========
   # ti-6_1
   # TI staging kernel 6.1, u-boot 2023.04
@@ -82,6 +94,9 @@ BSP_ROGUE_DRIVER_VERSION:bsp-ti-6_1 = "23%"
   BSP_ROGUE_UMLIBS_VERSION:bsp-ti-6_1 = "23%"
   BSP_MESA_PVR_VERSION:bsp-ti-6_1 = "22%"
+# GC320 support requires out-of-tree drivers not yet available in 6.1
+MACHINE_FEATURES:remove:bsp-ti-6_1 = "gc320"
+
   # ==========
   # sane fallback defaults
   # if specific values are not defined or bsp is set incorrectly
--
2.39.2

--
Ryan Eatmon                [email protected]
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#18188): 
https://lists.yoctoproject.org/g/meta-ti/message/18188
Mute This Topic: https://lists.yoctoproject.org/mt/110504595/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to