watchdog: tqmx86: Constify static struct watchdog_ops
authorRikard Falkeborn <rikard.falkeborn@gmail.com>
Tue, 27 Jul 2021 22:30:41 +0000 (00:30 +0200)
committerWim Van Sebroeck <wim@linux-watchdog.org>
Sun, 22 Aug 2021 08:28:10 +0000 (10:28 +0200)
The struct tqmx86_wdt_ops is only assigned to the ops pointer in the
watchdog_device struct, which is a pointer to const struct watchdog_ops.
Make it const to allow the compiler to put it in read-only memory.

Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Link: https://lore.kernel.org/r/20210727223042.48150-3-rikard.falkeborn@gmail.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@linux-watchdog.org>
drivers/watchdog/tqmx86_wdt.c

index 72d0b0adde3889e0c014e3a8197d2083e5ee3e23..83860e94ce9d72381bed5fe4eb8ed2aa2b94a992 100644 (file)
@@ -62,7 +62,7 @@ static const struct watchdog_info tqmx86_wdt_info = {
        .identity       = "TQMx86 Watchdog",
 };
 
-static struct watchdog_ops tqmx86_wdt_ops = {
+static const struct watchdog_ops tqmx86_wdt_ops = {
        .owner          = THIS_MODULE,
        .start          = tqmx86_wdt_start,
        .set_timeout    = tqmx86_wdt_set_timeout,