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.

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

Add loop back tests for MCTP stack #63

Merged
merged 15 commits into from
Dec 18, 2024
Merged

Add loop back tests for MCTP stack #63

merged 15 commits into from
Dec 18, 2024

Conversation

parvathib
Copy link
Collaborator

@parvathib parvathib commented Dec 17, 2024 •
edited
Loading

The PR adds the following

  1. Adds loopback tests for various message sizes: send a test message to the emulator test thread, receive the same message from the thread in loopback, and compare the results. This aims to test the message assembly and packetization logic.
  2. Refactors kernel tests for improved structure and readability.
  3. Adds couple fixes in emulator for I3C peripheral and the test threads.

Parvathi Bhogaraju added 14 commits December 12, 2024 18:21
mctp syscall driver draft
3231320
Merge branch 'main' of https://github.com/chipsalliance/caliptra-mcu-sw…
baf6ebc 
… into pbhogaraju/mctp_syscall_driver
add tx and rx message handling
2fc71d7
instantiate the syscall drivers for SPDM, PLDM and vendor defined PCI…
3bd8a55 
… message types
cleanup and add funtion headers
76fd074
mctp stack test framework draft
0a28289
refactor tests
6d19d07
loopback working
f3a5f3c
Merge branch 'main' of https://github.com/chipsalliance/caliptra-mcu-sw…
812753d 
… into pbhogaraju/mctp_syscall_driver
Address review comments
cd19f92
Merge branch 'pbhogaraju/mctp_syscall_driver' of https://github.com/c…
6ba5f3b 
…hipsalliance/caliptra-mcu-sw into pbhogaraju/mctp_mux_test
loopback test passing
4ed7d49
Merge branch 'main' of https://github.com/chipsalliance/caliptra-mcu-sw…
94fe2a2 
… into pbhogaraju/mctp_mux_test
Add more loopback tests for MCTP stack
f0e3da1
@parvathib parvathib requested review from swenson, helloxiling, mlvisaya, mhatrevi and vsonims and removed request for swenson December 17, 2024 23:29
swenson
swenson approved these changes Dec 18, 2024
View reviewed changes
Copy link
Collaborator

@swenson swenson left a comment

Choose a reason for hiding this comment

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

LGTM

@parvathib parvathib merged commit 68827e8 into main Dec 18, 2024
1 check passed
@parvathib parvathib deleted the pbhogaraju/mctp_mux_test branch December 18, 2024 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@swenson swenson swenson approved these changes

@helloxiling helloxiling Awaiting requested review from helloxiling

@mlvisaya mlvisaya Awaiting requested review from mlvisaya

@mhatrevi mhatrevi Awaiting requested review from mhatrevi

@vsonims vsonims Awaiting requested review from vsonims

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@parvathib @swenson