Fix lgtm

Signed-off-by: maipbui <maibui@microsoft.com>
sonic-net · Aug 29, 2022 · 3702046 · 3702046
1 parent a455cae
commit 3702046
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/device/marvell/arm64-marvell_db98cx8580_16cd-r0/plugins/sfputil.py b/device/marvell/arm64-marvell_db98cx8580_16cd-r0/plugins/sfputil.py
@@ -29,7 +29,7 @@ class SfpUtil(SfpUtilBase):
     def __init__(self):
         subprocess.run(split("modprobe i2c-dev"))
         if not os.path.exists("/sys/bus/i2c/devices/0-0050"):
-            subprocess.run(split("echo optoe2 0x50 > /sys/bus/i2c/devices/i2c-0/new_device")
+            subprocess.run(split("echo optoe2 0x50 > /sys/bus/i2c/devices/i2c-0/new_device"))
 
         eeprom_path = '/sys/bus/i2c/devices/0-0050/eeprom'
         # for x in range(self.port _start, self.port_end +1):

diff --git a/device/marvell/armhf-marvell_et6448m_52x-r0/plugins/sfputil.py b/device/marvell/armhf-marvell_et6448m_52x-r0/plugins/sfputil.py
@@ -63,7 +63,7 @@ def __init__(self):
             #print port_eeprom_path
             if not os.path.exists(port_eeprom_path):
                 bus_dev_path = bus_path.format(self.port_to_i2c_mapping[x])
-                subprocess.run("echo optoe2 0x50 > " + bus_dev_path + "/new_device"))
+                subprocess.run(split("echo optoe2 0x50 > " + bus_dev_path + "/new_device"))
             self.port_to_eeprom_mapping[x] = port_eeprom_path
             self._port_to_eeprom_mapping[x] = port_eeprom_path
         SfpUtilBase.__init__(self)