From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Jones Subject: Re: post-OLS pending cpufreq patches Date: Tue, 3 Aug 2004 15:19:47 +0100 Sender: cpufreq-bounces@www.linux.org.uk Message-ID: <20040803141947.GO12724@redhat.com> References: <88056F38E9E48644A0F562A38C64FB600283BD34@scsmsx403.amr.corp.intel.com> <20040803124223.GB8574@dominikbrodowski.de> <20040803131729.GL12724@redhat.com> <20040803133010.GA9447@dominikbrodowski.de> <20040803133724.GM12724@redhat.com> <20040803140750.GA10095@dominikbrodowski.de> Mime-Version: 1.0 Return-path: Content-Disposition: inline In-Reply-To: <20040803140750.GA10095@dominikbrodowski.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: cpufreq-bounces+glkc-cpufreq=gmane.org@www.linux.org.uk Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Pallipadi, Venkatesh" , cpufreq@www.linux.org.uk On Tue, Aug 03, 2004 at 04:07:50PM +0200, Dominik Brodowski wrote: > > > #include > > > > > > is only included around ~300, AFAICS [even in Venki's patch]... and if you > > > move that above to line ~25 [but please only #ifdef'ed], you need to > > > > > > #include > > > > > > first. > > > > > > Could you try that out, please? > > > > getting closer. just some missing defs introduced by this patch now. > > That's the patch to include/acpi/processor.h I was talking about... > > but that is actually include/asm/acpi.h, as can be seen here: > http://bugzilla.kernel.org/attachment.cgi?id=2919&action=view > > Can you try out applying _only_ the hunk for include/asm-i386/acpi.h, > please? Yeah, that works. Or compiles at least, I don't have centrino hardware to test this. I'll check it in, and see who screams. Dave