On Tue, 17 Apr 2012, DebBarma, Tarun Kanti wrote:
> On Mon, Apr 16, 2012 at 10:20 PM, Paul Walmsley <paul@xxxxxxxxx> wrote:
> >
> >> diff --git a/arch/arm/plat-omap/include/plat/dmtimer.h b/arch/arm/plat-omap/include/plat/dmtimer.h
> >> index 9418f00..be2b8c4 100644
> >> --- a/arch/arm/plat-omap/include/plat/dmtimer.h
> >> +++ b/arch/arm/plat-omap/include/plat/dmtimer.h
> >> @@ -259,7 +259,7 @@ struct omap_dm_timer {
> >> unsigned long phys_base;
> >> int id;
> >> int irq;
> >> - struct clk *iclk, *fclk;
> >> + struct clk *fclk;
> >>
> >> void __iomem *io_base;
> >> void __iomem *sys_stat; /* TISTAT timer status */
> >
> > Am dropping this stray change, it seems unrelated to the rest of the
> > patch.
> Since the only usage of timer->iclk is removed from mach-omap2/timer.c
> there is no
> point in keeping this field in struct omap_dm_timer {}.
> Unless you are suggesting that both the above should be separated into
> a separate patch?
Ah, I just missed the first removal line. Never mind ...
- Paul
