Hi Jon, On 5/16/2012 1:35 AM, Jon Hunter wrote:
From: Jon Hunter<jon-hunter@xxxxxx> In order to migrate the dmtimer driver to support device-tree I found that it was first necessary to clean-up the timer platform data. The goal of this series is to simplify the timer platform data structure from ... struct dmtimer_platform_data { int (*set_timer_src)(struct platform_device *pdev, int source); int timer_ip_version; u32 needs_manual_reset:1; bool reserved; bool loses_context; int (*get_context_loss_count)(struct device *dev); }; to ... struct dmtimer_platform_data { int (*set_timer_src)(struct platform_device *pdev, int source);
I guess that custom set_timer_src should not be there at all anymore. Well at least for OMAP2+. We should just use the regular clock API to change the parent. I do not see why we should add that wrapper on top of the clock API and thus store some internal clock name inside the timer device init code.
Regards, Benoit
u32 timer_capability; }; ... where timer_capability is a bit mask that indicates the timer features supported and uses the HWMOD timer capabilities flags described in plat/dmtimer.h. For OMAP2+ devices this allows us to read the timer capabilities from the HWMOD data and for OMAP1 devices the flags are simply populated by the timer initialisation code. Eventually, the aim is to read the timer capabilities from the device tree blob. This series includes some fixes as well as clean-up. If it is preferred to split the series into fixes and clean-up I can do that, but wanted to get some feedback on this approach. This series is based upon the current linux-omap master branch. I have built both omap1 and omap2plus configurations as well as booted the respective kernels on the omap5912 OSK (omap1), omap2430 SDP, OMAP3430 Beagle and OMAP4460 PANDA. Jon Hunter (9): ARM: OMAP: Remove unnecessary clk structure ARM: OMAP2+: Remove unused max number of timers definition ARM: OMAP2+: Add dmtimer platform function to reserve systimers ARM: OMAP: Represent timer features using HWMOD flags ARM: OMAP2+: HWMOD: Correct timer device attributes ARM: OMAP2+: Fix external clock support for dmtimers ARM: OMAP: Remove loses_context variable from timer platform data ARM: OMAP: Remove timer function pointer for context loss counter ARM: OMAP: Add flag to indicate if a timer needs a manual reset arch/arm/mach-omap1/timer.c | 3 +- arch/arm/mach-omap2/omap_hwmod_2xxx_ipblock_data.c | 24 ++++++---- arch/arm/mach-omap2/omap_hwmod_3xxx_data.c | 10 +---- arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 6 -- arch/arm/mach-omap2/timer.c | 22 ++------- arch/arm/plat-omap/dmtimer.c | 49 ++++++++++++-------- arch/arm/plat-omap/include/plat/dmtimer.h | 21 ++------ 7 files changed, 57 insertions(+), 78 deletions(-)
-- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html