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 ?


> >>>+
> >>>  # ==========
> >>>  # 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
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#18187): 
https://lists.yoctoproject.org/g/meta-ti/message/18187
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