diff --git a/.github/workflows/xls-modules-dma.yml b/.github/workflows/xls-modules-dma.yml index da6ecaa31f..f341586896 100644 --- a/.github/workflows/xls-modules-dma.yml +++ b/.github/workflows/xls-modules-dma.yml @@ -33,7 +33,7 @@ env: CACHE_RESTORE_KEY_IMPL: bazel-cache-dma-impl jobs: - build: + # build: name: BUILD runs-on: ubuntu-22.04 timeout-minutes: 600 @@ -82,67 +82,67 @@ jobs: //xls/tools:codegen_main \ //xls/dslx:dslx_fmt - test: - needs: build - name: Test - runs-on: ubuntu-22.04 - timeout-minutes: 600 - strategy: - fail-fast: false - matrix: - dslx_test: ["test_common", - "test_csr", - "test_axi_csr", - "test_address_generator", - "test_frontend_writer", - "test_frontend_reader", - "test_main_controller" - ] - steps: - - uses: actions/checkout@v4 - - - name: Bazel Cache - uses: actions/cache@v4 - with: - path: "~/.cache/bazel" - key: ${{env.CACHE_KEY}} - restore-keys: ${{env.CACHE_RESTORE_KEY}} - - - name: Test - run: | - bazel run -c opt --test_output=errors -- ${{env.XLS_MODULE}}:${{ matrix.dslx_test }} - - format: - name: Format - runs-on: ubuntu-22.04 - timeout-minutes: 600 - steps: - - uses: actions/checkout@v4 - - - name: Bazel Cache - uses: actions/cache@v4 - with: - path: "~/.cache/bazel" - key: ${{env.CACHE_KEY}} - restore-keys: ${{env.CACHE_RESTORE_KEY}} - - # Once https://github.com/google/xls/issues/1285 is implemented, - # we could replace these with a single rule - - name: Test formatting - run: | - bazel run -c opt --test_output=errors -- \ - //xls/modules/dma:fmt_address_generator \ - //xls/modules/dma:fmt_axi_csr \ - //xls/modules/dma:fmt_common \ - //xls/modules/dma:fmt_config \ - //xls/modules/dma:fmt_csr \ - //xls/modules/dma:fmt_fifo \ - //xls/modules/dma:fmt_frontend_reader \ - //xls/modules/dma:fmt_frontend_writer \ - //xls/modules/dma:fmt_gpf \ - //xls/modules/dma:fmt_main_controller \ - //xls/modules/dma:fmt_bus_axi_pkg \ - //xls/modules/dma:fmt_bus_axi_st_pkg + # test: + # needs: build + # name: Test + # runs-on: ubuntu-22.04 + # timeout-minutes: 600 + # strategy: + # fail-fast: false + # matrix: + # dslx_test: ["test_common", + # "test_csr", + # "test_axi_csr", + # "test_address_generator", + # "test_frontend_writer", + # "test_frontend_reader", + # "test_main_controller" + # ] + # steps: + # - uses: actions/checkout@v4 + + # - name: Bazel Cache + # uses: actions/cache@v4 + # with: + # path: "~/.cache/bazel" + # key: ${{env.CACHE_KEY}} + # restore-keys: ${{env.CACHE_RESTORE_KEY}} + + # - name: Test + # run: | + # bazel run -c opt --test_output=errors -- ${{env.XLS_MODULE}}:${{ matrix.dslx_test }} + + # format: + # name: Format + # runs-on: ubuntu-22.04 + # timeout-minutes: 600 + # steps: + # - uses: actions/checkout@v4 + + # - name: Bazel Cache + # uses: actions/cache@v4 + # with: + # path: "~/.cache/bazel" + # key: ${{env.CACHE_KEY}} + # restore-keys: ${{env.CACHE_RESTORE_KEY}} + + # # Once https://github.com/google/xls/issues/1285 is implemented, + # # we could replace these with a single rule + # - name: Test formatting + # run: | + # bazel run -c opt --test_output=errors -- \ + # //xls/modules/dma:fmt_address_generator \ + # //xls/modules/dma:fmt_axi_csr \ + # //xls/modules/dma:fmt_common \ + # //xls/modules/dma:fmt_config \ + # //xls/modules/dma:fmt_csr \ + # //xls/modules/dma:fmt_fifo \ + # //xls/modules/dma:fmt_frontend_reader \ + # //xls/modules/dma:fmt_frontend_writer \ + # //xls/modules/dma:fmt_gpf \ + # //xls/modules/dma:fmt_main_controller \ + # //xls/modules/dma:fmt_bus_axi_pkg \ + # //xls/modules/dma:fmt_bus_axi_st_pkg config-matrix: @@ -159,7 +159,7 @@ jobs: run: | sudo apt-get update sudo apt-get -qqy --no-install-recommends install jq - echo "json_rules=$(jq -rc .${{ env.XLS_MODULE_NAME }}.rules .github/workflows/xls-modules-${{ env.XLS_MODULE_NAME }}.json)" | tee -a "$GITHUB_ENV" + echo "json_rules=$(jq -rc 'del(.dma.name)|.dma' .github/workflows/xls-modules-${{ env.XLS_MODULE_NAME }}.json)" | tee -a "$GITHUB_ENV" implement: needs: config-matrix @@ -194,19 +194,19 @@ jobs: - name: IR run: | - bazel run -c opt ${{ env.BAZEL_RESOURCES_OPT }} -- ${{ env.XLS_MODULE }}:${{ matrix.ir }} + bazel run -c opt ${{ env.BAZEL_RESOURCES_OPT }} -- ${{ env.XLS_MODULE }}:${{ matrix.rules.ir }} - name: Verilog run: | - bazel build -c opt ${{ env.BAZEL_RESOURCES_OPT }} -- ${{ env.XLS_MODULE }}:${{ matrix.verilog }} + bazel build -c opt ${{ env.BAZEL_RESOURCES_OPT }} -- ${{ env.XLS_MODULE }}:${{ matrix.rules.verilog }} - name: Synthesis run: | - bazel run -c opt ${{ env.BAZEL_RESOURCES_OPT }} -- ${{ env.XLS_MODULE }}:${{ matrix.synthesis }} + bazel run -c opt ${{ env.BAZEL_RESOURCES_OPT }} -- ${{ env.XLS_MODULE }}:${{ matrix.rules.synthesis }} - name: P&R run: | - bazel build -c opt ${{ env.BAZEL_RESOURCES_OPT }} -- ${{ env.XLS_MODULE }}:${{ matrix.pnr }} + bazel build -c opt ${{ env.BAZEL_RESOURCES_OPT }} -- ${{ env.XLS_MODULE }}:${{ matrix.rules.pnr }} # ${variable/character_to_replace/new_character} # ${variable/ slash / underscore }