Hi Andy, > From: andriy.shevchenko@xxxxxxxxxxxxxxx > > > > > + data.base = of_iomap(np, 2); > > > > + if (!data.base) > > > > + return -ENXIO; > > > > + > > > > + ret = keembay_clocksource_init(np, &data); > > > > + if (ret) > > > > + goto exit; > > > > + > > > > + ret = keembay_clockevent_init(np, &data); > > > > > > Is this missing ? > > > > > > > Yes. Either case it goes to the exit path. So I thought of avoiding this error > handling code. > > The point is that in success you probably won't call keembay_timer_cleanup(). > Yes. You are right, if I use this error handling code. > > > if (ret) > > > goto exit; > > > > > > return 0; > > > > > > > +exit: > > > > + keembay_timer_cleanup(np, &data); > > > > + > > > > + return ret; > > > > +} > Thanks, Vijay
