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.

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

[Ufispace] Update the transceiver control virtual address to avoid an address conflict with SPD #19759

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

nonodark
Copy link
Contributor

@nonodark nonodark commented Aug 1, 2024

Why I did it

Update the transceiver control virtual address to avoid an address conflict with SPD

  • s6301-56st
  • s7801-54xs
  • s8901-54xc
  • s9110-32x
Work item tracking
  • Microsoft ADO (number only):

How I did it

Update the PDDF config files.

How to verify it

Run the PDDF commands and show commands to check the sysfs and transceiver status.

s6301-56st.txt
s7801-54xs.txt
s8901-54xc.txt
s9110-32x.txt

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

  • master

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Copy link

linux-foundation-easycla bot commented Aug 1, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: nonodark / name: Nonodark Huang (ce428ae)

@nonodark nonodark changed the title Update the transceiver control virtual address to avoid an address conflict with SPD [Ufispace] Update the transceiver control virtual address to avoid an address conflict with SPD Aug 1, 2024
…nflict with SPD

* s6301-56st
* s7801-54xs
* s8901-54sc
* s9110-32x
@nonodark nonodark marked this pull request as ready for review August 1, 2024 06:07
@nonodark
Copy link
Contributor Author

nonodark commented Aug 1, 2024

@cytsai0409 , please help review the change, thanks.

@nonodark
Copy link
Contributor Author

nonodark commented Aug 2, 2024

@yxieca , this PR only updates the ufispace platform PDDF config and will not impact anything else.
Please help merge it.
Thanks.

@yxieca yxieca requested a review from prgeor August 2, 2024 00:51
@yxieca
Copy link
Contributor

yxieca commented Aug 2, 2024

@prgeor can you review?

@nonodark
Copy link
Contributor Author

nonodark commented Aug 5, 2024

@prgeor , please help review the change, thanks.

@nonodark
Copy link
Contributor Author

@prgeor , please help review the change, thanks.

2 similar comments
@nonodark
Copy link
Contributor Author

@prgeor , please help review the change, thanks.

@nonodark
Copy link
Contributor Author

@prgeor , please help review the change, thanks.

@nonodark
Copy link
Contributor Author

nonodark commented Sep 2, 2024

@yxieca, it seems that there has been no feedback for a month. Could you please help me merge the changes?

1 similar comment
@nonodark
Copy link
Contributor Author

nonodark commented Sep 9, 2024

@yxieca, it seems that there has been no feedback for a month. Could you please help me merge the changes?

@yxieca
Copy link
Contributor

yxieca commented Sep 9, 2024

@prgeor please review this change. Thanks.

@nonodark
Copy link
Contributor Author

@prgeor please review this change. Thanks.

2 similar comments
@nonodark
Copy link
Contributor Author

@prgeor please review this change. Thanks.

@nonodark
Copy link
Contributor Author

@prgeor please review this change. Thanks.

@nonodark
Copy link
Contributor Author

nonodark commented Oct 7, 2024

@yxieca, it seems that there has been no feedback for two months. Could you please help me merge the changes?

@yxieca yxieca merged commit c2f37a6 into sonic-net:master Oct 8, 2024
22 checks passed
saksarav-nokia pushed a commit to saksarav-nokia/sonic-buildimage that referenced this pull request Oct 9, 2024
…nflict with SPD (sonic-net#19759)

Why I did it
Update the transceiver control virtual address to avoid an address conflict with SPD

s6301-56st
s7801-54xs
s8901-54xc
s9110-32x

How I did it
Update the PDDF config files.

How to verify it
Run the PDDF commands and show commands to check the sysfs and transceiver status.

s6301-56st.txt
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.

4 participants