Add negative case for pci_attach() and pci_reattach()

sonic-net · Oct 9, 2024 · 44a505b · 44a505b
1 parent daeed65
commit 44a505b
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/sonic_platform_base/module_base.py b/sonic_platform_base/module_base.py
@@ -264,7 +264,7 @@ def get_state_info(self):
 
     def get_bus_info(self, module_name):
         """
-        Retrieves the bus information for the specified by "module_name" on a SmartSwitch.
+        Retrieves the bus information for the module specified by "module_name" on a SmartSwitch.
 
         Returns:
             Returns the PCI bus information in BDF format like "[DDDD:]BB:SS:F"
@@ -276,6 +276,7 @@ def pci_detach(self, module_name):
         Detaches the DPU PCI device specified by "module_name" on a SmartSwitch.
 
         Returns: True once the PCI is successfully detached.
+        Returns False if PCI detachment fails or specified 'module_name' is not found.
         """
         raise NotImplementedError
 
@@ -284,6 +285,7 @@ def pci_reattach(self, module_name):
         Rescans and reconnects the DPU PCI device specified by "module_name" on a SmartSwitch.
 
         Returns: True once the PCI is successfully reconnected.
+        Returns False if the PCI scan fails or specified module_name' is not found
         """
         raise NotImplementedError