Hubris MCTP driver #4
Closed
+2,328
−307
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1. Build the firmware image:
```
$ cargo xtask dist app/ast1060-starter/app.toml
$ gen_uart_booting_image.sh target/ast1060-starter/dist/default/final.bin \
target/ast1060-starter/dist/default/uart_final.bin
```
2. Using uart_final.bin for boot from UART.
3. Using humility over jtag to verify
```
$ humility -e env.json -t ast1060 tasks
humility: attached via JLink
system time = 114057
ID TASK GEN PRI STATE
0 jefe 0 0 recv, notif: fault timer(T+43)
1 idle 0 5 RUNNING
$ humility -e env.json -t ast1060 stackmargin
humility: attached via JLink
ID TASK STACKBASE STACKSIZE MAXDEPTH MARGIN
0 jefe 0x21000 1536 272 1264
1 idle 0x20c00 256 104 152
```
Note: the humility has some bug in gdb connection.
TODO:
- Adding Secure Boot Header within final.bin
- JTAG halt at main configured by features
Signed-off-by: Troy Lee <[email protected]>
Signed-off-by: Troy Lee <[email protected]>
Initial support of ast1060
Collaborator
|
@embediver , this needs to go into the OpenPRoT/hubris repo. |
rusty1968
reviewed
Sep 12, 2025
| @@ -0,0 +1,73 @@ | |||
| // MCTP-Router IPC interface | |||
Collaborator
There was a problem hiding this comment.
We must add the license information here. We are not changing the kernel code, perhaps this could be the APL2.0
rusty1968
reviewed
Sep 12, 2025
| @@ -0,0 +1,22 @@ | |||
| [package] | |||
Collaborator
There was a problem hiding this comment.
Same license comment goes to all the new files. Perhaps this is not top priority right now.
* Implement send and recv(wip) * Handling of mctp recv calls * Implement update and timeout functionality * Update mctp-lib
|
|
* add mctp serial transport binding Signed-off-by: leongross <[email protected]> * wip * use ast1060 uart lib for uart Signed-off-by: leongross <[email protected]> --------- Signed-off-by: leongross <[email protected]> Signed-off-by: Marvin Gudel <[email protected]> Co-authored-by: leongross <[email protected]>
Collaborator
Author
|
Changed it to ready, since the implementation is complete at this point. Still some testing to do and some minor cleanups like the license headers. |
rusty1968
reviewed
Sep 29, 2025
| let mut recv_buf = [0; 255]; | ||
|
|
||
| loop { | ||
| let (_, _, msg, mut resp) = listener.recv(&mut recv_buf).unwrap_lite(); |
Collaborator
There was a problem hiding this comment.
The lifetime of the response channel is tied to the recv call. When integrating with spdm-lib we have a deferred processing flow.
|
/easycla |
…quests/listeners Responses now only have a bound on the MCTP stack, making them independend from the originating listener. Missing drop routines for `request` and `response` handles have been added, freeing the resources in the MCTP driver. Signed-off-by: Marvin Gudel <[email protected]>
Signed-off-by: Marvin Gudel <[email protected]>
Signed-off-by: Marvin Gudel <[email protected]>
- Fix multiple bugs related to ipc and uart - Fix sending of responses and cleanup Signed-off-by: Marvin Gudel <[email protected]>
Collaborator
Author
|
The listener/response echo example is now working and we are writing up a little guide on how to run this. |
Signed-off-by: Marvin Gudel <[email protected]>
Signed-off-by: Marvin Gudel <[email protected]>
Signed-off-by: Marvin Gudel <[email protected]>
Signed-off-by: Marvin Gudel <[email protected]>
|
This PR has been obsoleted with #11 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implements the following model for a MCTP Driver:
The core MCTP stack is still missing, but the scaffold and IDOL definitions are already implemented.