MdeModulePkg: Remove superfluous return statements

If the code eventually returns "Status" anyway, it does not make
sense to explicitly return "Status" in case of an error, too.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Star Zeng <star.zeng@intel.com>
This commit is contained in:
Thomas Huth 2017-01-19 18:16:04 +08:00 committed by Star Zeng
parent 4d580428a3
commit cd5b2ae437
4 changed files with 0 additions and 12 deletions

View File

@ -1046,9 +1046,6 @@ NvmeControllerInit (
// One for blocking I/O, one for non-blocking I/O. // One for blocking I/O, one for non-blocking I/O.
// //
Status = NvmeCreateIoSubmissionQueue (Private); Status = NvmeCreateIoSubmissionQueue (Private);
if (EFI_ERROR(Status)) {
return Status;
}
return Status; return Status;
} }

View File

@ -783,9 +783,6 @@ EmmcSwitchToHighSpeed (
HsTiming = 1; HsTiming = 1;
Status = EmmcSwitchClockFreq (PciIo, PassThru, Slot, Rca, HsTiming, ClockFreq); Status = EmmcSwitchClockFreq (PciIo, PassThru, Slot, Rca, HsTiming, ClockFreq);
if (EFI_ERROR (Status)) {
return Status;
}
return Status; return Status;
} }

View File

@ -2520,9 +2520,6 @@ EmmcPeimSwitchToHighSpeed (
HsTiming = 1; HsTiming = 1;
Status = EmmcPeimSwitchClockFreq (Slot, Rca, HsTiming, ClockFreq); Status = EmmcPeimSwitchClockFreq (Slot, Rca, HsTiming, ClockFreq);
if (EFI_ERROR (Status)) {
return Status;
}
return Status; return Status;
} }

View File

@ -1465,9 +1465,6 @@ ConSplitterStdErrDriverBindingStart (
// //
Status = ConSplitterTextOutAddDevice (&mStdErr, TextOut, NULL, NULL); Status = ConSplitterTextOutAddDevice (&mStdErr, TextOut, NULL, NULL);
ConSplitterTextOutSetAttribute (&mStdErr.TextOut, EFI_TEXT_ATTR (EFI_MAGENTA, EFI_BLACK)); ConSplitterTextOutSetAttribute (&mStdErr.TextOut, EFI_TEXT_ATTR (EFI_MAGENTA, EFI_BLACK));
if (EFI_ERROR (Status)) {
return Status;
}
return Status; return Status;
} }