On Tue, 2012-12-04 at 13:12 +0800, Prashant Gaikwad wrote: > On Monday 03 December 2012 08:30 AM, Joseph Lo wrote: > > Add suspend, resume and rail_off_ready API into tegra_cpu_car_ops. These > > functions were used for CPU powered-down state maintenance. > > > > Signed-off-by: Joseph Lo <josephl@xxxxxxxxxx> > > --- > > arch/arm/mach-tegra/tegra20_clocks.c | 102 ++++++++++++++++++++++++++++++++++ > > 1 files changed, 102 insertions(+), 0 deletions(-) > > I have a concern here, I am working on removing these ops and replacing > it with clock. > Adding suspend/resume to these will make it more difficult to remove. > > Any other way to implement this? > Prashant, Thanks for remind. Actually the "tegra_cpu_car_ops" is more like reset & suspend/resume handling, it's not really related to clock framework. So it's more like some functions for CPU power management control. Maybe I can move them (Tegra20 & Tegra30) to another pm related file later. How do you think, Stephen? Thanks, Joseph -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html