Paul, > -----Original Message----- > From: Paul Walmsley [mailto:paul@xxxxxxxxx] > Sent: Monday, October 11, 2010 7:49 PM > To: DebBarma, Tarun Kanti > Cc: Cousson, Benoit; Hilman, Kevin; linux-omap@xxxxxxxxxxxxxxx; 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
