Bug 15232 - meta-yocto-bsp/conf/machine/beaglebone-yocto.conf: zImage + KERNEL_EXTRA_ARGS
Summary: meta-yocto-bsp/conf/machine/beaglebone-yocto.conf: zImage + KERNEL_EXTRA_ARGS
Status: RESOLVED FIXED
Alias: None
Product: BSPs
Classification: Build System, Metadata & Runtime
Component: bsps-meta-yocto (show other bugs)
Version: 4.2.4
Hardware: Beagleboard Multiple
: Medium normal
Target Milestone: 5.1
Assignee: Kevin Hao
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2023-10-12 07:10 UTC by Robert Berger
Modified: 2024-09-26 18:18 UTC (History)
1 user (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: Don't know

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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