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