To: | Ralf Baechle <ralf@linux-mips.org>, Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>, Wim Van Sebroeck <wim@iguana.be>, Guenter Roeck <linux@roeck-us.net> |
---|---|
Subject: | [PATCH v2 4/8] watchdog: JZ4740: Drop module remove function |
From: | Paul Cercueil <paul@crapouillou.net> |
Date: | Sat, 30 Dec 2017 14:51:04 +0100 |
Cc: | devicetree@vger.kernel.org, linux-mips@linux-mips.org, linux-kernel@vger.kernel.org, linux-watchdog@vger.kernel.org, Paul Cercueil <paul@crapouillou.net> |
Dkim-signature: | v=1; a=rsa-sha256; c=relaxed/relaxed; d=crapouillou.net; s=mail; t=1514641893; bh=WOmD+OwLc3lFPkXz+p/dBuTtVvS3ZibFAew1o5sro8U=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References; b=BvfGmSjf/89d3o0J1byPeP6XQX+hQ0IVDmJvMsxG0UfCcTOTYbhopW5ynDsBpT/FJRNYA6xhsFOQAHB9B/peXTv3BeSNwbz94oHfNm0/hlWlM3VF4hzYSzzmcSeAXjDOqexzHnXmEjgsY1RMih1zR3OEyyq9JNm5PC5ogdvLcBs= |
In-reply-to: | <20171230135108.6834-1-paul@crapouillou.net> |
List-archive: | <http://www.linux-mips.org/archives/linux-mips/> |
List-help: | <mailto:ecartis@linux-mips.org?Subject=help> |
List-id: | linux-mips <linux-mips.eddie.linux-mips.org> |
List-owner: | <mailto:ralf@linux-mips.org> |
List-post: | <mailto:linux-mips@linux-mips.org> |
List-software: | Ecartis version 1.0.0 |
List-subscribe: | <mailto:ecartis@linux-mips.org?subject=subscribe%20linux-mips> |
List-unsubscribe: | <mailto:ecartis@linux-mips.org?subject=unsubscribe%20linux-mips> |
Original-recipient: | rfc822;linux-mips@linux-mips.org |
References: | <20171228162939.3928-2-paul@crapouillou.net> <20171230135108.6834-1-paul@crapouillou.net> |
Sender: | linux-mips-bounce@linux-mips.org |
When the watchdog was configured for nowayout, and after the userspace watchdog daemon closed the dev node without sending the magic character, unloading this module stopped the watchdog hardware, which was clearly a problem. Besides, unloading the module is not possible when the userspace watchdog daemon is running, so it's safe to assume that we don't need to stop the watchdog hardware in the jz4740_wdt_remove() function. For this reason, the jz4740_wdt_remove() function can then be dropped alltogether. Signed-off-by: Paul Cercueil <paul@crapouillou.net> --- drivers/watchdog/jz4740_wdt.c | 8 -------- 1 file changed, 8 deletions(-) v2: New patch in this series diff --git a/drivers/watchdog/jz4740_wdt.c b/drivers/watchdog/jz4740_wdt.c index fa7f49a3212c..02b9b8e946a2 100644 --- a/drivers/watchdog/jz4740_wdt.c +++ b/drivers/watchdog/jz4740_wdt.c @@ -205,16 +205,8 @@ static int jz4740_wdt_probe(struct platform_device *pdev) return 0; } -static int jz4740_wdt_remove(struct platform_device *pdev) -{ - struct jz4740_wdt_drvdata *drvdata = platform_get_drvdata(pdev); - - return jz4740_wdt_stop(&drvdata->wdt); -} - static struct platform_driver jz4740_wdt_driver = { .probe = jz4740_wdt_probe, - .remove = jz4740_wdt_remove, .driver = { .name = "jz4740-wdt", .of_match_table = of_match_ptr(jz4740_wdt_of_matches), -- 2.11.0 |
Previous by Date: | [PATCH v2 3/8] watchdog: JZ4740: Register a restart handler, Paul Cercueil |
---|---|
Next by Date: | [PATCH v2 5/8] MIPS: jz4740: dts: Add bindings for the jz4740-wdt driver, Paul Cercueil |
Previous by Thread: | [PATCH v2 3/8] watchdog: JZ4740: Register a restart handler, Paul Cercueil |
Next by Thread: | Re: [PATCH v2 4/8] watchdog: JZ4740: Drop module remove function, Guenter Roeck |
Indexes: | [Date] [Thread] [Top] [All Lists] |