Paul,

> -----Original Message-----
> From: Paul Walmsley [mailto:p...@pwsan.com]
> Sent: Monday, October 11, 2010 7:49 PM
> To: DebBarma, Tarun Kanti
> Cc: Cousson, Benoit; Hilman, Kevin; linux-omap@vger.kernel.org; Gopinath,
> Thara; Basak, Partha; Tony Lindgren
> Subject: RE: [PATCHv3 3/17] dmtimer: add omap2420 hwmod database
> 
> On Mon, 11 Oct 2010, DebBarma, Tarun Kanti wrote:
> 
> > In the present implementation there is inconsistency in the clock source
> > names for the different platforms, viz. OMAP2, OMAP3 and OMAP4 as shown
> > below. Therefore, I will have to modify the names so that they all have
> > common name across the platforms for the same type of clock. In this
> > regard I am proposing to modify the clock source names similar to OMAP4.
> > Of course we also have to look around to see if there are other modules
> > who are using the clock and make the necessary changes.
> 
> Please look again at the links that I sent you.  All you need to change
> are the clkdev alias names for those clocks for that particular platform
> device name, "timer" or whatever it will be called.  That won't affect any
> other modules, since they will have different platform device names.  The
> struct clk.name fields will stay the same.
I have already made the change to device names shown below. However, we are 
talking about other clock sources to devices. On OMAP4 here is are the clock 
sources:
CLK(NULL,       "sys_32k_ck",                   &sys_32k_ck,    CK_443X),
CLK(NULL,       "sys_clkin_ck",                 &sys_clkin_ck,  CK_443X),
CLK(NULL,       "syc_clk_div_ck",               &syc_clk_div_ck,        
CK_443X),

        CLK("omap-timer.1",     "fck",          &timer1_fck,    CK_443X),
        CLK("omap-timer.10",    "fck",          &timer10_fck,   CK_443X),
        CLK("omap-timer.11",    "fck",          &timer11_fck,   CK_443X),
        CLK("omap-timer.2",     "fck",          &timer2_fck,    CK_443X),
        CLK("omap-timer.3",     "fck",          &timer3_fck,    CK_443X),
        CLK("omap-timer.4",     "fck",          &timer4_fck,    CK_443X),
        CLK("omap-timer.5",     "fck",          &timer5_fck,    CK_443X),
        CLK("omap-timer.6",     "fck",          &timer6_fck,    CK_443X),
        CLK("omap-timer.7",     "fck",          &timer7_fck,    CK_443X),
        CLK("omap-timer.8",     "fck",          &timer8_fck,    CK_443X),
        CLK("omap-timer.9",     "fck",          &timer9_fck,    CK_443X),

Let me know if I am misunderstanding your point!
Thanks.
-tarun
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to