Re: [PATCH v2 1/2] ARM: OMAP2+: timer: remove CONFIG_OMAP_32K_TIMER

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

 



On Sunday 18 November 2012 08:36 PM, Igor Grinberg wrote:
CONFIG_OMAP_32K_TIMER is kind of standing on the single zImage way.
Make OMAP2+ timer code independant from the CONFIG_OMAP_32K_TIMER
setting.
To remove the dependancy, several conversions/additions had to be done:
1) Timer initialization functions are named by the platform
    name and the clock source in use.
    This also makes it possible to define and use the GPTIMER as the
    clock source instead of the 32K timer on platforms that do not have
    the 32K timer ip block or the 32K timer is not wired on the board.
    Cirrently, the the timer is chosen in the machine_desc structure on
    per board basis. Later, DT should be used to choose the timer.
2) Settings under the CONFIG_OMAP_32K_TIMER option are used as defaults
    and those under !CONFIG_OMAP_32K_TIMER are removed.
    This removes the CONFIG_OMAP_32K_TIMER on OMAP2+ timer code.
3) Since we have all the timers defined inside machine_desc structure
    and we no longer need the fallback to gp_timer clock source in case
    32k_timer clock source is unavailable (namely on AM33xx), we no
    longer need the #ifdef around __omap2_sync32k_clocksource_init()
    function. Remove the #ifdef CONFIG_OMAP_32K_TIMER around the
    __omap2_sync32k_clocksource_init() function.

Signed-off-by: Igor Grinberg <grinberg@xxxxxxxxxxxxxx>
Cc: Jon Hunter <jon-hunter@xxxxxx>
Cc: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
Cc: Vaibhav Hiremath <hvaibhav@xxxxxx>
---
nice clean-up Igor. Patch looks good to my eyes.
Feel free to add ,

Acked-by: Santosh Shilimkar <santosh.shilimkar@xxxxxx>

--
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