MdeModulePkg/UfsPciHc: Avoid overriding return value in BindingStart

In function UfsHcDriverBindingStart(), the return value 'Status' may be
overridden during the original PCI attributes restore process.

This commit refines the logic to avoid such override.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hao Wu <hao.a.wu@intel.com>
Reviewed-by: Star Zeng <star.zeng@intel.com>
(cherry picked from commit 1a5ae66175)

MdeModulePkg/Bus/Pci/UfsPciHcDxe/UfsPciHcDxe.c

@@ -671,13 +671,12 @@ Done:
       //
       // Restore original PCI attributes
       //
-      Status = PciIo->Attributes (
+      PciIo->Attributes (
-                        PciIo,
+               PciIo,
-                        EfiPciIoAttributeOperationSet,
+               EfiPciIoAttributeOperationSet,
-                        Private->PciAttributes,
+               Private->PciAttributes,
-                        NULL
+               NULL
-                        );
+               );
-      ASSERT_EFI_ERROR (Status);
     }
     gBS->CloseProtocol (
           Controller,