From 931b7182890fb85d697d2073dc353124c4da5d28 Mon Sep 17 00:00:00 2001 From: hathach Date: Tue, 15 Nov 2022 18:10:41 +0700 Subject: [PATCH 01/10] try running HiFiPhile self-host with stm32l412nucleo --- .github/workflows/build_arm.yml | 4 +- .github/workflows/test_hardware.yml | 82 +++++++++++++++++++++++++++++ 2 files changed, 84 insertions(+), 2 deletions(-) create mode 100644 .github/workflows/test_hardware.yml diff --git a/.github/workflows/build_arm.yml b/.github/workflows/build_arm.yml index 53f73a48f..d47a82173 100644 --- a/.github/workflows/build_arm.yml +++ b/.github/workflows/build_arm.yml @@ -163,11 +163,11 @@ jobs: # - pico + pico-probe connected via USB # - pico-probe is /dev/ttyACM0 # --------------------------------------- - hw-test: + hw-pi4-test: # Limit the run to only hathach due to limited resource on RPI4 if: github.repository_owner == 'hathach' needs: build-arm - runs-on: [self-hosted, Linux, ARM64] + runs-on: [self-hosted, Linux, ARM64, rp2040] steps: - name: Clean workspace diff --git a/.github/workflows/test_hardware.yml b/.github/workflows/test_hardware.yml new file mode 100644 index 000000000..534a17e6e --- /dev/null +++ b/.github/workflows/test_hardware.yml @@ -0,0 +1,82 @@ +name: Hardware Test +on: + pull_request: + push: + release: + types: + - created + +# Hardware in the loop (HIL) +# Current self-hosted instance is running on an EPYC 7232 server hosted by HiFiPhile user +# - STM32L412 Nucleo with on-board jlink (with 1 ttyACM) + +jobs: + stm32l412nucleo-test: + runs-on: [self-hosted, Linux, X64, hifiphile] + steps: + - name: Clean workspace + run: | + echo "Cleaning up previous run" + rm -rf "${{ github.workspace }}" + mkdir -p "${{ github.workspace }}" + + - name: Checkout TinyUSB + uses: actions/checkout@v3 + + - name: Get Dependencies and Build + run: | + git submodule update --init lib/FreeRTOS-Kernel lib/lwip + python3 tools/get_dependencies.py stm32l4 + python3 tools/build_family.py stm32l4 + + - name: Pick-up elf files + run: | + mkdir stm32l412nucleo/ + find examples/ -path "*stm32l412nucleo/*.elf" -exec mv {} stm32l412nucleo/ \; + + - name: Create flash.sh + run: | + touch flash.sh + chmod +x flash.sh + echo > flash.sh 'echo halt > flash.jlink' + echo > flash.sh 'echo r >> flash.jlink' + echo > flash.sh 'echo loadfile $1 >> flash.jlink' + echo > flash.sh 'echo r >> flash.jlink' + echo > flash.sh 'echo go >> flash.jlink' + echo > flash.sh 'echo exit >> flash.jlink' + echo > flash.sh 'JLinkExe -device stm32l412kb -if swd -JTAGConf -1,-1 -speed auto -CommandFile flash.jlink' + + - name: Test cdc_dual_ports + run: | + ./flash.sh cdc_dual_ports.elf + while (! ([ -e /dev/ttyACM1 ] && [ -e /dev/ttyACM2 ])) && [ $SECONDS -le 5 ]; do :; done + test -e /dev/ttyACM1 && echo "ttyACM1 exists" + test -e /dev/ttyACM2 && echo "ttyACM2 exists" + + - name: Test cdc_msc + run: | + ./flash.sh cdc_msc.elf + readme='/media/pi/TinyUSB MSC/README.TXT' + while (! ([ -e /dev/ttyACM1 ] && [ -f "$readme" ])) && [ $SECONDS -le 5 ]; do :; done + test -e /dev/ttyACM1 && echo "ttyACM1 exists" + test -f "$readme" && echo "$readme exists" + cat "$readme" + + - name: Test dfu + run: | + ./flash.sh dfu.elf + while (! (dfu-util -l | grep "Found DFU")) && [ $SECONDS -le 5 ]; do :; done + dfu-util -d cafe -a 0 -U dfu0 + dfu-util -d cafe -a 1 -U dfu1 + grep "TinyUSB DFU! - Partition 0" dfu0 + grep "TinyUSB DFU! - Partition 1" dfu1 + + - name: Test dfu_runtime + run: | + ./flash.sh dfu_runtime.elf + while (! (dfu-util -l | grep "Found Runtime")) && [ $SECONDS -le 5 ]; do :; done + +# - name: Test hid_boot_interface +# run: | +# ./flash.sh hid_boot_interface.elf +# while (! (dfu-util -l | grep "Found Runtime")) && [ $SECONDS -le 5 ]; do :; done From d8badc948fef7ffc332cd8e99fef6b2134b00147 Mon Sep 17 00:00:00 2001 From: hathach Date: Tue, 15 Nov 2022 18:13:19 +0700 Subject: [PATCH 02/10] update yml --- .github/workflows/test_hardware.yml | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/.github/workflows/test_hardware.yml b/.github/workflows/test_hardware.yml index 534a17e6e..891f8cd2f 100644 --- a/.github/workflows/test_hardware.yml +++ b/.github/workflows/test_hardware.yml @@ -36,15 +36,15 @@ jobs: - name: Create flash.sh run: | - touch flash.sh + touch flash.sh chmod +x flash.sh - echo > flash.sh 'echo halt > flash.jlink' - echo > flash.sh 'echo r >> flash.jlink' - echo > flash.sh 'echo loadfile $1 >> flash.jlink' - echo > flash.sh 'echo r >> flash.jlink' - echo > flash.sh 'echo go >> flash.jlink' - echo > flash.sh 'echo exit >> flash.jlink' - echo > flash.sh 'JLinkExe -device stm32l412kb -if swd -JTAGConf -1,-1 -speed auto -CommandFile flash.jlink' + echo > flash.sh "echo halt > flash.jlink" + echo > flash.sh "echo r >> flash.jlink" + echo > flash.sh "echo loadfile $1 >> flash.jlink" + echo > flash.sh "echo r >> flash.jlink" + echo > flash.sh "echo go >> flash.jlink" + echo > flash.sh "echo exit >> flash.jlink" + echo > flash.sh "JLinkExe -device stm32l412kb -if swd -JTAGConf -1,-1 -speed auto -CommandFile flash.jlink" - name: Test cdc_dual_ports run: | From 468ec83c1b6a84b46031c16a3ff1684b59b19745 Mon Sep 17 00:00:00 2001 From: hathach Date: Tue, 15 Nov 2022 23:07:58 +0700 Subject: [PATCH 03/10] correct flash.sh creation --- .github/workflows/test_hardware.yml | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/.github/workflows/test_hardware.yml b/.github/workflows/test_hardware.yml index 891f8cd2f..70bcb0cb5 100644 --- a/.github/workflows/test_hardware.yml +++ b/.github/workflows/test_hardware.yml @@ -13,6 +13,7 @@ on: jobs: stm32l412nucleo-test: runs-on: [self-hosted, Linux, X64, hifiphile] + steps: - name: Clean workspace run: | @@ -36,15 +37,14 @@ jobs: - name: Create flash.sh run: | - touch flash.sh - chmod +x flash.sh echo > flash.sh "echo halt > flash.jlink" - echo > flash.sh "echo r >> flash.jlink" - echo > flash.sh "echo loadfile $1 >> flash.jlink" - echo > flash.sh "echo r >> flash.jlink" - echo > flash.sh "echo go >> flash.jlink" - echo > flash.sh "echo exit >> flash.jlink" - echo > flash.sh "JLinkExe -device stm32l412kb -if swd -JTAGConf -1,-1 -speed auto -CommandFile flash.jlink" + echo >> flash.sh "echo r >> flash.jlink" + echo >> flash.sh "echo loadfile $1 >> flash.jlink" + echo >> flash.sh "echo r >> flash.jlink" + echo >> flash.sh "echo go >> flash.jlink" + echo >> flash.sh "echo exit >> flash.jlink" + echo >> flash.sh "JLinkExe -device stm32l412kb -if swd -JTAGConf -1,-1 -speed auto -CommandFile flash.jlink" + chmod +x flash.sh - name: Test cdc_dual_ports run: | From 2fb0cea7e21c6e985bd85ba876f43785026203b3 Mon Sep 17 00:00:00 2001 From: hathach Date: Tue, 15 Nov 2022 23:37:07 +0700 Subject: [PATCH 04/10] more test --- .github/workflows/test_hardware.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/test_hardware.yml b/.github/workflows/test_hardware.yml index 70bcb0cb5..796a2a035 100644 --- a/.github/workflows/test_hardware.yml +++ b/.github/workflows/test_hardware.yml @@ -8,7 +8,7 @@ on: # Hardware in the loop (HIL) # Current self-hosted instance is running on an EPYC 7232 server hosted by HiFiPhile user -# - STM32L412 Nucleo with on-board jlink (with 1 ttyACM) +# - STM32L412 Nucleo with on-board jlink as ttyACM0 jobs: stm32l412nucleo-test: @@ -39,7 +39,7 @@ jobs: run: | echo > flash.sh "echo halt > flash.jlink" echo >> flash.sh "echo r >> flash.jlink" - echo >> flash.sh "echo loadfile $1 >> flash.jlink" + echo >> flash.sh "echo loadfile stm32l412nucleo/$1 >> flash.jlink" echo >> flash.sh "echo r >> flash.jlink" echo >> flash.sh "echo go >> flash.jlink" echo >> flash.sh "echo exit >> flash.jlink" From 2f909c284a594a7cb853029d31b624eea788a349 Mon Sep 17 00:00:00 2001 From: hathach Date: Tue, 15 Nov 2022 23:47:24 +0700 Subject: [PATCH 05/10] more flash script --- .github/workflows/test_hardware.yml | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/.github/workflows/test_hardware.yml b/.github/workflows/test_hardware.yml index 796a2a035..2d3dc1e8c 100644 --- a/.github/workflows/test_hardware.yml +++ b/.github/workflows/test_hardware.yml @@ -37,13 +37,13 @@ jobs: - name: Create flash.sh run: | - echo > flash.sh "echo halt > flash.jlink" - echo >> flash.sh "echo r >> flash.jlink" - echo >> flash.sh "echo loadfile stm32l412nucleo/$1 >> flash.jlink" - echo >> flash.sh "echo r >> flash.jlink" - echo >> flash.sh "echo go >> flash.jlink" - echo >> flash.sh "echo exit >> flash.jlink" - echo >> flash.sh "JLinkExe -device stm32l412kb -if swd -JTAGConf -1,-1 -speed auto -CommandFile flash.jlink" + echo > flash.sh 'echo halt > flash.jlink' + echo >> flash.sh 'echo r >> flash.jlink' + echo >> flash.sh 'echo loadfile stm32l412nucleo/$1 >> flash.jlink' + echo >> flash.sh 'echo r >> flash.jlink' + echo >> flash.sh 'echo go >> flash.jlink' + echo >> flash.sh 'echo exit >> flash.jlink' + echo >> flash.sh 'JLinkExe -device stm32l412kb -if swd -JTAGConf -1,-1 -speed auto -CommandFile flash.jlink' chmod +x flash.sh - name: Test cdc_dual_ports From a6d9fda5f902726bb073541a122ab93ebc976730 Mon Sep 17 00:00:00 2001 From: hathach Date: Wed, 16 Nov 2022 01:01:03 +0700 Subject: [PATCH 06/10] skip usb msc test --- .github/workflows/test_hardware.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/test_hardware.yml b/.github/workflows/test_hardware.yml index 2d3dc1e8c..f8124c496 100644 --- a/.github/workflows/test_hardware.yml +++ b/.github/workflows/test_hardware.yml @@ -53,7 +53,9 @@ jobs: test -e /dev/ttyACM1 && echo "ttyACM1 exists" test -e /dev/ttyACM2 && echo "ttyACM2 exists" + # Debian does not auto mount usb drive. skip this test for now - name: Test cdc_msc + if: false run: | ./flash.sh cdc_msc.elf readme='/media/pi/TinyUSB MSC/README.TXT' From d17eb94e848fe0bab4a26e18cd5961bd51953c65 Mon Sep 17 00:00:00 2001 From: hathach Date: Wed, 16 Nov 2022 09:36:12 +0700 Subject: [PATCH 07/10] only print flash output if failed --- .github/workflows/test_hardware.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/test_hardware.yml b/.github/workflows/test_hardware.yml index f8124c496..3ef1fdc56 100644 --- a/.github/workflows/test_hardware.yml +++ b/.github/workflows/test_hardware.yml @@ -43,7 +43,8 @@ jobs: echo >> flash.sh 'echo r >> flash.jlink' echo >> flash.sh 'echo go >> flash.jlink' echo >> flash.sh 'echo exit >> flash.jlink' - echo >> flash.sh 'JLinkExe -device stm32l412kb -if swd -JTAGConf -1,-1 -speed auto -CommandFile flash.jlink' + echo >> flash.sh 'cmdout=$(JLinkExe -device stm32l412kb -if swd -JTAGConf -1,-1 -speed auto -NoGui 1 -ExitOnError 1 -CommandFile flash.jlink)' + echo >> flash.sh 'if (( $? )) ; then echo $cmdout ; fi' chmod +x flash.sh - name: Test cdc_dual_ports From ba0f1a3be51ee35241cac4e6abd579a1b5b43c9a Mon Sep 17 00:00:00 2001 From: hathach Date: Wed, 16 Nov 2022 11:01:54 +0700 Subject: [PATCH 08/10] disable orphan board workflow and minor update to pi4 workflow --- .github/workflows/build_arm.yml | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/.github/workflows/build_arm.yml b/.github/workflows/build_arm.yml index d47a82173..369048fc6 100644 --- a/.github/workflows/build_arm.yml +++ b/.github/workflows/build_arm.yml @@ -123,9 +123,11 @@ jobs: # --------------------------------------- # Build all no-family (orphaned) boards + # disable this workflow since it is often failed randomly # --------------------------------------- build-board: runs-on: ubuntu-latest + if: false strategy: fail-fast: false matrix: @@ -163,7 +165,7 @@ jobs: # - pico + pico-probe connected via USB # - pico-probe is /dev/ttyACM0 # --------------------------------------- - hw-pi4-test: + hw-rp2040-test: # Limit the run to only hathach due to limited resource on RPI4 if: github.repository_owner == 'hathach' needs: build-arm @@ -183,9 +185,9 @@ jobs: - name: Create flash.sh run: | - touch flash.sh + echo > flash.sh 'cmdout=$(openocd -f "interface/picoprobe.cfg" -f "target/rp2040.cfg" -c "program $1 reset exit")' + echo >> flash.sh 'if (( $? )) ; then echo $cmdout ; fi' chmod +x flash.sh - echo > flash.sh 'openocd -f "interface/picoprobe.cfg" -f "target/rp2040.cfg" -c "program $1 reset exit"' - name: Test cdc_dual_ports run: | From 9b9d84177025b4b3bc0cf5ef47dc5c13b10e8f19 Mon Sep 17 00:00:00 2001 From: hathach Date: Wed, 16 Nov 2022 11:11:26 +0700 Subject: [PATCH 09/10] fix other ci warnings --- .github/workflows/build_aarch64.yml | 2 +- .github/workflows/build_msp430.yml | 2 +- .github/workflows/build_renesas.yml | 2 +- .github/workflows/build_riscv.yml | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/build_aarch64.yml b/.github/workflows/build_aarch64.yml index 18989a5e9..af26f33f2 100644 --- a/.github/workflows/build_aarch64.yml +++ b/.github/workflows/build_aarch64.yml @@ -41,7 +41,7 @@ jobs: run: echo >> $GITHUB_ENV TOOLCHAIN_URL=https://developer.arm.com/-/media/Files/downloads/gnu-a/10.3-2021.07/binrel/gcc-arm-10.3-2021.07-x86_64-aarch64-none-elf.tar.xz - name: Cache Toolchain - uses: actions/cache@v2 + uses: actions/cache@v3 id: cache-toolchain with: path: ~/cache/ diff --git a/.github/workflows/build_msp430.yml b/.github/workflows/build_msp430.yml index b43d9e740..3a740a28a 100644 --- a/.github/workflows/build_msp430.yml +++ b/.github/workflows/build_msp430.yml @@ -36,7 +36,7 @@ jobs: run: echo >> $GITHUB_ENV TOOLCHAIN_URL=http://software-dl.ti.com/msp430/msp430_public_sw/mcu/msp430/MSPGCC/9_2_0_0/export/msp430-gcc-9.2.0.50_linux64.tar.bz2 - name: Cache Toolchain - uses: actions/cache@v2 + uses: actions/cache@v3 id: cache-toolchain with: path: ~/cache/ diff --git a/.github/workflows/build_renesas.yml b/.github/workflows/build_renesas.yml index d212708f7..2e272c832 100644 --- a/.github/workflows/build_renesas.yml +++ b/.github/workflows/build_renesas.yml @@ -38,7 +38,7 @@ jobs: run: echo >> $GITHUB_ENV TOOLCHAIN_URL=http://gcc-renesas.com/downloads/get.php?f=rx/8.3.0.202004-gnurx/gcc-8.3.0.202004-GNURX-ELF.run - name: Cache Toolchain - uses: actions/cache@v2 + uses: actions/cache@v3 id: cache-toolchain with: path: ~/cache/ diff --git a/.github/workflows/build_riscv.yml b/.github/workflows/build_riscv.yml index 96b8c676b..61c1c60c5 100644 --- a/.github/workflows/build_riscv.yml +++ b/.github/workflows/build_riscv.yml @@ -39,7 +39,7 @@ jobs: run: echo >> $GITHUB_ENV TOOLCHAIN_URL=https://github.com/xpack-dev-tools/riscv-none-embed-gcc-xpack/releases/download/v10.1.0-1.1/xpack-riscv-none-embed-gcc-10.1.0-1.1-linux-x64.tar.gz - name: Cache Toolchain - uses: actions/cache@v2 + uses: actions/cache@v3 id: cache-toolchain with: path: ~/cache/ From fc523733c188b3afdac772e525e7309358a233d9 Mon Sep 17 00:00:00 2001 From: hathach Date: Wed, 16 Nov 2022 11:16:40 +0700 Subject: [PATCH 10/10] more ci warnings --- .github/workflows/build_msp430.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/build_msp430.yml b/.github/workflows/build_msp430.yml index 3a740a28a..1ef25a6a0 100644 --- a/.github/workflows/build_msp430.yml +++ b/.github/workflows/build_msp430.yml @@ -16,9 +16,12 @@ jobs: family: # Alphabetical order - 'msp430' + steps: - name: Setup Python uses: actions/setup-python@v4 + with: + python-version: '3.x' - name: Checkout TinyUSB uses: actions/checkout@v3