Hi Sachin, > Initializing clk to NULL as a reset/error condition does not > help as NULL is not an invalid condition w.r.t clk. Remove this > initialization altogether as there is no state retention. > > Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> > --- > drivers/watchdog/s3c2410_wdt.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/watchdog/s3c2410_wdt.c b/drivers/watchdog/s3c2410_wdt.c > index aba6cd46b45b..a0f8f771adec 100644 > --- a/drivers/watchdog/s3c2410_wdt.c > +++ b/drivers/watchdog/s3c2410_wdt.c > @@ -607,7 +607,6 @@ static int s3c2410wdt_probe(struct platform_device *pdev) > > err_clk: > clk_disable_unprepare(wdt->clock); > - wdt->clock = NULL; > > err: > return ret; > @@ -627,7 +626,6 @@ static int s3c2410wdt_remove(struct platform_device *dev) > s3c2410wdt_cpufreq_deregister(wdt); > > clk_disable_unprepare(wdt->clock); > - wdt->clock = NULL; > > return 0; > } Patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |