Hi Rasmus, > These are never modified, so might as well be const. > > Signed-off-by: Rasmus Villemoes <rasmus.villemoes@xxxxxxxxx> > --- > drivers/watchdog/bcm2835_wdt.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/watchdog/bcm2835_wdt.c b/drivers/watchdog/bcm2835_wdt.c > index 2e6164c..733e402 100644 > --- a/drivers/watchdog/bcm2835_wdt.c > +++ b/drivers/watchdog/bcm2835_wdt.c > @@ -96,7 +96,7 @@ static unsigned int bcm2835_wdt_get_timeleft(struct watchdog_device *wdog) > return WDOG_TICKS_TO_SECS(ret & PM_WDOG_TIME_SET); > } > > -static struct watchdog_ops bcm2835_wdt_ops = { > +static const struct watchdog_ops bcm2835_wdt_ops = { > .owner = THIS_MODULE, > .start = bcm2835_wdt_start, > .stop = bcm2835_wdt_stop, > @@ -104,7 +104,7 @@ static struct watchdog_ops bcm2835_wdt_ops = { > .get_timeleft = bcm2835_wdt_get_timeleft, > }; > > -static struct watchdog_info bcm2835_wdt_info = { > +static const struct watchdog_info bcm2835_wdt_info = { > .options = WDIOF_SETTIMEOUT | WDIOF_MAGICCLOSE | > WDIOF_KEEPALIVEPING, > .identity = "Broadcom BCM2835 Watchdog timer", > -- > 2.5.0 > This patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html