zeus branch raspberrypi cm3 u-boot no messages in console

[nandra]

Using meta-raspberrypi branch zeus with raspberrypi cm3 module and enable u-boot in localconf. Build image doesn't show any info in console. I tried older u-boot 2018.01 and it works fine. Ideas?
Console is on PIN 14 and 15.
Thanks.

[agherzan]

Could it be related to raspberrypi/firmware#1391 ?

[nandra]

Issue was missgin u-boot patch: commit id: 29f7d05a347ab7a42577c67fdfb787ef91537302

[agherzan]

Do we have that in meta-raspberrypi?

[nandra]

Nope should be cherrypicke at least for have u-boot 2020.01 working on cm3.

[mike-scott]

@nandra Ah that makes sense. I ended up migrating to a newer u-boot and my boot issues on CM3 went away.

@agherzan
I notice that https://github.com/agherzan/meta-raspberrypi/blob/master/conf/machine/raspberrypi-cm3.conf doesn't set:
VC4DTBO ?= "vc4-fkms-v3d" like the other machine configurations.

I had to temporarily setup an append to get working fkms graphics.

Otherwise the default setting here is set (kms):
https://github.com/agherzan/meta-raspberrypi/blob/master/recipes-bsp/bootfiles/rpi-config_git.bb#L27

[mike-scott]

To clarify: I know that this setting in config.txt shouldn't really affect u-boot / Linux. But I was getting a GPU hang during Linux boot until I made sure the config.txt had the right setting here. Who knows what the RaspberryPi firmware does prior to u-boot handoff.

[nandra]

Fix: #660

[amgedr]

U-Boot was not working in dunfell for the CM3+. It was fixed when I manually added the changes in #660. Why was the commit reverted in the dunfell branch?

[agherzan]

Can you point me to the revert PR?

[amgedr]

Sorry, just found the PR. It's #699 and as you mentioned it is just missing an append to FILESEXTRAPATHS. I thought there is a bug in the patch.

[nandra]

Also stock zeus build (without u-boot) doesn't boot on CM3. Anyone have some idea? CM3 is light version without eMMC. Thanks.

[nandra]

Nah I was wrong. It just took some time until it start and need to enable console :). Sorry for noise.

[agherzan]

@amgedr That PR was a hefty backport. Would be hard for me to spot the issue from it. We would need to do a bit of investigation.

[amgedr]

Hope this helps when you get a chance to review: #660's commit f20ccfa solved the CM3+ booting issue when u-boot is enabled. But as you commented in #699, it was reverted in 8258e73 because it was missing the following line:

FILESEXTRAPATHS_prepend := "${THISDIR}/files:"

[agherzan]

That makes sense - and I remember now. Do you feel like pushing that? CC @nandra

[nandra]

Should I redo patch or @amgedr will you do it? Thanks.

[agherzan]

I didn't know you are up for it. If you want, it would probably be better if you could do it as you know better patch.

[nandra]

PR #747

[agherzan]

Thanks @nandra . @amgedr do you have a second to give the PR a spin?

[amgedr]

Sorry, didn't notice the notification email earlier. Works on CM3+.

[vkotarov]

Not sure if this is the right place to comment but the patch somehow made it to gatesgarth. On the other hand, it's not needed there as u-boot version is 2020.07 and already has it applied. Eventually, the do_patch task breaks on gatesgarth (probably on hardknott, too)

[agherzan]

Looks fixed. Ping me otherwise.