diff options
author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2020-06-11 21:17:46 +0200 |
---|---|---|
committer | Wim Van Sebroeck <wim@linux-watchdog.org> | 2020-08-05 18:42:53 +0200 |
commit | 5edc8c687357b40469d90e3dab0b59b72eaabf59 (patch) | |
tree | 972ff9be0adb3ece1e79d70ed752339057bc28d6 /drivers/watchdog/ixp4xx_wdt.c | |
parent | 4f39d575844148fbf3081571a1f3b4ae04150958 (diff) |
watchdog: f71808e_wdt: do stricter parameter validation
We check the f71862fg_pin module parameter every time a watchdog device
for the f71862fg is opened, but the parameter can't change at runtime.
If we move the check to the start of init:
- We catch userspace passing invalid, but unused, values
- We check the condition only once
- We simplify the code
Do so.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Link: https://lore.kernel.org/r/20200611191750.28096-6-a.fatoum@pengutronix.de
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@linux-watchdog.org>
Diffstat (limited to 'drivers/watchdog/ixp4xx_wdt.c')
0 files changed, 0 insertions, 0 deletions