Re: [patch] cpufreq: do not declare undefined functions

2007-04-30 Thread Dave Jones
On Mon, Apr 30, 2007 at 12:10:45PM -0700, David Rientjes wrote: > On Mon, 30 Apr 2007, Dave Jones wrote: > > > > > it's just a prototype, so no code is generated by declaring one > > > > to an unused (or even non-existant) function. > > > > > > > > Or are you using some oddball compiler

Re: [patch] cpufreq: do not declare undefined functions

2007-04-30 Thread David Rientjes
On Mon, 30 Apr 2007, Dave Jones wrote: > > > it's just a prototype, so no code is generated by declaring one > > > to an unused (or even non-existant) function. > > > > > > Or are you using some oddball compiler that warns about this? > > > > > > > If you consider gcc to be oddball, yes.

Re: [patch] cpufreq: do not declare undefined functions

2007-04-30 Thread Dave Jones
On Mon, Apr 30, 2007 at 11:42:44AM -0700, David Rientjes wrote: > On Mon, 30 Apr 2007, Dave Jones wrote: > > > it's just a prototype, so no code is generated by declaring one > > to an unused (or even non-existant) function. > > > > Or are you using some oddball compiler that warns about th

Re: [patch] cpufreq: do not declare undefined functions

2007-04-30 Thread David Rientjes
On Mon, 30 Apr 2007, Dave Jones wrote: > it's just a prototype, so no code is generated by declaring one > to an unused (or even non-existant) function. > > Or are you using some oddball compiler that warns about this? > If you consider gcc to be oddball, yes. - To unsubscribe from this list: s

Re: [patch] cpufreq: do not declare undefined functions

2007-04-30 Thread Dave Jones
On Mon, Apr 30, 2007 at 07:34:37AM -0700, David Rientjes wrote: > fill_powernow_table_pstate() and fill_powernow_table_fidvid() are only > defined and used for X86_POWERNOW_K8_ACPI. it's just a prototype, so no code is generated by declaring one to an unused (or even non-existant) function. Or

[patch] cpufreq: do not declare undefined functions

2007-04-30 Thread David Rientjes
fill_powernow_table_pstate() and fill_powernow_table_fidvid() are only defined and used for X86_POWERNOW_K8_ACPI. Cc: Dave Jones <[EMAIL PROTECTED]> Signed-off-by: David Rientjes <[EMAIL PROTECTED]> --- arch/i386/kernel/cpu/cpufreq/powernow-k8.h |2 ++ 1 files changed, 2 insertions(+), 0 dele