ARM: OMAP: Add DMTIMER capability variable to represent timer features
authorJon Hunter <jon-hunter@ti.com>
Tue, 5 Jun 2012 17:34:52 +0000 (12:34 -0500)
committerTony Lindgren <tony@atomide.com>
Thu, 14 Jun 2012 09:39:07 +0000 (02:39 -0700)
Although the OMAP timers share a common hardware design, there are some
differences between the timer instances in a given device. For example, a timer
maybe in a power domain that can be powered-of, so can lose its logic state and
need restoring where as another may be in power domain that is always be on.
Another example, is a timer may support different clock sources to drive the
timer. This information is passed to the dmtimer via the following platform data
structure.

struct dmtimer_platform_data {
int (*set_timer_src)(struct platform_device *pdev, int source);
int timer_ip_version;
u32 needs_manual_reset:1;
bool loses_context;
int (*get_context_loss_count)(struct device *dev);
};

The above structure uses multiple variables to represent the timer features.
HWMOD also stores the timer capabilities using a bit-mask that represents the
features supported. By using the same format for representing the timer
features in the platform data as used by HWMOD, we can ...

1. Use the flags defined in the plat/dmtimer.h to represent the features
   supported.
2. For devices using HWMOD, we can retrieve the features supported from HWMOD.
3. Eventually, simplify the platform data structure to be ...

struct dmtimer_platform_data {
int (*set_timer_src)(struct platform_device *pdev, int source);
u32 timer_capability;
}

Another benefit from doing this, is that it will simplify the migration of the
dmtimer driver to device-tree. For example, in the current OMAP2+ timer code the
"loses_context" variable is configured at runtime by calling an architecture
specific function. For device tree this creates a problem, because we would need
to call the architecture specific function from within the dmtimer driver.
However, such attributes do not need to be queried at runtime and we can look up
the attributes via HWMOD or device-tree.

This changes a new "capability" variable to the platform data and timer
structure so we can start removing and simplifying the platform data structure.

Signed-off-by: Jon Hunter <jon-hunter@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
arch/arm/mach-omap2/timer.c
arch/arm/plat-omap/dmtimer.c
arch/arm/plat-omap/include/plat/dmtimer.h

index b0b208077c96230392350815d553c904e5d3a322..d8a5dc3d695f1c42fc2c5b2e3e1ce3efed739733 100644 (file)
@@ -500,6 +500,9 @@ static int __init omap_timer_init(struct omap_hwmod *oh, void *unused)
        pdata->set_timer_src = omap2_dm_timer_set_src;
        pdata->timer_ip_version = oh->class->rev;
 
+       if (timer_dev_attr)
+               pdata->timer_capability = timer_dev_attr->timer_capability;
+
        pwrdm = omap_hwmod_get_pwrdm(oh);
        pdata->loses_context = pwrdm_can_ever_lose_context(pwrdm);
 #ifdef CONFIG_PM
index f5b5c89ac7c2e3225367cf78d24688f90b107486..30742d8e68194ebe2fdbb0f0c278016da5f11760 100644 (file)
@@ -694,6 +694,7 @@ static int __devinit omap_dm_timer_probe(struct platform_device *pdev)
        timer->pdev = pdev;
        timer->loses_context = pdata->loses_context;
        timer->get_context_loss_count = pdata->get_context_loss_count;
+       timer->capability = pdata->timer_capability;
 
        /* Skip pm_runtime_enable for OMAP1 */
        if (!pdata->needs_manual_reset) {
index 1e5ce5d5624098e0819d9937f57232e3a0270993..48e54caf90013ba8a96ff926b3c6f598b6afcb35 100644 (file)
@@ -101,6 +101,7 @@ struct dmtimer_platform_data {
        bool loses_context;
 
        int (*get_context_loss_count)(struct device *dev);
+       u32 timer_capability;
 };
 
 int omap_dm_timer_reserve_systimer(int id);
@@ -273,6 +274,7 @@ struct omap_dm_timer {
        bool loses_context;
        int ctx_loss_count;
        int revision;
+       u32 capability;
        struct platform_device *pdev;
        struct list_head node;