oxidecomputer / oxidecomputer/transceiver-control
Want support for more CMIS advertised pages
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 9
- Forks
- 0
- Avg merge
- 57m
- Merged PRs (30d)
- 3
Description
#26 added basic support for decoding the pages a module advertises support for. SFF-8636 modules are pretty simple here, but CMIS is a good deal more complicated. We're parsing a few of the more straightforward pages, but here is a list of some others we should add support for. That requires reading a few more bytes from the module and decoding the right bits. Not complicated, but not critical at this point.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in decode/src/lib.rs at the list of unsupported CMIS advertised pages around line 527, then review the existing CMIS and SFF-8636 page-decoding logic. Identify which additional pages and bit fields are in scope from that list, and verify that the selected pages are decoded from the module's bytes when done.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- embedded-iot, networking
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100