diff options
| author | AKASHI Takahiro <takahiro.akashi@linaro.org> | 2017-05-17 14:51:44 +0900 |
|---|---|---|
| committer | Simon Horman <horms@verge.net.au> | 2017-05-22 13:35:02 +0200 |
| commit | a17234fe94bce780ac36a0ba9bfc9b6e8ffd84f0 (patch) | |
| tree | bb75c1dd92cc315e186af6d13401fd21934575a1 /kexec/arch/arm64/kexec-elf-arm64.c | |
| parent | ef26cc33b8d66460c555600b0e388f55c5cd6f21 (diff) | |
arm64: change return values on error to negative
EFAILED is defined "-1" and so we don't need to negate it as a return value.
Signed-off-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
Tested-by: David Woodhouse <dwmw@amazon.co.uk>
Tested-by: Pratyush Anand <panand@redhat.com>
Signed-off-by: Simon Horman <horms@verge.net.au>
Diffstat (limited to 'kexec/arch/arm64/kexec-elf-arm64.c')
| -rw-r--r-- | kexec/arch/arm64/kexec-elf-arm64.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/kexec/arch/arm64/kexec-elf-arm64.c b/kexec/arch/arm64/kexec-elf-arm64.c index daf8bf0..2b6c127 100644 --- a/kexec/arch/arm64/kexec-elf-arm64.c +++ b/kexec/arch/arm64/kexec-elf-arm64.c @@ -48,7 +48,7 @@ int elf_arm64_load(int argc, char **argv, const char *kernel_buf, if (info->kexec_flags & KEXEC_ON_CRASH) { fprintf(stderr, "kexec: kdump not yet supported on arm64\n"); - return -EFAILED; + return EFAILED; } result = build_elf_exec_info(kernel_buf, kernel_size, &ehdr, 0); @@ -92,7 +92,7 @@ int elf_arm64_load(int argc, char **argv, const char *kernel_buf, if (i == ehdr.e_phnum) { dbgprintf("%s: Valid arm64 header not found\n", __func__); - result = -EFAILED; + result = EFAILED; goto exit; } @@ -100,7 +100,7 @@ int elf_arm64_load(int argc, char **argv, const char *kernel_buf, if (kernel_segment == ULONG_MAX) { dbgprintf("%s: Kernel segment is not allocated\n", __func__); - result = -EFAILED; + result = EFAILED; goto exit; } |