diff options
| author | Hui Wang <hui.wang@canonical.com> | 2026-06-06 20:17:54 -0600 |
|---|---|---|
| committer | Paul Walmsley <pjw@kernel.org> | 2026-06-06 23:48:14 -0600 |
| commit | edccaf7be062f3400cdadb45b0e2b4db4cdd8c26 (patch) | |
| tree | e554b491caa48116178727130806ece41a7701db | |
| parent | 2b2b207e1162e577cd6208e184d3d3a0fcfa9cca (diff) | |
| download | linux-edccaf7be062f3400cdadb45b0e2b4db4cdd8c26.tar.gz linux-edccaf7be062f3400cdadb45b0e2b4db4cdd8c26.zip | |
riscv: cpu_ops_sbi: No need to be bothered to check ret.error
If the ret.error equals to 0, the sbi_err_map_linux_errno() can also
handle it, i.e. if ret.error is SBI_SUCCESS, it will return 0
immediately, so no need to be bothered to check ret.error here.
Signed-off-by: Hui Wang <hui.wang@canonical.com>
Link: https://patch.msgid.link/20260413123515.48423-2-hui.wang@canonical.com
Signed-off-by: Paul Walmsley <pjw@kernel.org>
| -rw-r--r-- | arch/riscv/kernel/cpu_ops_sbi.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/arch/riscv/kernel/cpu_ops_sbi.c b/arch/riscv/kernel/cpu_ops_sbi.c index 146ceab1011f..ee6e4b5cc39e 100644 --- a/arch/riscv/kernel/cpu_ops_sbi.c +++ b/arch/riscv/kernel/cpu_ops_sbi.c @@ -30,10 +30,8 @@ static int sbi_hsm_hart_start(unsigned long hartid, unsigned long saddr, ret = sbi_ecall(SBI_EXT_HSM, SBI_EXT_HSM_HART_START, hartid, saddr, priv, 0, 0, 0); - if (ret.error) - return sbi_err_map_linux_errno(ret.error); - else - return 0; + + return sbi_err_map_linux_errno(ret.error); } #ifdef CONFIG_HOTPLUG_CPU @@ -43,10 +41,7 @@ static int sbi_hsm_hart_stop(void) ret = sbi_ecall(SBI_EXT_HSM, SBI_EXT_HSM_HART_STOP, 0, 0, 0, 0, 0, 0); - if (ret.error) - return sbi_err_map_linux_errno(ret.error); - else - return 0; + return sbi_err_map_linux_errno(ret.error); } static int sbi_hsm_hart_get_status(unsigned long hartid) |