From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 49D3AB6FEA for ; Thu, 26 Apr 2012 07:58:58 +1000 (EST) Message-ID: <1335391128.21961.55.camel@pasglop> Subject: Re: [PATCH 00/15] PowerMac i2c API conversions & windfarm updates From: Benjamin Herrenschmidt To: Andreas Schwab Date: Thu, 26 Apr 2012 07:58:48 +1000 In-Reply-To: References: <1334823416-9138-1-git-send-email-benh__1987.51832726902$1334826600$gmane$org@kernel.crashing.org> <1335343125.21961.33.camel@pasglop> <1335388465.21961.35.camel@pasglop> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Cc: khali@linux-fr.org, linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wed, 2012-04-25 at 23:54 +0200, Andreas Schwab wrote: > > This fuzzing is what makes it much more annoying. > Interesting, I didn't think that couple of percent of pwm would be noticable (there's probably too much ambiant noise in the lab here for me to notice). What if you just comment out the tickle code ? In theory it's only needed if we haven't changed any fan speed for a long time (the FCU can then timeout and assume we aren't driving it, ramping up all fans to full speed). I could implement that a bit more intelligently. Cheers, Ben.