[SRU] Add Xilinx ZCU111 board support and improve messaging in U-Boot boot script for board detection

Bug #1978798 reported by ethan.hsieh
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
flash-kernel (Ubuntu)
Fix Released
Medium
Unassigned
Jammy
Fix Released
Undecided
Unassigned
Kinetic
Fix Released
Medium
Unassigned

Bug Description

[Impact]

Add Xilinx ZCU111 board support and improve messaging in U-Boot boot script for board detection

[Where problems could occur]

The patch adds support for the Xilinx ZCU111, but affects the boot script used by:

* ZCU 102, 104, 106, 111
* Kria KR revisions A, B, 1
* Kria KV revisions A, B, Z, 1

[Test Case]

* Enable proposed updates (https://wiki.ubuntu.com/Testing/EnableProposed)
* Update flash-kernel to the version in -proposed: sudo apt install -t jammy-proposed flash-kernel
* Run flash-kernel (if the upgrade didn't already): sudo flash-kernel
* Reboot: sudo reboot
* Ensure system boots successfully

[Regression Potential]

As the patch touches the u-boot script used by several boards (listed above), booting should be re-tested against the potentially affected boards under the proposed package.

Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :
tags: added: originate-from-1975895
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "kinetic.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :

Re-create kinetic.debdiff because flash-kernel has been upgraded to (3.106ubuntu1).

Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :

Xilinx would like to update U-Boot boot script, so I'll upload a new debdiff for kinetic later.

Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :
Mathew Hodson (mhodson)
Changed in flash-kernel (Ubuntu):
importance: Undecided → Medium
Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :

Re-create kinetic.debdiff because flash-kernel has been upgrade to 3.106ubuntu2.

Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :
Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :

Some changes may be required for ZCU111. I'm confirming with Xilinx. I'll give an update soon.

Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :

Confirmed with AMD-Xilinx that debdiff (comment#6 and #7) for kinetic and jammy have all pieces for ZCU111.

Revision history for this message
Dave Jones (waveform) wrote :

I've uploaded the version for kinetic. For the SRU, I've updated the test case and regression potential sections because, as the patch now touches u-boot scripts used by several boards (and as the patch does change a fair bit of the u-boot logic, at least in so far as board recognition is concerned) those boards have a regression potential and should be tested against the -proposed version (which I'll upload shortly).

description: updated
summary: - Add Xilinx ZCU111 board support and improve messaging in U-Boot boot
- script for board detection
+ [SRU] Add Xilinx ZCU111 board support and improve messaging in U-Boot
+ boot script for board detection
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package flash-kernel - 3.106ubuntu3

---------------
flash-kernel (3.106ubuntu3) kinetic; urgency=medium

  [ Ethan Hsieh ]

  * Add ZCU111 support (LP: #1978798)
  * bootscr.zynqmp (LP: #1978798)
    - Improve messaging in U-Boot boot script for board detection
    - Add kria and zcu board support lists
    - Update kR260 and KV260's bootargs as per AMD-Xilinx's request
    - Support legacy KV260 U-Boot

  [ Alexandre Ghiti ]

  * Allow to bypass container exit test by introducing FK_FORCE_CONTAINER
    for RISC-V images build to run flash-kernel in a container (LP: #1980929)

 -- Dave Jones <email address hidden> Thu, 14 Jul 2022 13:19:57 +0100

Changed in flash-kernel (Ubuntu Kinetic):
status: New → Fix Released
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Please test proposed package

Hello ethan.hsieh, or anyone else affected,

Accepted flash-kernel into jammy-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/flash-kernel/3.104ubuntu12 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-jammy to verification-done-jammy. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-jammy. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in flash-kernel (Ubuntu Jammy):
status: New → Fix Committed
tags: added: verification-needed verification-needed-jammy
Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :

Upgraded flash-kernel to (3.104ubuntu12). System can boot with new fitimage without any problem.

Boards:
1. Kria KV260: Pass
2. ZCU 106: Pass

tags: added: verification-done-jammy
removed: verification-needed-jammy
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package flash-kernel - 3.104ubuntu12

---------------
flash-kernel (3.104ubuntu12) jammy; urgency=medium

  [ Ethan Hsieh ]

  * Add ZCU111 support (LP: #1978798)
  * bootscr.zynqmp (LP: #1978798)
    - Improve messaging in U-Boot boot script for board detection
    - Add kria and zcu board support lists
    - Update kR260 and KV260's bootargs as per AMD-Xilinx's request

  [ Alexandre Ghiti ]

  * Allow to bypass container exit test by introducing FK_FORCE_CONTAINER
    for RISC-V images build to run flash-kernel in a container (LP: #1980929)

 -- Dave Jones <email address hidden> Thu, 14 Jul 2022 13:49:46 +0100

Changed in flash-kernel (Ubuntu Jammy):
status: Fix Committed → Fix Released
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Update Released

The verification of the Stable Release Update for flash-kernel has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.