[v6,6/9] watchdog: rzg2l_wdt: Remove comparison with zero

Message ID	20240208105817.2619703-7-claudiu.beznea.uj@bp.renesas.com (mailing list archive)
State	Superseded
Delegated to:	Geert Uytterhoeven
Headers	show Received: from mail-wm1-f50.google.com (mail-wm1-f50.google.com [209.85.128.50]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 698FC7868A for <linux-renesas-soc@vger.kernel.org>; Thu, 8 Feb 2024 10:58:34 +0000 (UTC) From: Claudiu <claudiu.beznea@tuxon.dev> To: wim@linux-watchdog.org, linux@roeck-us.net, robh@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, p.zabel@pengutronix.de, geert+renesas@glider.be, magnus.damm@gmail.com, biju.das.jz@bp.renesas.com Cc: linux-watchdog@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, claudiu.beznea@tuxon.dev, Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com> Subject: [PATCH v6 6/9] watchdog: rzg2l_wdt: Remove comparison with zero Date: Thu, 8 Feb 2024 12:58:14 +0200 Message-Id: <20240208105817.2619703-7-claudiu.beznea.uj@bp.renesas.com> In-Reply-To: <20240208105817.2619703-1-claudiu.beznea.uj@bp.renesas.com> References: <20240208105817.2619703-1-claudiu.beznea.uj@bp.renesas.com> Precedence: bulk MIME-Version: 1.0 Content-Transfer-Encoding: 8bit
Series	watchdog: rzg2l_wdt: Add support for RZ/G3S \| expand [v6,0/9] watchdog: rzg2l_wdt: Add support for RZ/G3S [v6,1/9] watchdog: rzg2l_wdt: Restrict the driver to ARCH_RZG2L and ARCH_R9A09G011 [v6,2/9] watchdog: rzg2l_wdt: Make the driver depend on PM [v6,3/9] watchdog: rzg2l_wdt: Use pm_runtime_resume_and_get() [v6,4/9] watchdog: rzg2l_wdt: Check return status of pm_runtime_put() [v6,5/9] watchdog: rzg2l_wdt: Remove reset de-assert from probe [v6,6/9] watchdog: rzg2l_wdt: Remove comparison with zero [v6,7/9] watchdog: rzg2l_wdt: Rely on the reset driver for doing proper reset [v6,8/9] watchdog: rzg2l_wdt: Add suspend/resume support [v6,9/9] dt-bindings: watchdog: renesas,wdt: Document RZ/G3S support

Message ID

20240208105817.2619703-7-claudiu.beznea.uj@bp.renesas.com (mailing list archive)

State

Superseded

Delegated to:

Geert Uytterhoeven

Headers

From: Claudiu <claudiu.beznea@tuxon.dev>
To: wim@linux-watchdog.org,
	linux@roeck-us.net,
	robh@kernel.org,
	krzysztof.kozlowski+dt@linaro.org,
	conor+dt@kernel.org,
	p.zabel@pengutronix.de,
	geert+renesas@glider.be,
	magnus.damm@gmail.com,
	biju.das.jz@bp.renesas.com
Cc: linux-watchdog@vger.kernel.org,
	devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	linux-renesas-soc@vger.kernel.org,
	claudiu.beznea@tuxon.dev,
	Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
Subject: [PATCH v6 6/9] watchdog: rzg2l_wdt: Remove comparison with zero
Date: Thu,  8 Feb 2024 12:58:14 +0200
Message-Id: <20240208105817.2619703-7-claudiu.beznea.uj@bp.renesas.com>
In-Reply-To: <20240208105817.2619703-1-claudiu.beznea.uj@bp.renesas.com>
References: <20240208105817.2619703-1-claudiu.beznea.uj@bp.renesas.com>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit

Series

watchdog: rzg2l_wdt: Add support for RZ/G3S | expand

Commit Message

Claudiu Feb. 8, 2024, 10:58 a.m. UTC

From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>

devm_add_action_or_reset() could return -ENOMEM or zero. Thus, remove
comparison with zero of the returning value to make code simpler.

Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
---

Changes in v6:
- none

Changes in v5:
- none

Changes in v4:
- none

Changes in v3:
- none

Changes in v2:
- none


 drivers/watchdog/rzg2l_wdt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/watchdog/rzg2l_wdt.c b/drivers/watchdog/rzg2l_wdt.c
index 93a49fd0c7aa..29eb47bcf984 100644
--- a/drivers/watchdog/rzg2l_wdt.c
+++ b/drivers/watchdog/rzg2l_wdt.c
@@ -316,7 +316,7 @@  static int rzg2l_wdt_probe(struct platform_device *pdev)
 
 	watchdog_set_drvdata(&priv->wdev, priv);
 	ret = devm_add_action_or_reset(&pdev->dev, rzg2l_wdt_pm_disable, &priv->wdev);
-	if (ret < 0)
+	if (ret)
 		return ret;
 
 	watchdog_set_nowayout(&priv->wdev, nowayout);

[v6,6/9] watchdog: rzg2l_wdt: Remove comparison with zero

Commit Message

Patch