From 514a837a206f43942e5a7b97a37a045e7c1c95f2 Mon Sep 17 00:00:00 2001 From: Tim Gover Date: Thu, 22 Oct 2020 15:30:55 +0100 Subject: [PATCH] watchdog: bcm2835: Ignore params after the partition number Use sscanf to extract the partition number and ignore extra parameters which are only relevant to other reboot notifiers. --- drivers/watchdog/bcm2835_wdt.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/watchdog/bcm2835_wdt.c b/drivers/watchdog/bcm2835_wdt.c index 1931c80..2ce96ba0 100644 --- a/drivers/watchdog/bcm2835_wdt.c +++ b/drivers/watchdog/bcm2835_wdt.c @@ -127,10 +127,12 @@ static int bcm2835_restart(struct watchdog_device *wdog, { struct bcm2835_wdt *wdt = watchdog_get_drvdata(wdog); - unsigned long long val; + unsigned long val; u8 partition = 0; - if (data && !kstrtoull(data, 0, &val) && val <= 63) + // Allow extra arguments separated by spaces after + // the partition number. + if (data && sscanf(data, "%lu", &val) && val < 63) partition = val; __bcm2835_restart(wdt, partition); -- 2.7.4