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

[Accton] Set pca954x idle_state to enable deselect #10011

Closed

Conversation

jostar-yang
Copy link
Contributor

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

Why I did it

Kernel5.10 pca954x drv deselect is disable, it provide user to set via idle_state to "-2".
If deselect is disable, some device will get error when pca channel is not closed.

How I did it

Input "-2" to /sys/bus/i2c/drivers/pca954x/-00/idle_state.

How to verify it

Test to get fan or psu sysfs, it will not data.
After set "-2" to idle_state, all sysf work well.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

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 February 17, 2022 07:37
@jostar-yang
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@jostar-yang
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jostar-yang
Copy link
Contributor Author

Because there is file conflict,
platform/broadcom/sonic-platform-modules-accton/as7816-64x/utils/accton_as7816_util.py
But no write access, so add this change to below PR,
#10385

@jostar-yang
Copy link
Contributor Author

This PR become invalid due to PR10053 merged as7816 code. It get file conflict, but no write access to resolve this issue. So close this PR and add as7816 change to #10385

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.

2 participants