Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update optoe driver to add CMIS (QSFP-DD, OSFP, ...) in 201811 branch #164

Open
wants to merge 1 commit into
base: 201811
Choose a base branch
from
Open

Update optoe driver to add CMIS (QSFP-DD, OSFP, ...) in 201811 branch #164

wants to merge 1 commit into from

Conversation

mudsut4ke
Copy link

@mudsut4ke mudsut4ke commented Oct 21, 2020 •
edited
Loading

Porting back optoe driver update from : #96 to 201811 branch

  • Update to optoe driver to improve EOF handling

    Added patch: 'driver-support-optoe-EOF_fix.patch' to patch directory
    Added the patch to 'series

  • Patch to add support for CMIS (QSFP-DD, OSFP, ...) class devices
    to optoe driver. Backward compatible, still supports SFP*, QSFP*

@donboll
Update optoe driver to add CMIS (QSFP-DD, OSFP, ...) support (sonic-n…
b0df6cb 
…et#96)

* Update to optoe driver to improve EOF handling

	Added patch: 'driver-support-optoe-EOF_fix.patch' to patch directory
	Added the patch to 'series

Signed-off-by: Don Bollinger

* Patch to add support for CMIS (QSFP-DD, OSFP, ...) class devices
to optoe driver.  Backward compatible, still supports SFP*, QSFP*
@mudsut4ke mudsut4ke changed the title Update optoe driver to add CMIS (QSFP-DD, OSFP, ...) Update optoe driver to add CMIS (QSFP-DD, OSFP, ...) in 201811 branch Oct 21, 2020
@zhenggen-xu
Copy link
Contributor

What is the impact without this patch?

@mudsut4ke
Copy link
Author

@zhenggen-xu , Without this patch we will not be able to read all the data contained in the CMIS device class EEPROM (QSFP-DD, OSFP, ...).

@jarias-lfx
Copy link

/easycla

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jul 19, 2023 •
edited
Loading

CLA Not Signed

  • ❌ login: @donboll / The commit (b0df6cb). This user is authorized, but they must confirm their affiliation with their company. Start the authorization process by clicking here, click "Corporate", select the appropriate company from the list, then confirm your affiliation on the page that appears. For further assistance with EasyCLA, please submit a support request ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mudsut4ke @zhenggen-xu @jarias-lfx @donboll