mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Jon Smirl" <jonsmirl@gmail.com>
To: "Lennart Sorensen" <lsorense@csclub.uwaterloo.ca>
Cc: "Jean Delvare" <khali@linux-fr.org>,
	"Russell King" <rmk+kernel@arm.linux.org.uk>,
	"Adrian Bunk" <bunk@kernel.org>,
	i2c@lm-sensors.org, "Lennert Buytenhek" <buytenh@wantstofly.org>,
	linux-kernel@vger.kernel.org
Subject: Re: [i2c] [2.6 patch] some overdue I2C driver removal
Date: Wed, 28 Nov 2007 12:43:00 -0500	[thread overview]
Message-ID: <9e4733910711280943t75564912m805cfdb2fe099528@mail.gmail.com> (raw)
In-Reply-To: <20071128173022.GD2308@csclub.uwaterloo.ca>

On 11/28/07, Lennart Sorensen <lsorense@csclub.uwaterloo.ca> wrote:
> On Wed, Nov 28, 2007 at 11:23:57AM +0100, Jean Delvare wrote:
> > There's no driver to write (i2c-gpio is the driver). Instead, the
> > scx200 platform needs to implement the standard GPIO API. I don't think
> > that it has happened yet, and last time this was discussed, someone
> > (can't remember who, sorry), complained that the GPIO API "sucked" and
> > that the scx200 platform would not be updated to use it. I didn't buy
> > the claim due to a lack of argumentation and the fact that the GPIO
> > infrastructure seems to work well enough for many other platforms.
> >
> > As far as I can see, the ixp2000 platform also doesn't implement the
> > standard GPIO API yet, so of the 3 drivers that are about to be
> > removed, only i2c-ixp4xx can be removed without functionality loss at
> > the moment. Lennert, Russell, are there any plans to convert the
> > ixp2000 platform to use the generic GPIO layer?
> >
> > Maybe I shouldn't have added this entry in feature-removal-schedule.txt
> > in the first place: these drivers should ideally be dropped in favor of
> > i2c-gpio, but it can only happen for platforms that implement the
> > standard GPIO API. As I am not the one who will convert these
> > platforms, and some of them might as well never be converted (I don't
> > know how active they are nowadays), there's not much I can do.
>
> Well if I get a chance I might try to do it for the scx200 at some point
> (since I do use it) although given the current i2c driver works just
> fine and I have no need for the rest of the gpio lines what so ever
> there isn't that much insentive to do so yet.
>
> Removing the existing i2c driver would seem like a mistake at this time
> though.

Hasn't their been a one year notice that these drivers were going to
be removed? How about removing them and putting them into a tarball on
the i2c web site with a note explaining about how they need to be
updated to use the gpio driver? Then if someone really wants to use
the old drivers they can use them out of tree.

-- 
Jon Smirl
jonsmirl@gmail.com

  reply	other threads:[~2007-11-28 17:43 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-24 16:25 Adrian Bunk
2007-10-24 16:43 ` Lennart Sorensen
2007-11-28 10:23   ` Jean Delvare
2007-11-28 17:30     ` Lennart Sorensen
2007-11-28 17:43       ` Jon Smirl [this message]
2007-11-28 17:53         ` [i2c] " Jean Delvare
2007-11-28 17:54         ` Russell King
2007-11-28 19:04         ` Lennart Sorensen
2007-11-28 17:41     ` Russell King
2007-11-28 18:37     ` Lennert Buytenhek
2007-11-28 19:07       ` 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=9e4733910711280943t75564912m805cfdb2fe099528@mail.gmail.com \
    --to=jonsmirl@gmail.com \
    --cc=bunk@kernel.org \
    --cc=buytenh@wantstofly.org \
    --cc=i2c@lm-sensors.org \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lsorense@csclub.uwaterloo.ca \
    --cc=rmk+kernel@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

all inboxes | Powered by JetHome®