From owner-svn-src-head@FreeBSD.ORG Sat Jun 4 09:45:46 2011 Return-Path: Delivered-To: svn-src-head@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 0AAF3106564A; Sat, 4 Jun 2011 09:45:46 +0000 (UTC) (envelope-from andreast@FreeBSD.org) Received: from smtp.fgznet.ch (mail.fgznet.ch [81.92.96.47]) by mx1.freebsd.org (Postfix) with ESMTP id 715C18FC08; Sat, 4 Jun 2011 09:45:44 +0000 (UTC) Received: from deuterium.andreas.nets (dhclient-91-190-8-131.flashcable.ch [91.190.8.131]) by smtp.fgznet.ch (8.13.8/8.13.8/Submit_SMTPAUTH) with ESMTP id p549jeds022080; Sat, 4 Jun 2011 11:45:41 +0200 (CEST) (envelope-from andreast@FreeBSD.org) Message-ID: <4DE9FEC4.4020507@FreeBSD.org> Date: Sat, 04 Jun 2011 11:45:40 +0200 From: Andreas Tobler User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2.15) Gecko/20110303 Lightning/1.0b2 Thunderbird/3.1.9 MIME-Version: 1.0 To: Henrik Brix Andersen References: <201105291425.p4TEPggZ033861@svn.freebsd.org> <9F1CE0EF-CCA6-4A8A-8FE1-2E593626025A@FreeBSD.org> In-Reply-To: <9F1CE0EF-CCA6-4A8A-8FE1-2E593626025A@FreeBSD.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.64 on 81.92.96.47 Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, Andreas Tobler Subject: Re: svn commit: r222449 - in head/sys: conf dev/iicbus powerpc/conf X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 04 Jun 2011 09:45:46 -0000 Hi, sorry for the delay. On 29.05.11 17:10, Henrik Brix Andersen wrote: > Hi, > > On May 29, 2011, at 16:25, Andreas Tobler wrote: >> Author: andreast >> Date: Sun May 29 14:25:42 2011 >> New Revision: 222449 >> URL: http://svn.freebsd.org/changeset/base/222449 >> >> Log: >> Add a new driver, the ad7417, to read temperatures and voltages on some >> PowerMac's. >> >> Approved by: nwhitehorn (mentor) >> >> Added: >> head/sys/dev/iicbus/ad7417.c (contents, props changed) >> Modified: >> head/sys/conf/files.powerpc >> head/sys/powerpc/conf/GENERIC >> head/sys/powerpc/conf/GENERIC64 >> head/sys/powerpc/conf/NOTES > > Since this driver is powerpc-only (the chip is not powerpc-specific - but this driver depends on OFW?) shouldn't it live somewhere under sys/powerpc/ instead of the generic sys/dev/iicbus/? > > Or perhaps be rewritten to a more generic form? The chip itself is not PowerMac specific. It contains four times the functionality of the ad7418 which is already in there. (dev/iicbus) The ofw is only needed to detect the chip and to wire the sensor place to the right fan property. I think this could be refactored that it is PowerMac only and others could benefit from the driver too. The same would apply for the ds1775.c and the max6690.c. Those chips are not PowerMac only, they are simple i2c temp sensors. To get there, it would really be helpful if there are other people with non PowerMac hardware who would have an interest in these drivers. Then we could sit together, refactor and test. W/o the corresponding hardware it seems a bit difficult for me to test. A few months ago I asked on hackers@ how I could combine those two drivers. I got no feedback. Now we're heading towards 9.0 and we (nwhitehorn@) wanted to get this driver in. We need these drivers to run certain G5 PowerMac's in a silent mode. W/o we have a swarm of bees which makes it very hard to concentrate ;) Gruss, Andreas