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

[as7726-32x] Fix module_reset sysfs #7715

Merged

Conversation

jostar-yang
Copy link
Contributor

Signed-off-by: Jostar Yang jostar_yang@accton.com.tw

Why I did it

Fix modules_reset sysfs due to need to do revert.

How I did it

Add to do revert when read modules_reset sysfs

How to verify it

#cat module_reset_1
1
#echo 1 > module_reset_1
#cat module_reset_1
1
#echo 0 > module_reset_1
#cat module_reset_1
0

cat module_reset_31

0

echo 1 > module_reset_31

cat module_reset_31

1

echo 0 > module_reset_31

cat module_reset_31

0

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

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

Signed-off-by: Jostar Yang <jostar_yang@accton.com.tw>
@jostar-yang jostar-yang requested a review from lguohan as a code owner May 26, 2021 09:00
@jleveque
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lguohan lguohan merged commit f547cad into sonic-net:master Jun 8, 2021
qiluo-msft pushed a commit that referenced this pull request Jun 9, 2021
Fix modules_reset sysfs due to need to do revert.

Signed-off-by: Jostar Yang <jostar_yang@accton.com.tw>
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
Fix modules_reset sysfs due to need to do revert.

Signed-off-by: Jostar Yang <jostar_yang@accton.com.tw>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants