Re: [PATCH] Replace clk_use/unuse with clk_enable/disable, please test

[Tony Lindgren <tony@atomide.com>]

* Toshihiro.Kobayashi@nokia.com <Toshihiro.Kobayashi@nokia.com> [060116 02:16]:
> Hi,
> 
> Here's a further patch to centralize the pm code. (patch-dsp_ckfix3)

Thanks, pushing today.
 
> >From: ext Tony Lindgren [mailto:tony@atomide.com] 
> >Sent: Saturday, January 14, 2006 10:13 AM
> 
> >Thanks for deciphering it :) I wonder if we could just take care of the
> >temporary enable in the clock framework?
> 
> I tried, please look at the one more patch.
> (patch-clk_enable_force-test)
> But I myself don't agree with this patch ;-(

OK
 
> I feel it's too arbitrate to add clk_enable_force() and
> clk_disable_force() to the clk_functions struct, only for the
> pm code. And I think it's OK to do raw register accesses within
> the pm code, including clock enableing / disabling, since anyway
> the assembler part (omapXXXX_idle_loop_suspend) does such
> operations.

Yeah, some direct register access is needed in PM code...

Tony