diff options
| author | Hao Wu <hao.a.wu@intel.com> | 2017-04-24 13:32:04 +0800 |
|---|---|---|
| committer | Guo Mang <mang.guo@intel.com> | 2017-07-12 11:24:26 +0800 |
| commit | 65172a08ab202b757f8d418d8a03d4b874c09118 (patch) | |
| tree | 73412dcee2b00b8014bef240adca377ef052a3c3 /Core | |
| parent | 791d3efcace6d3b9f245aca8453f23c44e24f527 (diff) | |
| download | edk2-platforms-65172a08ab202b757f8d418d8a03d4b874c09118.tar.xz | |
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 1a5ae661754625b8f4bba39214006e87216747e9)
Diffstat (limited to 'Core')
| -rw-r--r-- | Core/MdeModulePkg/Bus/Pci/UfsPciHcDxe/UfsPciHcDxe.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/Core/MdeModulePkg/Bus/Pci/UfsPciHcDxe/UfsPciHcDxe.c b/Core/MdeModulePkg/Bus/Pci/UfsPciHcDxe/UfsPciHcDxe.c index 373e55b4c2..96cf39adc1 100644 --- a/Core/MdeModulePkg/Bus/Pci/UfsPciHcDxe/UfsPciHcDxe.c +++ b/Core/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,
|