Bug 15232

Summary: meta-yocto-bsp/conf/machine/beaglebone-yocto.conf: zImage + KERNEL_EXTRA_ARGS
Product: [Build System, Metadata & Runtime] BSPs Reporter: Robert Berger <pokylinux>
Component: bsps-meta-yoctoAssignee: Kevin Hao <kexin.hao>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Medium CC: randy.macleod
Version: 4.2.4   
Target Milestone: 5.1   
Hardware: Beagleboard   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: Don't know

Description Robert Berger 2023-10-12 07:10:56 UTC
Here[1]:

KERNEL_IMAGETYPE = "zImage"

and

KERNEL_EXTRA_ARGS += "LOADADDR=${UBOOT_ENTRYPOINT}"
UBOOT_ENTRYPOINT = "0x80008000"
UBOOT_LOADADDRESS = "0x80008000"

are defined.

I would say that in order to build a zImage this is not required. 
A uImage would need it.

Does it make any sense to define a zImage together with KERNEL_EXTRA_ARGS += "LOADADDR=${UBOOT_ENTRYPOINT}"?

[1] https://git.yoctoproject.org/poky/tree/meta-yocto-bsp/conf/machine/beaglebone-yocto.conf
Comment 1 Kevin Hao 2023-10-12 23:52:46 UTC
Yes, these variables seem useless for the zImage. I will make a patch to drop all of them.
Comment 2 Robert Berger 2023-10-13 06:51:15 UTC
OK this makes sense.Thanks!
Comment 3 Robert Berger 2024-05-23 06:35:46 UTC
It looks like this has been fixed.
Comment 4 Robert Berger 2024-05-23 06:37:25 UTC
commit 5d7a4304d91800ee1cb8d4618ce1867c0a47abd5
Author: Kevin Hao <kexin.hao@windriver.com>
Date:   Tue Jan 2 19:57:09 2024 +0800

    beaglebone-yocto: Remove the obsolete variables for uImage
    
    Now we only support the zImage kernel type for this machine, so remove
    the obsolete variables for uImage kernel type.
    
    [YOCTO #15232]
    
    Reported-by: Robert Berger <pokylinux@reliableembeddedsystems.com>
    (From meta-yocto rev: a8a5abe05ec4e96287acc994883f4b977290cde1)
    
    Signed-off-by: Kevin Hao <kexin.hao@windriver.com>
    Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Comment 5 Randy MacLeod 2024-09-26 18:18:52 UTC
Kevin,

Looks like this is resolved:
❯ git branch -a --contains  5d7a4304d91800ee1cb8d4618ce1867c0a47abd5
* master
  scarthgap
  remotes/origin/HEAD -> origin/master
  remotes/origin/genericarm64
  remotes/origin/master
  remotes/origin/master-next
  remotes/origin/scarthgap
  remotes/origin/styhead