Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

scarthgap/raspberrypi5 u-boot #1304

Merged
merged 3 commits into from
May 28, 2024

Conversation

moto-timo
Copy link
Contributor

- What I did

  1. Re-enable u-boot for raspberrypi5.
  2. Add the meta-lts-mixins scarthgap/u-boot layer to LAYERRECOMMENDS (so that, for instance, layerindex can know about the recommendation).
  3. Update the KERNEL_IMAGETYPE_UBOOT to be in line with raspberypi4-64 so that it is functional.

- How I did it
Create a meta-lts-mixins scarthgap/u-boot branch so we can have newer u-boot than what is released in openembedded-core.

Known issues: For the author at least, a debug UART cable must be attached to the UART port between HDMI0 and HDMI. We know the u-boot support in v2024.04 was flagged as initial so we should not expect everything to be completely functional yet (usb, ethernet).

This is the same content as the mailing list pull request: https://lore.kernel.org/yocto/[email protected]/T/#m9b15ef9ee8601531fdbf3afce60444d25848b06f

Add lts-u-boot-mixin to LAYERRECOMMENDS

https://git.yoctoproject.org/meta-lts-mixins 'scarthgap/u-boot' branch
adds u-boot v2024.04 support which includes the initial support for
raspberrypi5.

Signed-off-by: Tim Orling <[email protected]>
With meta-lts-mixins scarthgap/u-boot we can now support raspberrypi5

Signed-off-by: Tim Orling <[email protected]>
Similar to raspberrypi4-64.conf, we need to set KERNEL_IMAGETYPE_UBOOT to
"Image" and set KERNEL_BOOTCMD to "booti"

Signed-off-by: Tim Orling <[email protected]>
@moto-timo moto-timo changed the title Scarthgap/raspberrypi5 u boot scarthgap/raspberrypi5 u-boot Apr 7, 2024
Copy link
Contributor

@leon-anavi leon-anavi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I tested successfully this GitHub pull request on Raspberry Pi 5 and U-Boot 2022.04 using layer meta-lts-mixins:

U-Boot 2024.04 (Apr 02 2024 - 10:58:58 +0000)

DRAM:  1020 MiB (effective 4 GiB)
mbox: Header response code invalid
RPI 5 Model B (0xc04170)
Core:  20 devices, 10 uclasses, devicetree: board
MMC:   mmc@fff000: 0, mmc@1100000: 1
Loading Environment from FAT... OK
In:    serial,usbkbd
Out:   serial,vidconsole
Err:   serial,vidconsole
mbox: Header response code invalid
bcm2835: Could not query MAC address
Net:   No ethernet found.
starting USB...
No working controllers found
Hit any key to stop autoboot:  0 

Thanks, Leon

jpuhlman pushed a commit to MontaVista-OpenSourceTechnology/meta-agl that referenced this pull request May 23, 2024
Changes:
- Add meta-lts-mixins_u-boot layer to BBLAYERS for raspberrypi5
  to get U-Boot v2024.04.
- Add required configuration to make the newer U-Boot buildable to
  the raspberrypi5 machine configuration.  The tweaks to re-enable
  U-Boot were sourced from the commits in this meta-raspberrypi PR:
  agherzan/meta-raspberrypi#1304
  If those get merged our configuration can be simplified.
- Add required LICENSE_FLAGS_ACCEPTED tweak to raspberrypi5 machine
  configuration (copied from raspberrypi4).
- Remove obsolete Xen tweaks from the raspberrypi4 and raspberrypi5
  machine configurations.

Bug-AGL: SPEC-5144

Change-Id: If02e0cc7baada46f4a9db305d3babe5b4a6784ff
Signed-off-by: Scott Murray <[email protected]>
Reviewed-on: https://gerrit.automotivelinux.org/gerrit/c/AGL/meta-agl/+/29908
ci-image-boot-test: Jenkins Job builder account
Tested-by: Jenkins Job builder account
ci-image-build: Jenkins Job builder account
Reviewed-by: Jan-Simon Moeller <[email protected]>
conf/layer.conf Show resolved Hide resolved
@agherzan agherzan merged commit fb03b44 into agherzan:master May 28, 2024
18 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants