Re: [PATCH V2 4/7] watchdog: nuc900_wdt: use devm_*() functions

2013-05-25 Thread Wim Van Sebroeck
Hi Jingoo, > Use devm_*() functions to make cleanup paths simpler. > > Signed-off-by: Jingoo Han > --- > Changes since v1: > - used sizeof(*nuc900_wdt) instead of sizeof(struct nuc900_wdt) > - declared 'nuc900_wdt->res' variable as logical variable > > drivers/watchdog/nuc900_wdt.c | 50 > +

Re: [PATCH V2 4/7] watchdog: nuc900_wdt: use devm_*() functions

2013-04-29 Thread Guenter Roeck
On Tue, Apr 30, 2013 at 02:01:25PM +0900, Jingoo Han wrote: > Use devm_*() functions to make cleanup paths simpler. > > Signed-off-by: Jingoo Han > --- > Changes since v1: > - used sizeof(*nuc900_wdt) instead of sizeof(struct nuc900_wdt) > - declared 'nuc900_wdt->res' variable as logical variable

[PATCH V2 4/7] watchdog: nuc900_wdt: use devm_*() functions

2013-04-29 Thread Jingoo Han
Use devm_*() functions to make cleanup paths simpler. Signed-off-by: Jingoo Han --- Changes since v1: - used sizeof(*nuc900_wdt) instead of sizeof(struct nuc900_wdt) - declared 'nuc900_wdt->res' variable as logical variable drivers/watchdog/nuc900_wdt.c | 50 +-