Skip to content

Doc processor #193

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 39 commits into
base: main
Choose a base branch
from
Draft

Doc processor #193

wants to merge 39 commits into from

Conversation

IstvanZsSzekely
Copy link
Collaborator

PR Description

Added packet processor sandbox testbench documentation.
Review with #179

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)
  • Documentation (change that adds or modifies documentation)

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
infrastructure refactorization
f1e88cc 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
infrastructure refactorization: Fixes
e0e4835 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
infrastructure refactorization: Refactoring
836fbe3 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
infrastructure refactorization: Generalized scoreboard
c99cfd7 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
infrastructure refactorization: Updated DMA loopback project
a463503 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
infrastructure refactorization: Fixed address value in DMA loopback f…
fb7eadf 
…or DDR

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
infrastructure refactorization: Updated IP level testbenches
5ee6bb3 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
infrastructure refactorization: IP level updates
29b8873 
- Fixed SPI engine environment
- Updated environment creation and macros

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
infrastructure refactorization: Updated Project level testbenches
53c1327 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
infrastructure refactorization: Updates and fixes
d2aa755 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
ad7616: Fixes
1670e8e 
- Fixed the serial test
- Parallel data acquisition test still failing

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
dma_flock: Fixed testbench after rebase
269401b 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
adi_datatypes: Added FIFO and LIFO class implementation
41dfa6f 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
infrastructure refactorization: Minor fixes
2991e44 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
util_axis_fifo_asym: Initial testbench commit
1fe651d 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
util_axis_fifo_asym: Updated after rebase
fc60809 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
util_axis_fifo_asym: Updates and fixes
2693a75 
- Extended the ADI agent with additional functions
- Fixes issues with the AXIS VIP sequencer
- Updated the testbench

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
util_axis_fifo_asym: Updated waveform configuration
98691b5 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
util_axis_fifo: Initial testbench commit
2f11dc9 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
util_axis_fifo: Updated ADI AXI and AXIS agents
76665e5 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
util_axis_fifo: Fixed ADI AXI agent run call
30018ba 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
General updates:
8ee0dee 
- Added/Updated licensce headers
- Updated testbench depndencies
- Updated class hierarchy

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
SPI VIP update
a5337ee 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
General updates: Updated VIP clocking and reset calls
a888f41 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
general: Change while(1) calls to forever
46b53bb 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
ADI AMD agent abstractization
afcd1be 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
ADI AMD agent abstractization: Added linker macro to simplify VIP lin…
e87f281 
…king

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
ADI AMD agent abstractication: Updated testbenches for the base envir…
a6483d7 
…onment

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
library/vip/amd/axi/s_axi_sequencer: Added memory access functions
a8c9d2d 
- Updated testbenches as well

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
library/vip/amd/axi/s_axi_sequencer: Cosmetic change
bf9ae58 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
library/vip/amd: Updated ADI agent function calls to fix a null-objec…
85d89c1 
…t access

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
scoreboard: Simplified to AXIS datastream verification
13863e7 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
scoreboard: Commented test program
1961396 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
packet_filter: Initial commit
bffecca 
- Added packet filter base class
- Updated publisher with the filter class

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
packet_filter: Add dependencies
baf31fd 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
packet_filter: Add sandbox testbench
a878252 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
packet_processor: Initial commit
920634f 
- Created the base class and added it to the publisher class
- Created a sandbox testbench for the packet processor
- Updated the main library files to use the ADI FIFOs due to language limitations
- Added a separate include set for publisher-subscriber, packet filter and processor classes

Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely
docs: Packet processor testbench
d869b1b 
Signed-off-by: Istvan-Zsolt Szekely <[email protected]>
@IstvanZsSzekely IstvanZsSzekely added the documentation Improvements or additions to documentation label Mar 3, 2025
@IstvanZsSzekely IstvanZsSzekely self-assigned this Mar 3, 2025
StancaPop
StancaPop requested changes Mar 18, 2025
View reviewed changes
docs/testbenches/ip_based/packet_processor/index.rst Outdated
Packet processing testing
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

* Processing classes are implemented which are application specific
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not clear. What do you mean? The processing classes (which are application specific) are implemented?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated description

@IstvanZsSzekely IstvanZsSzekely requested review from a team as code owners March 18, 2025 13:38
@IstvanZsSzekely IstvanZsSzekely marked this pull request as draft March 18, 2025 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@StancaPop StancaPop StancaPop requested changes

Requested changes must be addressed to merge this pull request.

Assignees

@IstvanZsSzekely IstvanZsSzekely

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@IstvanZsSzekely @StancaPop