diff options
| author | Hao Wu <hao.a.wu@intel.com> | 2017-04-24 13:32:04 +0800 |
|---|---|---|
| committer | Hao Wu <hao.a.wu@intel.com> | 2017-04-25 09:08:33 +0800 |
| commit | 1a5ae661754625b8f4bba39214006e87216747e9 (patch) | |
| tree | c1d5e99da742f1e4d9666121bc7efd409d882e21 /MdeModulePkg | |
| parent | c36ea72ebd4c0fd2c73f881d4708fa623c20c320 (diff) | |
| download | edk2-1a5ae661754625b8f4bba39214006e87216747e9.tar.gz edk2-1a5ae661754625b8f4bba39214006e87216747e9.tar.bz2 edk2-1a5ae661754625b8f4bba39214006e87216747e9.zip | |
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>
Diffstat (limited to 'MdeModulePkg')
| -rw-r--r-- | MdeModulePkg/Bus/Pci/UfsPciHcDxe/UfsPciHcDxe.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/MdeModulePkg/Bus/Pci/UfsPciHcDxe/UfsPciHcDxe.c b/MdeModulePkg/Bus/Pci/UfsPciHcDxe/UfsPciHcDxe.c index 373e55b4c2..96cf39adc1 100644 --- a/MdeModulePkg/Bus/Pci/UfsPciHcDxe/UfsPciHcDxe.c +++ b/MdeModulePkg/Bus/Pci/UfsPciHcDxe/UfsPciHcDxe.c @@ -671,13 +671,12 @@ Done: //
// Restore original PCI attributes
//
- Status = PciIo->Attributes (
- PciIo,
- EfiPciIoAttributeOperationSet,
- Private->PciAttributes,
- NULL
- );
- ASSERT_EFI_ERROR (Status);
+ PciIo->Attributes (
+ PciIo,
+ EfiPciIoAttributeOperationSet,
+ Private->PciAttributes,
+ NULL
+ );
}
gBS->CloseProtocol (
Controller,
|