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

[platform/broadcom] DX010: configure pca954x device with idle_state #6429

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

Conversation

pphuchar
Copy link
Contributor

@pphuchar pphuchar commented Jan 12, 2021

- Why I did it
Use the proper way to initialize the per-device configuration of the PCA954x MUX driver.

- How I did it
Configure the PCA9548 device behavior with the idle_state sysfs provides by the driver. Refer to the sonic-linux-kernel#177

- How to verify it
Boot the DX010 device and read the idle_state sysfs of each device. The result must be -2 (deselect channel on exit).

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

  • 201811
  • 201911
  • 202006
  • 202012

- Description for the changelog

dx010 use idle_state sysfs for pca9548 device configuration.

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

Test Log:
dx010-pca954x-using-idle_state-sysfs.log

@lguohan
Copy link
Collaborator

lguohan commented Jan 12, 2021

thanks, is this verified?

@pphuchar
Copy link
Contributor Author

thanks, is this verified?

I have updated the PR description with a test log.

@paulmenzel
Copy link
Contributor

Is cat: /sys/bus/i2c/drivers/pca95: No such file or directory a leftover in the test.log?

@pphuchar
Copy link
Contributor Author

Is cat: /sys/bus/i2c/drivers/pca95: No such file or directory a leftover in the test.log?

Oh, right, its a left over. I have re-submit the log.

@pphuchar pphuchar marked this pull request as ready for review January 20, 2021 08:27
@pphuchar pphuchar requested a review from lguohan as a code owner February 6, 2021 20:29
@lguohan
Copy link
Collaborator

lguohan commented Sep 23, 2023

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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