Any comments Len ?
On Thu, 24 Mar 2011 17:02:07 +0800 <[email protected]> wrote: > There is no filed called "success indicator" of SFI spec. > Remove the declaration in sfi_processor.h and relative printing > message in sfi-cpufreq.c. > > Signed-off-by: Major Lee <[email protected]> > --- > arch/x86/kernel/cpu/cpufreq/sfi-cpufreq.c | 5 ++--- > include/linux/sfi_processor.h | 1 - > 2 files changed, 2 insertions(+), 4 deletions(-) > > diff -pruN linux-2.6.37.old/arch/x86/kernel/cpu/cpufreq/sfi-cpufreq.c > linux-2.6.37.new/arch/x86/kernel/cpu/cpufreq/sfi-cpufreq.c > --- linux-2.6.37.old/arch/x86/kernel/cpu/cpufreq/sfi-cpufreq.c > 2011-03-17 11:17:13.690941011 +0800 > +++ linux-2.6.37.new/arch/x86/kernel/cpu/cpufreq/sfi-cpufreq.c > 2011-03-24 16:40:16.987566000 +0800 > @@ -147,11 +147,10 @@ static int sfi_processor_get_performance > sfi_cpufreq_array[i].ctrl_val; > printk(KERN_INFO "State [%d]: core_frequency[%d] \ > transition_latency[%d] \ > - control[0x%x] status[0x%x]\n", i, > + control[0x%x]\n", i, > (u32) > pr->performance->states[i].core_frequency, (u32) > pr->performance->states[i].transition_latency, > - (u32) pr->performance->states[i].control, > - (u32) pr->performance->states[i].status); > + (u32) pr->performance->states[i].control); > } > > return result; > diff -pruN linux-2.6.37.old/include/linux/sfi_processor.h > linux-2.6.37.new/include/linux/sfi_processor.h > --- linux-2.6.37.old/include/linux/sfi_processor.h 2011-03-17 > 11:17:13.690941011 +0800 > +++ linux-2.6.37.new/include/linux/sfi_processor.h 2011-03-24 > 16:39:12.809050999 +0800 > @@ -48,7 +48,6 @@ struct sfi_processor_px { > u32 core_frequency; /* megahertz */ > u32 transition_latency; /* microseconds */ > u32 control; /* control value */ > - u32 status; /* success indicator */ > }; > > struct sfi_processor_performance { _______________________________________________ MeeGo-kernel mailing list [email protected] http://lists.meego.com/listinfo/meego-kernel
