From: Jean Delvare <khali@linux-fr.org>
To: Lennart Sorensen <lsorense@csclub.uwaterloo.ca>
Cc: Haavard Skinnemoen <hskinnemoen@atmel.com>,
David Brownell <david-b@pacbell.net>,
Bryan Wu <bryan.wu@analog.com>,
Andrew Morton <akpm@linux-foundation.org>,
Deepak Saxena <dsaxena@plexity.net>,
linux-kernel@vger.kernel.org,
Christer Weinigel <wingel@nano-system.com>,
Jordan Crouse <jordan.crouse@amd.com>
Subject: Re: [PATCH v3] Bitbanging i2c bus driver using the GPIO API
Date: Fri, 20 Apr 2007 19:49:33 +0200 [thread overview]
Message-ID: <20070420194933.0813a359@hyperion.delvare> (raw)
In-Reply-To: <20070419205942.GC5577@csclub.uwaterloo.ca>
Hi Lennart,
On Thu, 19 Apr 2007 16:59:42 -0400, Lennart Sorensen wrote:
> On Thu, Apr 19, 2007 at 08:54:13AM +0200, Jean Delvare wrote:
> > The major difference is that the implementation in scx200_i2c is
> > hardware-specific, while the i2c-gpio driver is a generic one, so it's
> > a lot better.
> >
> > What this means is that i2c-gpio obsoletes scx200_i2c, so I am inclined
> > to delete scx200_i2c right away. I'm not even sure anyone still uses it
> > now that scx200_acb has been fixed and is reported to work very well.
> > If anyone really needs to do I2C over GPIO pins on SCx200, this should
> > be reimplemented on top of i2c-gpio.
>
> I use scx200_i2c. The SBC I am using doesn't work with the scx200_acb
> (different pins on the cpu after all). I would love to see it
> reimplemented using the generic interface, since there isn't that many
> lines of code involved as far as I can tell. Well scx200_acb works fine
> on a Geode LX board, but not on a Geode SC1200 board. Unless it has
> been fixed since 2.6.18, then it still doesn't work, but scx200_i2c
> works perfectly fine.
The scx200_acb driver was heavily modified in 2.6.17 and 2.6.18, not
much since then. I am not familiar with the hardware so I can't comment
on which chips are supposed to work and which aren't.
--
Jean Delvare
next prev parent reply other threads:[~2007-04-20 17:49 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-04-14 12:56 [PATCH v3] Bitbanging i2c bus driver using the GPIO API Haavard Skinnemoen
2007-04-14 14:34 ` Haavard Skinnemoen
2007-04-14 17:28 ` Jean Delvare
2007-04-18 17:42 ` Lennart Sorensen
2007-04-19 6:54 ` Jean Delvare
2007-04-19 14:27 ` Jordan Crouse
2007-04-19 20:59 ` [PATCH v3] " Lennart Sorensen
2007-04-20 17:49 ` Jean Delvare [this message]
2007-04-22 15:41 ` Lennart Sorensen
2007-04-22 23:41 ` Jordan Crouse
2007-04-23 9:42 ` [PATCH v3] " Jean Delvare
2007-04-23 14:47 ` Jordan Crouse
2007-04-26 12:56 ` Jean Delvare
2007-04-26 13:29 ` Lennart Sorensen
2007-04-26 13:39 ` Jordan Crouse
2007-04-26 14:03 ` Lennart Sorensen
2007-04-27 8:02 ` Jean Delvare
2007-04-27 14:19 ` Lennart Sorensen
2007-04-27 14:49 ` Jordan Crouse
2007-04-27 18:53 ` Lennart Sorensen
2007-05-06 9:24 ` Jean Delvare
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=20070420194933.0813a359@hyperion.delvare \
--to=khali@linux-fr.org \
--cc=akpm@linux-foundation.org \
--cc=bryan.wu@analog.com \
--cc=david-b@pacbell.net \
--cc=dsaxena@plexity.net \
--cc=hskinnemoen@atmel.com \
--cc=jordan.crouse@amd.com \
--cc=linux-kernel@vger.kernel.org \
--cc=lsorense@csclub.uwaterloo.ca \
--cc=wingel@nano-system.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.