From owner-freebsd-acpi@FreeBSD.ORG Fri Mar 11 13:37:41 2005 Return-Path: Delivered-To: freebsd-acpi@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 29F6D16A4CE for ; Fri, 11 Mar 2005 13:37:41 +0000 (GMT) Received: from poup.poupinou.org (poup.poupinou.org [195.101.94.96]) by mx1.FreeBSD.org (Postfix) with ESMTP id BB57843D60 for ; Fri, 11 Mar 2005 13:37:40 +0000 (GMT) (envelope-from ducrot@poupinou.org) Received: from ducrot by poup.poupinou.org with local (Exim) id 1D9kKC-0002nn-00; Fri, 11 Mar 2005 14:37:08 +0100 Date: Fri, 11 Mar 2005 14:37:08 +0100 To: Nate Lawson Message-ID: <20050311133708.GD2298@poupinou.org> References: <20050310171025.GB2298@poupinou.org> <4230AF39.4090100@root.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4230AF39.4090100@root.org> User-Agent: Mutt/1.5.6+20040907i From: Bruno Ducrot cc: acpi@freebsd.org Subject: Re: [cpufreq] powernow drivers X-BeenThere: freebsd-acpi@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: ACPI and power management development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Mar 2005 13:37:41 -0000 On Thu, Mar 10, 2005 at 12:34:01PM -0800, Nate Lawson wrote: > Bruno Ducrot wrote: > >The powernow stuff is nearly complete but need testings and more > >cleanup, especially the 2 drivers shoud be merged. > >I'm providing them right now at > >http://www.poupinou.org/cpufreq/bsd/powernow.tar.gz > >before the merge because I maybe offline for one or two weeks in > >a few days. > > > >The powernow k6 is not finished, but mainly because I dont have the > >hardware. It would be fine though that I know for sure that someone > >test it before I make it public, so if you have a cyrix K6/II or III > >with powernow! capability and you want to test please write me. > > Excellent, your work looks really nice. Just a few comments: > > Please remove my name from the copyright. You did all the work and I > just provided suggestions. > > Both drivers should just add a child called "powernow" instead of > k*_powernow. You can't have k7 and k8 powernow on the same system, > right? They can announce what hardware they are via device_set_desc() > which it appears you're already doing. Also, DRIVER_MODULE for them > should not have "pn8" as the first arg. I got a message like "cpu/pn already exist" or so if I do that. That's why I have to do a pn7/pn8 distinction for the DRIVER_MODULE. > You can remove the check for a freq >= 0xffff. I committed this to > acpi_perf already. Done. > > Have you tested the ACPI attachment for this? Does it work ok? I'm > just curious because I don't have hw to test acpi-assisted probing. This work like a charm, or else I would have send to you a complaint already ;) > It looks like these can be merged pretty easily. Once you've done that, > I'd like to import this driver. I'm not sure if I will have time right now to do so. I will do this this week-end before my vacation. It's also why I have send this email to acpi- before that merge so that people can atleast have those drivers to play with. -- Bruno Ducrot -- Which is worse: ignorance or apathy? -- Don't know. Don't care.