Messages in this thread | | | Date | Tue, 10 May 2011 13:14:00 +0200 | From | Wim Van Sebroeck <> | Subject | Re: [PATCHv1 -next] WATCHDOG: Watchdog module of DA9052 device driver |
| |
Hi Ashish, David,
Driver looks OK at first glance. Will look at it some more later this week.
> +static int __devexit da9052_sm_remove(struct platform_device *dev) > +{ > + misc_deregister(&da9052_wdt_miscdev); > + strobe_interval = 0; > + strobe_mode = 0; > + return 0; > +}
Small remark: here I am missing a kfree(wdt);
Kind regards, Wim.
| |