lkml.org 
[lkml]   [2013]   [Nov]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 5/8] ARM: OMAP2+: timer: Introduce OF-friendly clocksource/clockevent system timers
    On 11/21/2013 09:51 PM, Felipe Balbi wrote:
    > Hi,
    >
    > On Thu, Nov 21, 2013 at 07:56:51PM -0600, Joel Fernandes wrote:
    >
    > [...]
    >
    >> New functions for OF-only boot are introduced, and we can soon delete the old
    >> versions once we migrate all platforms. Currently only AM335x is migrated and
    >
    > actually, you don't need to initialize .init_timer at all in DT boot.

    Actually we still do, because the plan is to keep the hwmod stuff that's
    required in timer.c in a custom .init_time, and then of_clocksource_init maybe
    called to do what you're suggesting but (not yet) more on that below.

    > Just use CLKSOURCE_OF_DECLARE() and pass your omap_generic_timer_init()
    > as argument (although, I'd call it omap_of_timer_init()).
    > That will put of_device_id structure on a special section
    > (__clksource_of_table) and pass your function as a data argument. That
    > function will be called automatically during init.
    >

    I thought of doing that, but currently the timer selection for clocksource
    is not a simple matching of compatible string, rather it is selecting the timer
    based on properties such as ti,timer-alwon and such.

    In omap3 for example, there are needs for specific timers and such have been
    provided with the macros passing in timer id etc. Right now, this can be forced
    through DT with the "ti,timer-clocksource" property I introduced. All this
    selection algorithm is too complex to be handle directly by the
    CLOCKSOURCE_OF_DECLARE / clocksource_of_init matching mechanism.

    thanks,

    -Joel




    \
     
     \ /
      Last update: 2013-11-22 16:21    [W:6.345 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site