From: Tony Lindgren <tony@atomide.com>
To: "DebBarma, Tarun Kanti" <tarun.kanti@ti.com>
Cc: "Shilimkar, Santosh" <santosh.shilimkar@ti.com>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>
Subject: Re: [PATCH v11 6/8] dmtimer: switch-over to platform device driver
Date: Mon, 7 Mar 2011 16:07:28 -0800 [thread overview]
Message-ID: <20110308000727.GA26463@atomide.com> (raw)
In-Reply-To: <5A47E75E594F054BAF48C5E4FC4B92AB037A360AD0@dbde02.ent.ti.com>
* DebBarma, Tarun Kanti <tarun.kanti@ti.com> [110307 04:52]:
> >
> > I agree with Tony. We can still registers only the system
> > timer because the timer ID is known well before we do
> > initialization.
> I must say, there was a gap in my understanding of the comment.
> So, here is the implementation change:
>
> -> Call omap2_dm_timer_early_init(u8 gptimer_id) from omap2_gp_timer_init()
> -> Initialize only gptimer_id inside this function
> -> In the regular init skip initialization of gptimer_id
>
> As a result of this, we can rid of double registration and related code.
I suggest we keep the system timer completely separate from the
rest of the timers. This way we avoid adding hwmod dependencies
early on when the system timer is initialized, and can still
use hwmod for handling the the rest of the timers nicely.
The rest of the timers can then become just a regular device
driver that ignores the physical system timer.
Regards,
Tony
next prev parent reply other threads:[~2011-03-08 0:07 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-24 11:26 [PATCH v11 0/8] dmtimer adaptation to platform_driver Tarun Kanti DebBarma
2011-02-24 11:26 ` [PATCH v11 1/8] OMAP2+: dmtimer: add device names to flck nodes Tarun Kanti DebBarma
2011-02-24 11:26 ` [PATCH v11 2/8] OMAP4: hwmod data: add dmtimer version information Tarun Kanti DebBarma
2011-03-04 0:24 ` Kevin Hilman
2011-03-04 5:49 ` DebBarma, Tarun Kanti
2011-03-04 11:16 ` Cousson, Benoit
2011-03-07 11:19 ` DebBarma, Tarun Kanti
2011-02-24 11:26 ` [PATCH v11 3/8] OMAP1: dmtimer: conversion to platform devices Tarun Kanti DebBarma
2011-02-24 11:26 ` [PATCH v11 4/8] OMAP2+: dmtimer: convert " Tarun Kanti DebBarma
2011-03-04 1:01 ` Kevin Hilman
2011-03-04 6:34 ` DebBarma, Tarun Kanti
2011-03-04 17:25 ` Kevin Hilman
2011-03-04 19:24 ` DebBarma, Tarun Kanti
2011-02-24 11:26 ` [PATCH v11 5/8] OMAP: dmtimer: platform driver Tarun Kanti DebBarma
2011-03-04 0:35 ` Kevin Hilman
2011-03-04 9:07 ` DebBarma, Tarun Kanti
2011-03-04 1:29 ` Kevin Hilman
2011-03-04 6:56 ` DebBarma, Tarun Kanti
2011-03-04 16:53 ` Kevin Hilman
2011-03-04 19:07 ` DebBarma, Tarun Kanti
2011-02-24 11:26 ` [PATCH v11 6/8] dmtimer: switch-over to platform device driver Tarun Kanti DebBarma
2011-03-04 1:25 ` Kevin Hilman
2011-03-04 6:57 ` DebBarma, Tarun Kanti
2011-03-04 17:23 ` Tony Lindgren
2011-03-04 18:52 ` DebBarma, Tarun Kanti
2011-03-04 19:37 ` Tony Lindgren
2011-03-04 19:41 ` DebBarma, Tarun Kanti
2011-03-05 7:54 ` Santosh Shilimkar
2011-03-07 12:54 ` DebBarma, Tarun Kanti
2011-03-08 0:07 ` Tony Lindgren [this message]
2011-03-08 0:11 ` DebBarma, Tarun Kanti
2011-02-24 11:26 ` [PATCH v11 7/8] OMAP: dmtimer: pm_runtime support Tarun Kanti DebBarma
2011-03-04 1:22 ` Kevin Hilman
2011-03-04 9:18 ` DebBarma, Tarun Kanti
2011-03-04 17:28 ` Tony Lindgren
2011-03-04 18:57 ` DebBarma, Tarun Kanti
2011-03-04 19:37 ` Tony Lindgren
2011-03-04 19:52 ` DebBarma, Tarun Kanti
2011-03-05 0:01 ` Tony Lindgren
2011-03-05 0:22 ` DebBarma, Tarun Kanti
2011-03-08 0:10 ` Tony Lindgren
2011-03-08 0:13 ` DebBarma, Tarun Kanti
2011-02-24 11:26 ` [PATCH v11 8/8] OMAP: dmtimer: add timeout to low-level routines Tarun Kanti DebBarma
2011-02-25 14:08 ` [PATCH v11 0/8] dmtimer adaptation to platform_driver DebBarma, Tarun Kanti
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110308000727.GA26463@atomide.com \
--to=tony@atomide.com \
--cc=linux-omap@vger.kernel.org \
--cc=santosh.shilimkar@ti.com \
--cc=tarun.kanti@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).