diff options
| author | Ahmad Fatoum <ahmad@a3f.at> | 2019-11-21 09:40:04 +0100 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2019-12-05 09:10:26 +0100 |
| commit | aade8d53cf6cccf7958d2c11e5791dfcd85a584f (patch) | |
| tree | 6750480db84ad34ccff3b8e2105c8e24bc6ad8fc /drivers/watchdog/wd_core.c | |
| parent | 713a601bde07bb498f99bc6288d0c10ddcbe2e0a (diff) | |
| download | barebox-aade8d53cf6cccf7958d2c11e5791dfcd85a584f.tar.gz barebox-aade8d53cf6cccf7958d2c11e5791dfcd85a584f.tar.xz | |
watchdog: core: use new dev_add_param_tristate helper for .running param
Previous commit added a dev_add_param_tristate_ro that can be readily
used instead of the enum parameter here. Use it.
This also fixes the issue that running_names had external linkage.
Signed-off-by: Ahmad Fatoum <ahmad@a3f.at>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'drivers/watchdog/wd_core.c')
| -rw-r--r-- | drivers/watchdog/wd_core.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/drivers/watchdog/wd_core.c b/drivers/watchdog/wd_core.c index fcead11755..b6e2a37b1f 100644 --- a/drivers/watchdog/wd_core.c +++ b/drivers/watchdog/wd_core.c @@ -152,12 +152,6 @@ static unsigned int dev_get_watchdog_priority(struct device_d *dev) return priority; } -const char *running_names[] = { - [WDOG_HW_RUNNING_UNSUPPORTED] = "unknown", - [WDOG_HW_RUNNING] = "1", - [WDOG_HW_NOT_RUNNING] = "0", -}; - int watchdog_register(struct watchdog *wd) { struct param_d *p; @@ -176,8 +170,7 @@ int watchdog_register(struct watchdog *wd) if (ret) return ret; - p = dev_add_param_enum_ro(&wd->dev, "running", &wd->running, - running_names, ARRAY_SIZE(running_names)); + p = dev_add_param_tristate_ro(&wd->dev, "running", &wd->running); if (IS_ERR(p)) return PTR_ERR(p); |