From: Andrew Goodbody <andrew.goodbody@linaro.org>
Date: Thu, 7 Aug 2025 14:28:57 +0000 (+0100)
Subject: power: axp: Remove redundant code
X-Git-Url: https://git.openpandora.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=446b7b8f2d9b5b9bcd473c91f1e45cbb77be6c5f;p=pandora-u-boot.git

power: axp: Remove redundant code

In axp_init after checking the chip ID there is an else clause that
returns ret. ret is guaranteed to be 0 at this point as the code would
have returned above if not. The next statement is a return 0 so the
return ret is redundant, remove it.

This issue was found by Smatch.

Signed-off-by: Andrew Goodbody <andrew.goodbody@linaro.org>
Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>
---

diff --git a/drivers/power/axp818.c b/drivers/power/axp818.c
index 83ae6ecc138..01f9a209dc6 100644
--- a/drivers/power/axp818.c
+++ b/drivers/power/axp818.c
@@ -248,8 +248,6 @@ int axp_init(void)
 
 	if (!(axp_chip_id == 0x51))
 		return -ENODEV;
-	else
-		return ret;
 
 	return 0;
 }