watchdog: make watchdog_deferred_registration_add() void

It cannot fail, so no return value needed.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@linux-watchdog.org>
This commit is contained in:
Wolfram Sang 2019-05-18 23:27:16 +02:00 committed by Wim Van Sebroeck
parent 46713c3d2f
commit 80030425e0

View File

@ -60,11 +60,10 @@ static DEFINE_MUTEX(wtd_deferred_reg_mutex);
static LIST_HEAD(wtd_deferred_reg_list); static LIST_HEAD(wtd_deferred_reg_list);
static bool wtd_deferred_reg_done; static bool wtd_deferred_reg_done;
static int watchdog_deferred_registration_add(struct watchdog_device *wdd) static void watchdog_deferred_registration_add(struct watchdog_device *wdd)
{ {
list_add_tail(&wdd->deferred, list_add_tail(&wdd->deferred,
&wtd_deferred_reg_list); &wtd_deferred_reg_list);
return 0;
} }
static void watchdog_deferred_registration_del(struct watchdog_device *wdd) static void watchdog_deferred_registration_del(struct watchdog_device *wdd)
@ -265,13 +264,13 @@ static int __watchdog_register_device(struct watchdog_device *wdd)
int watchdog_register_device(struct watchdog_device *wdd) int watchdog_register_device(struct watchdog_device *wdd)
{ {
int ret; int ret = 0;
mutex_lock(&wtd_deferred_reg_mutex); mutex_lock(&wtd_deferred_reg_mutex);
if (wtd_deferred_reg_done) if (wtd_deferred_reg_done)
ret = __watchdog_register_device(wdd); ret = __watchdog_register_device(wdd);
else else
ret = watchdog_deferred_registration_add(wdd); watchdog_deferred_registration_add(wdd);
mutex_unlock(&wtd_deferred_reg_mutex); mutex_unlock(&wtd_deferred_reg_mutex);
return ret; return ret;
} }