increase bias - #6054
increase bias#6054hmod2001 wants to merge 2 commits into
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request updates the validation logic for the ZRP laser bias current test to accommodate higher current ranges specified by the CMIS standard. The previous hardcoded limit was insufficient for modules using x4 scaling, leading to false failures for valid telemetry data. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request updates the upper limit validation for the laser bias value from 131.0 to 524.28 in the transceiver test. Feedback points out that the error message still references the old limit of 131 and needs to be updated to match the new validation logic.
|
/fptest virtual |
Update TRANSCEIVER-9_ZRP laser bias current validation to allow the full CMIS-scaled range for modules using the x4 laser-bias-current multiplier.
The failing optic reported 219.9 mA, which is valid for the observed CMIS multiplier. The previous test limit of 131 mA only covered the base 2 uA LSB range and falsely failed valid telemetry.
Evidence
CMIS laser bias current encoding uses a 16-bit unsigned value with an LSB of 2 uA multiplied by the scale factor from Page 01h, Byte 160.
https://www.oiforum.com/wp-content/uploads/CMIS3p0_Third_Party_Spec.pdf
Observed module data:
Page 01h Byte 160 = 0x17
bits [4:3] = 10b
10b => x4 multiplier
Maximum valid value for x4 multiplier:
65535 * 2 uA * 4 = 524280 uA
524280 uA = 524.28 mA
Pass logs: https://firex-north.cisco.com/auto/firex-logs-ott-2/b4-featureprofiles/FireX-mananpat-260913-154059-26183/tests_logs/TRANSCEIVER_9_B4CE55F8/