[PATCH v2] [2.6.22] pasemi: cpufreq driver

Segher Boessenkool segher at kernel.crashing.org
Fri Apr 27 05:05:59 EST 2007


>> Unfortunately the setting of the current active state is done to an 
>> SDC
>> register, while information of the states is in the PMU, so access to
>> both is needed in the driver.
>
> One thing that you could do is to list only this one register of the
> SDC in the reg property, not all of the SDC.

The system-controller node should really describe
all of the system controller.  If the Linux driver
needs to access two separate devices, it should
really just do that.  Don't try to make things
"simpler" in the device tree.

You probably should map the SDC registers in the
platform startop code, and then have the various
platform drivers use that.  Maybe even do an
abstraction ;-)


Segher




More information about the Linuxppc-dev mailing list