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 <m.felsch@pengutronix.de>
Acked-by: Steve Twiss <stwiss.opensource@diasemi.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@linux-watchdog.org>
This commit is contained in:
Marco Felsch 2018-06-04 17:00:58 +02:00 committed by Wim Van Sebroeck
parent f587e478b5
commit 6f4cedb7b4

View File

@ -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);
}