Re: [PATCH] ARM: omap: clock: Get rid of unwanted clkdm assocations within clks

2012-06-11 Thread Rajendra Nayak
Hi Paul, So why should getting rid of some *unused* data have any testing overhead? Your definition of 'unused' seems to be different than mine: $ egrep -rn 'c(lk|)-clkdm' arch/arm/ arch/arm/mach-omap2/omap_hwmod.c:560: if (oh-_clk-clkdm oh-_clk-clkdm-flags CLKDM_NO_AUTODEPS)

Re: [PATCH] ARM: omap: clock: Get rid of unwanted clkdm assocations within clks

2012-06-08 Thread Paul Walmsley
Hi On Thu, 7 Jun 2012, Rajendra Nayak wrote: Yes, I can include this as part of the common clock conversion series. What I was trying to say is that neither the clock framework not any other OMAP PM layer today makes any use of this information except for gate clocks. The only 2 places in

Re: [PATCH] ARM: omap: clock: Get rid of unwanted clkdm assocations within clks

2012-06-08 Thread Rajendra Nayak
Hi Paul, On Friday 08 June 2012 01:10 PM, Paul Walmsley wrote: Hi On Thu, 7 Jun 2012, Rajendra Nayak wrote: Yes, I can include this as part of the common clock conversion series. What I was trying to say is that neither the clock framework not any other OMAP PM layer today makes any use of

Re: [PATCH] ARM: omap: clock: Get rid of unwanted clkdm assocations within clks

2012-06-08 Thread Paul Walmsley
On Fri, 8 Jun 2012, Rajendra Nayak wrote: This patch only gets rid of clkdm assocation from *non gate* clocks. And what I am trying to say is that, this information in not used at all (for non gate clocks) in any of the frameworks today including clock framework. So why should getting rid of

Re: [PATCH] ARM: omap: clock: Get rid of unwanted clkdm assocations within clks

2012-06-07 Thread Rajendra Nayak
Hi Paul, On Thursday 17 May 2012 03:54 PM, Rajendra Nayak wrote: clkdm assocations with clocks in the clock framework are useful only for 'gate' clocks which have enable/disable ops populated. Get rid of the clkdm_names populated in any other type of clocks. Any comments on this one?

Re: [PATCH] ARM: omap: clock: Get rid of unwanted clkdm assocations within clks

2012-06-07 Thread Paul Walmsley
Hi On Thu, 7 Jun 2012, Rajendra Nayak wrote: On Thursday 17 May 2012 03:54 PM, Rajendra Nayak wrote: clkdm assocations with clocks in the clock framework are useful only for 'gate' clocks which have enable/disable ops populated. Get rid of the clkdm_names populated in any other type of

Re: [PATCH] ARM: omap: clock: Get rid of unwanted clkdm assocations within clks

2012-06-07 Thread Rajendra Nayak
Hi Paul, On Thursday 07 June 2012 12:37 PM, Paul Walmsley wrote: Hi On Thu, 7 Jun 2012, Rajendra Nayak wrote: On Thursday 17 May 2012 03:54 PM, Rajendra Nayak wrote: clkdm assocations with clocks in the clock framework are useful only for 'gate' clocks which have enable/disable ops

[PATCH] ARM: omap: clock: Get rid of unwanted clkdm assocations within clks

2012-05-17 Thread Rajendra Nayak
clkdm assocations with clocks in the clock framework are useful only for 'gate' clocks which have enable/disable ops populated. Get rid of the clkdm_names populated in any other type of clocks. Signed-off-by: Rajendra Nayak rna...@ti.com --- arch/arm/mach-omap2/clock2420_data.c | 16