Skip to content
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

docs: Add ip_based i3c_controller testbench #155

Merged
merged 5 commits into from
Jan 14, 2025
Merged

Conversation

gastmaier
Copy link
Contributor

PR Description

Merge after #154.
Considering main testbench, not the (future) rework.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • New test (change that adds new test program and/or testbench)
  • Breaking change (has dependencies in other repositories/testbenches)

PR Checklist

  • I have followed the code style guidelines
  • I have performed a self-review of changes
  • I have ran all testbenches affected by this PR
  • I have commented my code, at least hard-to-understand parts
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe files, Copyright etc)
  • I have not introduced new Warnings/Errors on compilation/elaboration/simulation
  • I have set the verbosity level to none for the test program

@IstvanZsSzekely IstvanZsSzekely added the documentation Improvements or additions to documentation label Jan 10, 2025
@gastmaier
Copy link
Contributor Author

V1:

  • Fix preposition
  • uppercase ID
  • Remove watchdog
  • HDL dependencies

Force push to rebase on main

@gastmaier gastmaier changed the base branch from doc-dma-flock to main January 13, 2025 15:19
@gastmaier
Copy link
Contributor Author

V2:

  • testbench misspell

  • rebased

@gastmaier gastmaier merged commit 3fa7a7c into main Jan 14, 2025
2 checks passed
@gastmaier gastmaier deleted the doc-i3c-controller branch January 14, 2025 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants