[i2c] [PATCH 00/03] i2c-i801: Add basic interrupt support
khali at linux-fr.org
Sun Aug 10 19:05:22 CEST 2008
On Wed, 23 Jul 2008 18:56:00 +0200, Ivo Manca wrote:
> Hey All,
> It's been a while since I last posted about this matter, but the patch
> is finally there.
> This patch adds basic interrupt support to the i2c-i801 bus driver. This
> means i2c transactions can be completed without useless and pricy
> waiting loops (the polling) and speeds things up.
> Currently only basic transactions are support. The HW PEC or i2c block
> transactions still use polling.
Ah, really great! Thanks for working on this :)
Is there any inherent difficulty to get interrupts to work with HW PEC?
And with I2C block transactions? Or did you just not get around to
> To write this patch, I've extensively used Mark M. Hoffman's i2c-i801
> rewrite of the i2c-i801 driver dating back from 28 Nov 2008. This can be
> found at http://archives.andrew.net.au/lm-sensors/msg28496.html
Yeah, Mark has been traveling a lot in time lately. Unfortunately this
also means that we can't watch this archived post yet, it'll be a few
more months before it shows up! ;)
(Seriously, archives.andrew.net.au is the old list archive and
apparently it has been discontinued, please use lists.lm-sensors.org
> I've tested this patch extensively on a machine provided by the Hague
> University (Thanks Hans! ;) which is ICH5 based, but it needs a lot of
> more testing. So please be welcome to give the patch a shot.
> Since I’m completely new to linux device driver development, interrupts
> and i2c, stupid mistakes can be very well possible. Please use this
> patch with caution and let me know whatever I did wrong :). Oh, and if
> your PC won’t start up anymore, removing power from the PC always worked
> for me ;p.
I've just given it a quick try on my old ICH3-M-based laptop and it
seems to work fine there. The speed improvement compared to the polling
code is very impressive.
I'm going to do more tests on 2 other machines (with ICH5 and ICH7
respectively) and will report when I'm done. A patch review will also
> All patches are made against 2.6.26-rc9
The i2c-i801 driver has changed a lot since then. Any chance you could
respin the patches against 2.6.27-rc2?
> Patch 1 implements the functionality, while patch 2 only adds the
> alignment changes. This is done to make it easier reviewable. Patch 3 is
> quite unrelated and only fixes some checkPatch warnings..
More information about the i2c