From: Marco Felsch Date: Mon, 4 Jun 2018 15:00:58 +0000 (+0200) Subject: watchdog: da9063: rename helper function to avoid misunderstandings X-Git-Tag: v5.15~8630^2~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6f4cedb7b4c35473f8f600cc09fa084185803265;p=platform%2Fkernel%2Flinux-starfive.git watchdog: da9063: rename helper function to avoid misunderstandings _da9063_wdt_set_timeout() is called by da9063_wdg_set_timeout(), da9063_wdg_start() and da9063_wdg_probe() but the name expect only to be called by da9063_wdg_set_timeout(). Rename the function to avoid misunderstandings. Signed-off-by: Marco Felsch Acked-by: Steve Twiss Reviewed-by: Guenter Roeck Signed-off-by: Guenter Roeck Signed-off-by: Wim Van Sebroeck --- diff --git a/drivers/watchdog/da9063_wdt.c b/drivers/watchdog/da9063_wdt.c index a4380a8..760aa9b 100644 --- a/drivers/watchdog/da9063_wdt.c +++ b/drivers/watchdog/da9063_wdt.c @@ -64,7 +64,7 @@ static int da9063_wdt_disable_timer(struct da9063 *da9063) DA9063_TWDSCALE_DISABLE); } -static int _da9063_wdt_set_timeout(struct da9063 *da9063, unsigned int regval) +static int da9063_wdt_update_timeout(struct da9063 *da9063, unsigned int regval) { int ret; @@ -93,7 +93,7 @@ static int da9063_wdt_start(struct watchdog_device *wdd) int ret; selector = da9063_wdt_timeout_to_sel(wdd->timeout); - ret = _da9063_wdt_set_timeout(da9063, selector); + ret = da9063_wdt_update_timeout(da9063, selector); if (ret) dev_err(da9063->dev, "Watchdog failed to start (err = %d)\n", ret); @@ -148,7 +148,7 @@ static int da9063_wdt_set_timeout(struct watchdog_device *wdd, * enabling the watchdog, so the timeout must be buffered by the driver. */ if (watchdog_active(wdd)) - ret = _da9063_wdt_set_timeout(da9063, selector); + ret = da9063_wdt_update_timeout(da9063, selector); if (ret) dev_err(da9063->dev, "Failed to set watchdog timeout (err = %d)\n", @@ -223,7 +223,7 @@ static int da9063_wdt_probe(struct platform_device *pdev) unsigned int timeout; timeout = da9063_wdt_timeout_to_sel(DA9063_WDG_TIMEOUT); - _da9063_wdt_set_timeout(da9063, timeout); + da9063_wdt_update_timeout(da9063, timeout); set_bit(WDOG_HW_RUNNING, &wdd->status); }