Repository navigation
ci, docs: build, smoke-test and release openipc-novatek #54
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
Merged
Merged
Changes from all commits
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,355 @@ | ||
| name: Build novatek | ||
|
|
||
| on: | ||
| # Pull requests against this branch change the workflow itself; they | ||
| # build the current openipc-novatek. Code changes reach it through | ||
| # workflow_call from the caller on openipc-novatek, which passes the revision | ||
| # to build. Only workflow_dispatch can release or publish. | ||
| pull_request: | ||
| branches: [openipc] | ||
| paths: | ||
| - .github/workflows/build-novatek.yml | ||
| workflow_call: | ||
| inputs: | ||
| ref: | ||
| description: openipc-novatek revision to build | ||
| type: string | ||
| required: true | ||
| workflow_dispatch: | ||
| inputs: | ||
| release: | ||
| description: Publish to this repo's Novatek release | ||
| type: boolean | ||
| default: false | ||
| publish: | ||
| description: Publish to OpenIPC/firmware latest and report to openipc.org | ||
| type: boolean | ||
| default: false | ||
|
|
||
| # The shape of build-hisilicon.yml: | ||
| # build (matrix, every SoC) -> qemu (prep) -> qemu_smoke (hard gate) -> | ||
| # release (own, `release` input) + publish (OpenIPC/firmware latest + | ||
| # openipc.org report, `publish` input). | ||
| # build.sh takes SoCs as arguments; the matrix calls it with one SoC per leg. | ||
|
|
||
| jobs: | ||
| # Resolve the openipc-novatek revision this run builds from. The workflow | ||
| # lives on the openipc branch, so github.sha is the openipc commit, not | ||
| # the revision being built; release provenance must name the commit that | ||
| # build.sh actually compiled. | ||
| resolve: | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 5 | ||
| outputs: | ||
| sha: ${{ steps.rev.outputs.sha }} | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| with: | ||
| ref: ${{ inputs.ref || 'openipc-novatek' }} | ||
| path: src | ||
|
|
||
| - name: Resolve revision | ||
| id: rev | ||
| run: echo "sha=$(git -C src rev-parse HEAD)" >> "$GITHUB_OUTPUT" | ||
|
|
||
| build: | ||
| needs: resolve | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 30 | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| include: | ||
| - { soc: nt98562, files: "output/u-boot-nt98562-*.bin" } | ||
| - { soc: nt98566, files: "output/u-boot-nt98566-*.bin" } | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| # The commit resolve recorded, not the ref again: every leg then | ||
| # builds the revision the release and openipc.org report name, | ||
| # even if the branch moves while the matrix runs. | ||
| ref: ${{ needs.resolve.outputs.sha }} | ||
|
|
||
| - name: Install dependencies | ||
| timeout-minutes: 15 | ||
| run: | | ||
| # apt mirrors on hosted runners intermittently HANG on the | ||
| # mirrorlist fetch (not just fail). Bound each attempt with its own | ||
| # `timeout` so a hang is killed fast and the retry actually fires. | ||
| install() { | ||
| sudo timeout 180 apt-get update && \ | ||
| sudo timeout 240 apt-get install -y --no-install-recommends \ | ||
| build-essential gcc-arm-linux-gnueabi gcc-arm-none-eabi \ | ||
| device-tree-compiler \ | ||
| bison flex bc libssl-dev python3 | ||
| } | ||
| for i in 1 2 3 4; do | ||
| install && break | ||
| [ "$i" = 4 ] && { echo "apt failed after 4 attempts" >&2; exit 1; } | ||
| echo "apt attempt $i failed/timed out; retrying in $((i*10))s" >&2 | ||
| sleep $((i*10)) | ||
| done | ||
|
|
||
| - name: Build ${{ matrix.soc }} | ||
| run: bash build.sh ${{ matrix.soc }} | ||
|
qodo-free-for-open-source-projects[bot] marked this conversation as resolved.
|
||
|
|
||
| - name: Check partition fit | ||
| run: | | ||
| # The boot partition from this tree's mtdparts: NOR boot = 256 KiB | ||
| # (env at 0x40000). An image larger than it overflows into env and | ||
| # bricks the device on upgrade. The image includes the loader | ||
| # and the device tree it reads. | ||
| soc='${{ matrix.soc }}' | ||
| rc=0 | ||
| check_kind() { | ||
| kind=$1 | ||
| limit_kb=$2 | ||
| pattern=$3 | ||
| found=0 | ||
| for f in $pattern; do | ||
| [ -f "$f" ] || continue | ||
| found=1 | ||
| # Compare bytes: rounding down to KiB would pass an image up | ||
| # to 1023 bytes into the environment. | ||
| size=$(stat -c %s "$f") | ||
| limit=$((limit_kb * 1024)) | ||
| echo "- $f: [${size}/${limit} bytes]" | ||
| if [ "$size" -gt "$limit" ]; then | ||
| echo "-- size exceeded by: $((size - limit)) bytes" >&2 | ||
| rc=1 | ||
| fi | ||
| done | ||
| [ "$found" = 1 ] || { echo "missing ${kind} image for ${soc} (${pattern})" >&2; rc=1; } | ||
| } | ||
| check_kind nor 256 "output/*${soc}*-nor.bin" | ||
| exit $rc | ||
|
|
||
| - name: Upload artifact | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: u-boot-${{ matrix.soc }} | ||
| path: ${{ matrix.files }} | ||
| if-no-files-found: error | ||
|
|
||
| # Build qemu-system-arm once and share it with every smoke leg. | ||
| qemu: | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 45 | ||
| steps: | ||
| - name: Install QEMU build deps | ||
| run: | | ||
| sudo apt-get update -qq | ||
| sudo apt-get install -y -qq --no-install-recommends \ | ||
| build-essential ninja-build meson pkg-config \ | ||
| libglib2.0-dev libpixman-1-dev libfdt-dev zlib1g-dev \ | ||
| libslirp-dev python3 python3-venv | ||
|
|
||
| - name: Build qemu-system-arm | ||
| run: | | ||
| git clone --depth 1 --branch master \ | ||
| https://github.com/widgetii/qemu-hisilicon qemu-hisilicon | ||
| cd qemu-hisilicon | ||
| bash qemu/setup.sh | ||
| cp qemu-src/build/qemu-system-arm "$GITHUB_WORKSPACE/qemu-arm" | ||
| chmod +x "$GITHUB_WORKSPACE/qemu-arm" | ||
|
|
||
| - name: Upload qemu-arm | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: qemu-arm | ||
| path: qemu-arm | ||
|
|
||
| # QEMU smoke test -- boots each freshly-built image from flash through | ||
| # qemu-hisilicon's nt98562/nt98566 machines, which do the loader's job | ||
| # (they do not train DDR, so they do not run it): read the device tree | ||
| # at 0x10000, unpack U-Boot from its BCL1 container as the tree says, | ||
| # fill SHMINFO and start it; then assert it reaches the autoboot prompt. | ||
| # Both SoCs have a machine model, so both are a hard gate on release. | ||
| # | ||
| # The second boot must load the environment the first saved, which | ||
| # proves the driver's writes land where its reads find them. | ||
| qemu_smoke: | ||
| needs: [build, qemu] | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 15 | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| soc: [nt98562, nt98566] | ||
| steps: | ||
| - name: Download qemu-arm | ||
| uses: actions/download-artifact@v4 | ||
| with: | ||
| name: qemu-arm | ||
|
|
||
| - name: Download u-boot artifact | ||
| uses: actions/download-artifact@v4 | ||
| with: | ||
| name: u-boot-${{ matrix.soc }} | ||
|
|
||
| - name: Install QEMU runtime deps | ||
| run: | | ||
| # qemu-arm is built once in the `qemu` job and downloaded here, | ||
| # so this leg needs the shared libraries it links against. | ||
| sudo apt-get update -qq | ||
| sudo apt-get install -y -qq --no-install-recommends \ | ||
| libglib2.0-0 libpixman-1-0 libfdt1 zlib1g libslirp0 | ||
| if ldd ./qemu-arm | grep -qi 'not found'; then | ||
| echo "qemu-arm has unresolved libraries:" >&2 | ||
| ldd ./qemu-arm >&2 | ||
| exit 1 | ||
| fi | ||
|
|
||
| - name: Smoke-test in QEMU | ||
| timeout-minutes: 3 | ||
| run: | | ||
| soc='${{ matrix.soc }}' | ||
| UBOOT="u-boot-${soc}-nor.bin" | ||
| test -f "$UBOOT" | ||
| echo "booting $UBOOT" | ||
| # 8 MiB NOR, U-Boot at 0, the rest erased | ||
| python3 - "$UBOOT" /tmp/flash.bin <<'PY' | ||
| import sys | ||
| img = bytearray(b'\xff' * (8 << 20)) | ||
| u = open(sys.argv[1], 'rb').read() | ||
| img[:len(u)] = u | ||
| open(sys.argv[2], 'wb').write(img) | ||
| PY | ||
| FLASH="-M ${soc},flash-file=/tmp/flash.bin" | ||
| chmod +x ./qemu-arm | ||
| # Two boots: the first finds an erased environment and saves the | ||
| # default one; the second must load it back. | ||
| for boot in 1 2; do | ||
| mkfifo /tmp/uart.in /tmp/uart.out | ||
| ./qemu-arm $FLASH \ | ||
| -nographic -serial pipe:/tmp/uart \ | ||
| -monitor none > /tmp/qemu.log 2>&1 & | ||
| QEMU_PID=$! | ||
| # Wait for the autoboot prompt, not the banner: the banner prints | ||
| # before the board init that can hang. Up to 30 s. | ||
| cat /tmp/uart.out > /tmp/uboot$boot.txt & | ||
| CAT_PID=$! | ||
| for i in $(seq 1 60); do | ||
| grep -q 'Hit Ctrl-C' /tmp/uboot$boot.txt 2>/dev/null && break | ||
| sleep 0.5 | ||
| done | ||
| kill "$QEMU_PID" "$CAT_PID" 2>/dev/null || true | ||
| wait 2>/dev/null || true | ||
| rm -f /tmp/uart.in /tmp/uart.out | ||
| echo "== boot $boot: captured u-boot output ==" | ||
| cat /tmp/uboot$boot.txt | ||
| echo | ||
| if ! grep -q 'Hit Ctrl-C' /tmp/uboot$boot.txt; then | ||
| echo "FAIL: boot $boot did not reach the autoboot prompt" | ||
| echo | ||
| echo "== qemu stderr ==" | ||
| tail -40 /tmp/qemu.log || true | ||
| exit 1 | ||
| fi | ||
| done | ||
| if grep -q 'bad CRC' /tmp/uboot2.txt; then | ||
| echo "FAIL: the environment saved on the first boot did not land" | ||
| exit 1 | ||
| fi | ||
| echo "PASS: u-boot reached the autoboot prompt, and its saved environment survived a reset" | ||
|
|
||
| # Mirror every built image to this repository's own Novatek release. | ||
| release: | ||
| needs: [resolve, build, qemu_smoke] | ||
| if: ${{ github.event_name == 'workflow_dispatch' && inputs.release }} | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 15 | ||
| permissions: | ||
| contents: write | ||
| steps: | ||
| - name: Download artifacts | ||
| uses: actions/download-artifact@v4 | ||
| with: | ||
| path: artifacts | ||
| pattern: u-boot-* | ||
|
|
||
| - name: Upload to releases | ||
| uses: softprops/action-gh-release@v2 | ||
| with: | ||
| tag_name: novatek | ||
| name: Novatek | ||
| target_commitish: ${{ needs.resolve.outputs.sha }} | ||
| prerelease: true | ||
| files: artifacts/**/*.bin | ||
| fail_on_unmatched_files: true | ||
|
|
||
| # Publish every image to the OpenIPC/firmware `latest` release and tell | ||
| # openipc.org what was uploaded. openipc.org keeps, per file name, the | ||
| # newest uboot build that names it, so this run adds or refreshes the | ||
| # u-boot-nt985* names and leaves the other repositories' files alone. The | ||
| # concurrency group, shared with build-hisilicon.yml, keeps a second run | ||
| # from uploading between this run's upload and its report. | ||
| publish: | ||
| needs: [resolve, build, qemu_smoke] | ||
| if: ${{ github.event_name == 'workflow_dispatch' && inputs.publish }} | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 30 | ||
| concurrency: | ||
| group: publish-firmware-latest | ||
| cancel-in-progress: false | ||
| permissions: | ||
| contents: read | ||
| id-token: write | ||
| steps: | ||
| - name: Download artifacts | ||
| uses: actions/download-artifact@v4 | ||
| with: | ||
| path: artifacts | ||
| pattern: u-boot-* | ||
|
|
||
| - name: Upload to OpenIPC/firmware latest release | ||
| env: | ||
| GH_TOKEN: ${{ secrets.FIRMWARE_RELEASE_TOKEN }} | ||
| run: | | ||
| set -eu | ||
| # Releases API occasionally returns 502/503. Retry with backoff: | ||
| # 10s, 20s, 30s, 40s -- total ~100s before giving up. | ||
| upload() { | ||
| local f="$1" i | ||
| for i in 1 2 3 4 5; do | ||
| if gh release upload latest "$f" --clobber -R OpenIPC/firmware; then | ||
| return 0 | ||
| fi | ||
| [ "$i" = "5" ] && return 1 | ||
| echo "upload attempt $i failed; sleeping $((i*10))s" >&2 | ||
| sleep $((i*10)) | ||
| done | ||
| } | ||
| find artifacts -type f -name '*.bin' | sort | while read -r f; do | ||
| echo "publishing $f" | ||
| upload "$f" | ||
| done | ||
|
|
||
| # The push script is OpenIPC/firmware's, at a fixed commit: the only | ||
| # code this job checks out, since it holds an id-token. | ||
| - uses: actions/checkout@v4 | ||
| with: | ||
| repository: OpenIPC/firmware | ||
| ref: 7fa63ce694cb5d27c19d4f19058ba2c044849db4 | ||
| sparse-checkout: .github/scripts | ||
| path: firmware | ||
|
|
||
| # The contract is PUSH.md in OpenIPC/website (source `uboot`, release | ||
| # `latest`, one id per run). The digests are of the files uploaded | ||
| # above, hashed here, so the push describes exactly those bytes. | ||
| - name: Report to openipc.org | ||
| env: | ||
| NVT_SHA: ${{ needs.resolve.outputs.sha }} | ||
| run: | | ||
| set -eu | ||
| mkdir published | ||
| find artifacts -type f -name '*.bin' -exec cp {} published/ \; | ||
| python3 firmware/.github/scripts/push_build.py \ | ||
| --source uboot \ | ||
| --build-id "uboot-novatek-$(date -u +%Y%m%dT%H%M%SZ)-${NVT_SHA::7}" \ | ||
| --release latest \ | ||
| --sha "$NVT_SHA" \ | ||
| --built-at "$(date -u +%Y-%m-%dT%H:%M:%SZ)" \ | ||
| --assets 'published/*.bin' | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.