On Thu, May 04, 2023 at 05:37:35AM -0700, Daniel Schultz wrote:
> Select the correct WKS file according to the MACHINE_FEATURE efi.
> 
> Right now, the sdimage-2part-efi.wks is set by default. Switch to the
> sdimage-2part.wks file, if efi was removed from MACHINE_FEATURES to allow
> non-efi boot methods.
> 
> Signed-off-by: Daniel Schultz <[email protected]>
> ---
>  meta-ti-bsp/conf/machine/include/k3.inc | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-ti-bsp/conf/machine/include/k3.inc 
> b/meta-ti-bsp/conf/machine/include/k3.inc
> index 31da2ca9..fff15505 100644
> --- a/meta-ti-bsp/conf/machine/include/k3.inc
> +++ b/meta-ti-bsp/conf/machine/include/k3.inc
> @@ -50,7 +50,8 @@ IMAGE_EFI_BOOT_FILES ?= "${IMAGE_BOOT_FILES}"
>  EFI_PROVIDER ?= "grub-efi"
>  MACHINE_FEATURES += "efi"
>  
> -WKS_FILE ?= "sdimage-2part-efi.wks"
> +WKS_FILE ?= "${@bb.utils.contains("MACHINE_FEATURES", "efi", 
> "sdimage-2part-efi.wks", "sdimage-2part.wks", d)}"

Well, "efi" gets added to MACHINE_FEATURES just one line above. So, in order 
to disable it, you'd have to modify MACHINE_FEATURES in your configuration, at 
which point it is even more easier to set WKS_FILE there too, as it's weakly 
assigned here anyway...

But I guess I don't mind a little bit more automation either, even if it gets 
the code more complicated.


>  do_image_wic[depends] += "virtual/bootloader:do_deploy"
>  do_image_complete[mcdepends] += "mc::k3r5:ti-sci-fw:do_deploy"
>  
> -- 
> 2.25.1
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#16491): 
https://lists.yoctoproject.org/g/meta-ti/message/16491
Mute This Topic: https://lists.yoctoproject.org/mt/98682620/21656
Group Owner: [email protected]
Unsubscribe: 
https://lists.yoctoproject.org/g/meta-ti/leave/6695321/21656/1393940836/xyzzy 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to