* [PATCH] EFI loader: remove pointless "if" statement
@ 2016-11-02 15:03 ` Eugene Korenevsky
0 siblings, 0 replies; 2+ messages in thread
From: Eugene Korenevsky @ 2016-11-02 15:03 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-efi-u79uwXL29TY76Z2rM5mHXA
Cc: Matt Fleming, Linn Crosetto, Ard Biesheuvel, H. Peter Anvin,
Thomas Gleixner, Ingo Molnar
An obvious cleanup. "if" statement does nothing and should be removed.
---
arch/x86/boot/compressed/eboot.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/arch/x86/boot/compressed/eboot.c b/arch/x86/boot/compressed/eboot.c
index cc69e37..202f2aa 100644
--- a/arch/x86/boot/compressed/eboot.c
+++ b/arch/x86/boot/compressed/eboot.c
@@ -1054,11 +1054,7 @@ static efi_status_t exit_boot(struct boot_params *boot_params,
/* Historic? */
boot_params->alt_mem_k = 32 * 1024;
- status = setup_e820(boot_params, e820ext, e820ext_size);
- if (status != EFI_SUCCESS)
- return status;
-
- return EFI_SUCCESS;
+ return setup_e820(boot_params, e820ext, e820ext_size);
}
/*
--
2.10.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH] EFI loader: remove pointless "if" statement
@ 2016-11-02 15:03 ` Eugene Korenevsky
0 siblings, 0 replies; 2+ messages in thread
From: Eugene Korenevsky @ 2016-11-02 15:03 UTC (permalink / raw)
To: linux-kernel, linux-efi
Cc: Matt Fleming, Linn Crosetto, Ard Biesheuvel, H. Peter Anvin,
Thomas Gleixner, Ingo Molnar
An obvious cleanup. "if" statement does nothing and should be removed.
---
arch/x86/boot/compressed/eboot.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/arch/x86/boot/compressed/eboot.c b/arch/x86/boot/compressed/eboot.c
index cc69e37..202f2aa 100644
--- a/arch/x86/boot/compressed/eboot.c
+++ b/arch/x86/boot/compressed/eboot.c
@@ -1054,11 +1054,7 @@ static efi_status_t exit_boot(struct boot_params *boot_params,
/* Historic? */
boot_params->alt_mem_k = 32 * 1024;
- status = setup_e820(boot_params, e820ext, e820ext_size);
- if (status != EFI_SUCCESS)
- return status;
-
- return EFI_SUCCESS;
+ return setup_e820(boot_params, e820ext, e820ext_size);
}
/*
--
2.10.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-11-02 15:03 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-02 15:03 [PATCH] EFI loader: remove pointless "if" statement Eugene Korenevsky
2016-11-02 15:03 ` Eugene Korenevsky
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.