public inbox for linux-i2c@vger.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: Kumar Gala <galak@kernel.crashing.org>
Cc: Darius <augulis.darius@gmail.com>,
	linux-arm-kernel@lists.arm.linux.org.uk, i2c@lm-sensors.org,
	Ben Dooks <ben-linux-arm@fluff.org>
Subject: Re: [PATCH V3] I2C driver for IMX
Date: Thu, 10 Apr 2008 12:01:41 +0200	[thread overview]
Message-ID: <20080410120141.4d220d27@hyperion.delvare> (raw)
In-Reply-To: <Pine.LNX.4.64.0804092223540.10428@gate.crashing.org>

On Wed, 9 Apr 2008 22:25:39 -0500 (CDT), Kumar Gala wrote:
> 
> On Thu, 10 Apr 2008, Ben Dooks wrote:
> 
> > On Tue, Apr 08, 2008 at 11:01:25AM +0300, Darius wrote:
> > > changes:
> > > 1. Removed hardcoded constants
> > > 2. Fixed bug in chip hardware
> > > 3. Added i2c device for mx1ads
> > >
> > > Tested with mx1ads v2.0 board and LCD on I2C interface driver
> > >
> > > Signed-off-by: Darius Augulis <augulis.darius@gmail.com>
> > > ---
> >
> > Hmm, this hasn't appeared on the i2c list where it should be being
> > reviewed before submission. Not bothered to look or snip the rest.
> 
> looking at this it seems the HW is identical to the driver handled by
> i2c-mpc.c.  (which is used on Motorola and now Freescale PPC chips).

Then I guess that we want to add support to the i2c-mpc driver rather
than adding another driver to the kernel tree?

-- 
Jean Delvare

-------------------------------------------------------------------
List admin: http://lists.arm.linux.org.uk/mailman/listinfo/linux-arm-kernel
FAQ:        http://www.arm.linux.org.uk/mailinglists/faq.php
Etiquette:  http://www.arm.linux.org.uk/mailinglists/etiquette.php

  reply	other threads:[~2008-04-10 10:01 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-08  8:01 [PATCH V3] I2C driver for IMX Darius
2008-04-09 23:28 ` Ben Dooks
     [not found]   ` <20080409232834.GG1197-SMNkleLxa3Z6Wcw2j4pizdi2O/JbrIOy@public.gmane.org>
2008-04-10  3:25     ` Kumar Gala
2008-04-10 10:01       ` Jean Delvare [this message]
     [not found]         ` <20080410120141.4d220d27-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2008-04-10 10:09           ` Kumar Gala
2008-04-10 10:33         ` Darius
2008-04-10 11:33           ` Juergen Beisert
2008-04-10 12:24             ` Russell King - ARM Linux
2008-04-10 12:31               ` Darius
2008-04-14  0:55                 ` Pavel Pisa
2008-04-15 11:03 ` Darius

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=20080410120141.4d220d27@hyperion.delvare \
    --to=khali@linux-fr.org \
    --cc=augulis.darius@gmail.com \
    --cc=ben-linux-arm@fluff.org \
    --cc=galak@kernel.crashing.org \
    --cc=i2c@lm-sensors.org \
    --cc=linux-arm-kernel@lists.arm.linux.org.uk \
    /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