starfive visionfive2: use upstream u-boot v2024.01-rc5

[NickCao]

Description of changes

Things done

• Tested the changes in your own NixOS Configuration
• Tested the changes end-to-end by using your fork of nixos-hardware and
  importing it via <nixos-hardware> or Flake input

[NickCao]

cc @leifhelm, please test.

[NickCao]

Also @colemickens and @6543

[NickCao]

Tested booting. visionfive2-firmware-update-flash fails due to u-boot.itb too big to fit into mtd1.

[6543]

Uh let me try that patch out :)

[leifhelm]

I can boot. However I cannot ssh into it. (can’t even ping it). I guess this is a problem of either my config or the kernel.

[Mic92]

Tested booting. visionfive2-firmware-update-flash fails due to u-boot.itb too big to fit into mtd1.

Is this a blocker of this pull request?

[NickCao]

Is this a blocker of this pull request?

Kind of, we need a fix like #646

[leifhelm]

According to jh7110-starfive-visionfive-2.dtsi there should be 3 MiB space and u-boot.itb is only 869 KiB. However I notice that there are 2 further partitions. Maybe we have to switch to mtd2 if it exists?

[NickCao]

According to jh7110-starfive-visionfive-2.dtsi there should be 3 MiB space and u-boot.itb is only 869 KiB. However I notice that there are 2 further partitions. Maybe we have to switch to mtd2 if it exists?

Reasonable, the partition is also named uboot in stead of uboot-env, seemingly more appropriate for our usage.

[Mic92]

Please run the merge queue command (mergify) when you think this is ready.

[NickCao]

According to jh7110-starfive-visionfive-2.dtsi there should be 3 MiB space and u-boot.itb is only 869 KiB. However I notice that there are 2 further partitions. Maybe we have to switch to mtd2 if it exists?

Reasonable, the partition is also named uboot in stead of uboot-env, seemingly more appropriate for our usage.

Confirmed by vendor documentation: https://doc-en.rvspace.org/VisionFive2/Quick_Start_Guide/VisionFive2_SDK_QSG/spl_new.html

[NickCao]

@mergify queue

[mergify]

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 8e34f33