After if (ret) return ret;
we know that ret is zero. Don't check it again. Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de> --- arch/riscv/lib/andes_plic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/riscv/lib/andes_plic.c b/arch/riscv/lib/andes_plic.c index 5cf29df670..c2a8fe4d9e 100644 --- a/arch/riscv/lib/andes_plic.c +++ b/arch/riscv/lib/andes_plic.c @@ -52,7 +52,7 @@ static int init_plic(void) if (ret) return ret; - if (ret == 0 && dev) { + if (dev) { ofnode_for_each_subnode(node, dev_ofnode(dev->parent)) { const char *device_type; -- 2.27.0