Skip to content

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

Closed
pphuchar wants to merge 1 commit into
sonic-net:masterfrom
pphuchar:dx010-use-new-pca9548-idle_state-opt
Closed

pphuchar wants to merge 1 commit into
sonic-net:masterfrom
pphuchar:dx010-use-new-pca9548-idle_state-opt

Conversation

@pphuchar

@pphuchar pphuchar commented Jan 12, 2021 •

Copy link
Copy Markdown
Contributor

- 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

lguohan commented Jan 12, 2021

Copy link
Copy Markdown
Collaborator

thanks, is this verified?

@pphuchar

Copy link
Copy Markdown
Contributor Author

thanks, is this verified?

I have updated the PR description with a test log.

@paulmenzel

Copy link
Copy Markdown
Contributor

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

@pphuchar

Copy link
Copy Markdown
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

lguohan commented Sep 23, 2023

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
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