UART integration - #2
Conversation
|
👋 Hello! Before merging this PR, you can use the Before running the GDS workflow:
To run the GDS workflow:
📚 For more information about the hardening command, see the TinyTapeout GDS Action. |
7e7e452 to
03f93b3
Compare
03f93b3 to
d6340de
Compare
a00edec to
1b55ab6
Compare
| - name: hack | ||
| run: touch src/utoss-risc-v/src/Instruction_Decode/RegisterFile.sv2v.v |
There was a problem hiding this comment.
need to figure out why this file does not seem to be created in ci after sv2v run
| # - name: Run tests | ||
| # run: | | ||
| # cd test | ||
| # make clean | ||
| # make | ||
| # # make will return success even if the test fails, so check for failure in the results.xml | ||
| # ! grep failure results.xml | ||
|
|
||
| # - name: Test Summary | ||
| # uses: test-summary/action@v2.4 | ||
| # with: | ||
| # paths: "test/results.xml" | ||
| # if: always() | ||
|
|
||
| # - name: upload waveform and test results | ||
| # if: success() || failure() | ||
| # uses: actions/upload-artifact@v7 | ||
| # with: | ||
| # name: test-results | ||
| # path: | | ||
| # test/tb.fst | ||
| # test/results.xml | ||
| # test/output/* |
| "//": "Attempt to fix attennas more agressively", | ||
| "GRT_ANTENNA_ITERS": 20, | ||
| "DIODE_PADDING": 20, | ||
|
|
There was a problem hiding this comment.
not 100% sure we need these, i have not tried running hardening without this after the version bump
| , .bus_write_data ( dbg_write_data ) | ||
| , .bus_write_enable ( dbg_write_enable ) | ||
| , .bus_read_data ( read_data ) | ||
| // , .dbg_regs ( dbg_regs_zeros ) |
There was a problem hiding this comment.
removed this due to heavy fanout, need to bring back
| @@ -0,0 +1,79 @@ | |||
| # Makefile for UART-specific testing | |||
There was a problem hiding this comment.
need to add this to test workflow in actions
…rtl), added additional data integrity tests
029bd63 to
2df1a6b
Compare
|
I'll merge this for now, i think this is good shape; we'll do follow ups in separate PRs. Huge thanks @marwannismail @RunzeZhu28 and @Parzival129 ! |
4 participants
No description provided.