From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jon Smirl Subject: Re: [PATCH 13/13] IR: Port ene driver to new IR subsystem and enable it. Date: Thu, 29 Jul 2010 22:39:09 -0400 Message-ID: References: <1280456235-2024-1-git-send-email-maximlevitsky@gmail.com> <1280456235-2024-14-git-send-email-maximlevitsky@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: In-Reply-To: <1280456235-2024-14-git-send-email-maximlevitsky@gmail.com> Sender: linux-media-owner@vger.kernel.org To: Maxim Levitsky Cc: lirc-list@lists.sourceforge.net, Jarod Wilson , linux-input@vger.kernel.org, linux-media@vger.kernel.org, Mauro Carvalho Chehab , Christoph Bartelmus List-Id: linux-input@vger.kernel.org On Thu, Jul 29, 2010 at 10:17 PM, Maxim Levitsky wrote: > note that error_adjustment module option is added. > This allows to reduce input samples by a percent. > This makes input on my system more correct. > > Default is 4% as it works best here. > > Note that only normal input is adjusted. I don't know > what adjustments to apply to fan tachometer input. > Maybe it is accurate already. Do you have the manual for the ENE chip in English? or do you read Chinese? Maybe you can figure out why the readings are off by 4%. I suspect that someone has set a clock divider wrong when programming the chip. For example setting the divider for a 25Mhz clock when the clock is actually 26Mhz would cause the error you are seeing. Or they just made a mistake in computing the divisor. It is probably a bug in the BIOS of your laptop. If that's the case you could add a quirk in the system boot code to fix the register setting. -- Jon Smirl jonsmirl@gmail.com