linux-i2c.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Daniel Kurtz <djkurtz@chromium.org>
To: khali@linux-fr.org, ben-linux@fluff.org, seth.heasley@intel.com,
	ben@decadent.org.uk, David.Woodhouse@intel.com
Cc: linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org,
	olofj@chromium.org, dlaurie@chromium.org, bleung@chromium.org,
	Daniel Kurtz <djkurtz@chromium.org>
Subject: [PATCH 0/3] i2c: i801: enable irq
Date: Wed, 14 Dec 2011 15:56:29 +0800	[thread overview]
Message-ID: <1323849392-20413-1-git-send-email-djkurtz@chromium.org> (raw)

This set of patches enables the Intel PCH SMBus controller interrupt.

The Interrupt can signify:
 INTR - the end of a complete transaction
 DEV_ERR - that a device did not ACK a transaction
 BYTE_DONE - the completion of a single byte during a byte-by-byte transaction

This patchset arrives with the following caveats:

 1)  It has only been tested with a Cougar Point (Intel 6 Series PCH) SMBus
controller, so the irq is only enabled for that chip type.

 2) It has not been tested with any devices that do transactions that use the
    PEC.  In fact, I believe that an additional small patch would be required
    to the driver working correctly in interrupt mode with PEC.

 3) It has not been tested in SMBus Slave mode.

 4) It has not been tested with SMI#-type interrupts.

 5) The BIOS has to configure the PCH SMBus IRQ properly.

 6) It has not been tested with a device that does byte-by-byte smbus (non-i2c)
    reads.

 7) It has not been tested with smbus 'process call' transactions.

If would be very helpful if somebody could help test on other chipsets, with
a PEC device, or on additional BIOS that woudl be very helpful.

In the meantime, the interrupt behavior is only enabled on the Cougar Point,
and even here, it can be completely disabled with the "Interrupt" feature like
other advanced features of the driver.

Daniel Kurtz (3):
  i2c: i801: refactor i801_block_transaction_byte_by_byte
  i2c: i801: enable irq for i801 smbus transactions
  i2c: i801: enable irq for byte_by_byte transactions

 drivers/i2c/busses/i2c-i801.c |  193 ++++++++++++++++++++++++++++++++++++-----
 1 files changed, 170 insertions(+), 23 deletions(-)

-- 
1.7.3.1

             reply	other threads:[~2011-12-14  7:56 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-14  7:56 Daniel Kurtz [this message]
     [not found] ` <1323849392-20413-1-git-send-email-djkurtz-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2011-12-14  7:56   ` [PATCH 1/3] i2c: i801: refactor i801_block_transaction_byte_by_byte Daniel Kurtz
2011-12-14  7:56   ` [PATCH 2/3] i2c: i801: enable irq for i801 smbus transactions Daniel Kurtz
2011-12-14  7:56 ` [PATCH 3/3] i2c: i801: enable irq for byte_by_byte transactions Daniel Kurtz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1323849392-20413-1-git-send-email-djkurtz@chromium.org \
    --to=djkurtz@chromium.org \
    --cc=David.Woodhouse@intel.com \
    --cc=ben-linux@fluff.org \
    --cc=ben@decadent.org.uk \
    --cc=bleung@chromium.org \
    --cc=dlaurie@chromium.org \
    --cc=khali@linux-fr.org \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=olofj@chromium.org \
    --cc=seth.heasley@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).