From b5cfc0255261086e3c41a0e5cf65e96e3c535593 Mon Sep 17 00:00:00 2001 From: Ying Xie Date: Tue, 10 Nov 2020 16:53:27 -0800 Subject: [PATCH] [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 --- .../debian/platform-modules-dx010.init | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/platform/broadcom/sonic-platform-modules-cel/debian/platform-modules-dx010.init b/platform/broadcom/sonic-platform-modules-cel/debian/platform-modules-dx010.init index 1e124565a41d..b1fb3c60fd61 100644 --- a/platform/broadcom/sonic-platform-modules-cel/debian/platform-modules-dx010.init +++ b/platform/broadcom/sonic-platform-modules-cel/debian/platform-modules-dx010.init @@ -63,7 +63,14 @@ start) # Attach PCA9541 Ox70 Master Selector chmod 755 /sys/bus/i2c/devices/i2c-${devnum}/new_device - echo pca9541 0x70 > /sys/bus/i2c/devices/i2c-${devnum}/new_device + # FIXME: commenting out the following line. + # there had been no pca9541 driver loaded on Celestica platform, + # the recent addition of this driver casued following line + # becoming effictive, but negatively impacted the platform + # stability. Commenting it out restores the original behavior + # on Celestica platform. + # This change should be further analyzed and updated. + # echo pca9541 0x70 > /sys/bus/i2c/devices/i2c-${devnum}/new_device sleep 1 # Attach PCA9548 0x71 Channel Extender for Main Board