Re: [PATCH 0/9] ARM: OMAP: DMTIMER clean-up in preparation for device-tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



+ Tarun for any comments

On Wednesday 16 May 2012 05:05 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);
> 	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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux