Skip to content

Fd readme - #18

Open
CourtneyDrant wants to merge 7 commits into
OpenPRoT:mainfrom
CourtneyDrant:FD-readme
Open

Fd readme#18
CourtneyDrant wants to merge 7 commits into
OpenPRoT:mainfrom
CourtneyDrant:FD-readme

Conversation

@CourtneyDrant

Copy link
Copy Markdown
Collaborator

Added a README for FdOps and fd_context to assist in development.

…t. Adjusted 'now' default to safer value of 0, disabling timestamps. Users will need to implement now using a counter from their environment.

@chrysh chrysh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Four wording fixes inline. Separately in another PR, four things the README currently describes are better fixed in code than documented (the now default, the unchecked descriptor count, the overloaded FdInitiatorModeError, and the T1 check sitting behind T2). I am opening a PR for those, so please leave those sentences short rather than describing the current behavior in detail.

Two small things: README.md has no trailing newline, and nothing wires it into rustdoc

Comment thread pldm-interface/src/firmware_device/README.md Outdated
Comment thread pldm-interface/src/firmware_device/README.md Outdated
Comment thread pldm-interface/src/firmware_device/README.md Outdated
Comment thread pldm-interface/src/firmware_device/fd_context.rs
CourtneyDrant and others added 4 commits September 2, 2026 13:44
Co-authored-by: Christina Quast <chrysh.ng+github@gmail.com>
Co-authored-by: Christina Quast <chrysh.ng+github@gmail.com>
Co-authored-by: Christina Quast <chrysh.ng+github@gmail.com>
Co-authored-by: Christina Quast <chrysh.ng+github@gmail.com>
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.

2 participants