Skip to content

Conversation

@spargaon
Copy link
Collaborator

Enhance the mctpRaw subcommand with options to read request data from a file (--file-in), write response data to a file (--file-out), and append or validate checksum (--checksum). These additions improve usability for large payloads and ensure data integrity during raw MCTP transactions.

Contents of the file specified in option --file-in are appended to the contents of the option --data.

@spargaon
Copy link
Collaborator Author

spargaon commented Jan 7, 2026

$ pldmtool mctpraw -e 1 -m 15 --data 0x00 0x80 0x0a 0x00
pldmtool: Tx: 00 80 0a 00
pldmtool: Rx: 00 0a 00 ff 02 01 10 00 06 30 01 0b 01 11 00 06 30 01 0b

root@congo-0234:/tmp# rm outf
root@congo-0234:/tmp# ./pldmtool mctpraw -m 17 -e 1 --data 0x7F 0x00 0x00 0x0E 0x78 0x80 0x01 -o outf
root@congo-0234:/tmp# ls -l outf
-rw-r--r-- 1 root root 8192 Jan 7 15:15 outf

root@congo-0234:/tmp# rm outf
root@congo-0234:/tmp# ./pldmtool mctpraw -m 17 -e 1 --data 0x7F 0x00 0x00 0x0E 0x78 0x80 0x01 -o outf -c
root@congo-0234:/tmp# ls -l outf
-rw-r--r-- 1 root root 8192 Jan 7 15:16 outf

@spargaon spargaon force-pushed the fwdeve125145_review_28 branch from e59f678 to f24a061 Compare January 7, 2026 17:59
@spargaon
Copy link
Collaborator Author

spargaon commented Jan 7, 2026

@supven01 @srgovard , would appreciate any review comments you may have

Enhance the `mctpRaw` subcommand with options to read request
data from a file (`--file-in`), write response data to a file
(`--file-out`), and append or validate checksum (`--checksum`).
These additions improve usability for large payloads and
ensure data integrity during raw MCTP transactions.

Contents of the file specified in option --file-in are appended
to the contents of the option --data.

Signed-off-by: Shirish Pargaonkar <Shirish.Pargaonkar@amd.com>
@spargaon spargaon force-pushed the fwdeve125145_review_28 branch from f24a061 to 518c68f Compare January 7, 2026 20:35
@spargaon spargaon requested a review from mahkurap January 7, 2026 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants