Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965110AbWIQVTT (ORCPT ); Sun, 17 Sep 2006 17:19:19 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965107AbWIQVTT (ORCPT ); Sun, 17 Sep 2006 17:19:19 -0400 Received: from gprs189-60.eurotel.cz ([160.218.189.60]:43663 "EHLO amd.ucw.cz") by vger.kernel.org with ESMTP id S965111AbWIQVTS (ORCPT ); Sun, 17 Sep 2006 17:19:18 -0400 Date: Sun, 17 Sep 2006 19:48:35 +0200 From: Pavel Machek To: David Singleton Cc: Greg KH , linux-pm@lists.osdl.org, kernel list Subject: Re: OpPoint summary Message-ID: <20060917174835.GA2225@elf.ucw.cz> References: <20060911195546.GB11901@elf.ucw.cz> <4505CCDA.8020501@gmail.com> <20060911210026.GG11901@elf.ucw.cz> <4505DDA6.8080603@gmail.com> <20060911225617.GB13474@elf.ucw.cz> <20060912001701.GC14234@linux.intel.com> <20060912033700.GD27397@kroah.com> <20060914055529.GA18031@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Warning: Reading this can be dangerous to your mental health. User-Agent: Mutt/1.5.11+cvs20060126 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1454 Lines: 37 Hi! > >Care to resend your patches in the proper format, through email so that > >we can see them, and possibly get some testing in -mm if they look sane? > > Greg, > here's the patch that implements operating points for different > frequencies > for the speedstep-centrino line of processors. Operating points are created > in much the same manner that cpufreq tables are. This works for both > simple implementations like the centrino and more complex SoC systems > like the arm-pxa72x which has several clocks to control, and different clock > divisors and multipliers. > +static struct oppoint lowest = { > + .name = "lowest", > + .type = PM_FREQ_CHANGE, > + .frequency = 0, > + .voltage = 0, > + .latency = 15, > + .prepare_transition = cpufreq_prepare_transition, > + .transition = centrino_transition, > + .finish_transition = cpufreq_finish_transition, > +}; We had nice, descriptive interface... with numbers. Now you want to introduce english state names... looks like a step back to me. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/