Re: [meta-ti][scarthgap][RFC 2/6] ti-bsp: Do not remove gpu MACHINE_FEATURE conditionally on BSP version

[Denys Dmytriyenko <denis@denix.org>]

On Tue, Sep 09, 2025 at 10:10:24AM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> Setting or removing the gpu MACHINE_FEATURE does not actually change
> if the GPU driver is built or not. That is done based on the setting
> of the BSP_{SGX,ROGUE}_{DRIVER,UMLIBS}_{PROVIDER,VERSION} vars. These
> are default empty for BSP versions without supported GPU drivers, which
> will correctly cause a fallback to SW rendering when the GPU driver is
> not available.

While cleaning the remaining uses of "gpu" MACHINE_FEATURE, why not remove it 
completely from all the machine configs?


> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  meta-ti-bsp/conf/machine/include/ti-bsp.inc | 8 --------
>  1 file changed, 8 deletions(-)
> 
> diff --git a/meta-ti-bsp/conf/machine/include/ti-bsp.inc b/meta-ti-bsp/conf/machine/include/ti-bsp.inc
> index 262aa884..a5d9a51f 100644
> --- a/meta-ti-bsp/conf/machine/include/ti-bsp.inc
> +++ b/meta-ti-bsp/conf/machine/include/ti-bsp.inc
> @@ -18,10 +18,6 @@ BSP_KERNEL_VERSION:bsp-mainline = "%"
>  BSP_BOOTLOADER_PROVIDER:bsp-mainline = "u-boot-ti-mainline"
>  BSP_BOOTLOADER_VERSION:bsp-mainline = "%"
>  
> -# GPU support requires out-of-tree SGX and Rogue drivers not available
> -# 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"
>  
> @@ -34,10 +30,6 @@ BSP_KERNEL_VERSION:bsp-next = "%"
>  BSP_BOOTLOADER_PROVIDER:bsp-next = "u-boot-ti-next"
>  BSP_BOOTLOADER_VERSION:bsp-next = "%"
>  
> -# GPU support requires out-of-tree SGX and Rogue drivers not available
> -# 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"
>  
> -- 
> 2.39.2