On Tue, 17 Apr 2012, DebBarma, Tarun Kanti wrote:

> On Mon, Apr 16, 2012 at 10:20 PM, Paul Walmsley <p...@pwsan.com> 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

Reply via email to