Skip to content

Commit b5cfc02

Browse files
authored
[celestica dx010] comment out the initialization of PCA9541 (#5891)
The original code tried to initialize PCA9541 without having the driver loaded. As result the initialization didn't take effect. Recently PCA9541 driver was added to the kernel and since then the initialization takes effect and has negatively impacted the platform stability. Commenting the initialization code out to restore the original behavior while analyzing further. Signed-off-by: Ying Xie <ying.xie@microsoft.com>
1 parent 3307634 commit b5cfc02

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

platform/broadcom/sonic-platform-modules-cel/debian/platform-modules-dx010.init

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,14 @@ start)
6363

6464
# Attach PCA9541 Ox70 Master Selector
6565
chmod 755 /sys/bus/i2c/devices/i2c-${devnum}/new_device
66-
echo pca9541 0x70 > /sys/bus/i2c/devices/i2c-${devnum}/new_device
66+
# FIXME: commenting out the following line.
67+
# there had been no pca9541 driver loaded on Celestica platform,
68+
# the recent addition of this driver casued following line
69+
# becoming effictive, but negatively impacted the platform
70+
# stability. Commenting it out restores the original behavior
71+
# on Celestica platform.
72+
# This change should be further analyzed and updated.
73+
# echo pca9541 0x70 > /sys/bus/i2c/devices/i2c-${devnum}/new_device
6774
sleep 1
6875

6976
# Attach PCA9548 0x71 Channel Extender for Main Board

0 commit comments

Comments
 (0)