Regarding the CPUFreq interface in HAL

[Anup Pemmaiah]

Richard,

Really appreciate your comments and sorry for the delay in my response. 
Some comments inline.

Richard Hughes wrote:

>On Thu, 2007-07-05 at 17:44 -0700, Anup Pemmaiah wrote:
>  
>
>>Hi,
>>
>>I'm working with a group inside of Sun Microsystems to port Gnome Power
>>Manager to Solaris.
>>    
>>
>
>Cool! Yell if you guys need a hand getting changes upstream :-)
>
>  
>
>>This effort will require me to implement one of the
>>HAL interfaces,"org.freedesktop.Hal.Device.CPUFreq", on Solaris. As I've
>>been considering how we might use this interface with our governors, 
>>I've begun to think that the interface, as it is currently defined, is 
>>not very flexible in its ability to handle multiple properties.
>>
>>For example, while considering one of our governors (one that is similar 
>>to the Linux OnDemand governor), it occurred to me that we might want 
>>the ability to configure properties related to threshold and sampling 
>>rates (and possibly others). While the "SetCPUFreqPerformance" method 
>>does provide a mechanism for defining at least one of these properties, 
>>it doesn't seem like a good solution for multiple properties.
>>    
>>
>
>Do you _need_ to be able to change all the parameters from the session -
>the idea of SetCPUFreqPerformance was to set a general "setting" rather
>than be able to tweak every single knob.
>
>  
>
As of now, would really like to change atleast some of the parameters of 
a session. Regarding all the parameters, we are not sure, but it seems 
likely.

>>I'm curious whether or not there are any plans to extend this interface 
>>in the future to better support more properties. I'd be happy to discuss 
>>this further if you are interested.
>>    
>>
>
>Sure, suggest away.
>  
>
Instead of having just a single value, will be nice to have a (key, 
value) pair so that most of  the currently supported parameters can be 
tweaked.Where "key" is the parameter to be tweaked and "value" is the 
new value.
Would like to propose the addition of the following two methods.

void setCPUFreqProperty(char *key, int value)
int getCPUFreqProperty(char *key)

Thanks
Anup

>Richard.
>
>
>  
>