public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: Lennert Buytenhek <buytenh@wantstofly.org>
Cc: Lennart Sorensen <lsorense@csclub.uwaterloo.ca>,
	Adrian Bunk <bunk@kernel.org>,
	i2c@lm-sensors.org, linux-kernel@vger.kernel.org,
	Russell King <rmk+kernel@arm.linux.org.uk>
Subject: Re: [2.6 patch] some overdue I2C driver removal
Date: Wed, 28 Nov 2007 20:07:29 +0100	[thread overview]
Message-ID: <20071128200729.121dd793@hyperion.delvare> (raw)
In-Reply-To: <20071128183720.GB697@xi.wantstofly.org>

On Wed, 28 Nov 2007 19:37:20 +0100, Lennert Buytenhek wrote:
> On Wed, Nov 28, 2007 at 11:23:57AM +0100, Jean Delvare wrote:
> (...)
> > 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?
> 
> No plans at this point from my side.
> 
> I'm not entirely sold on the GPIO API, but nevertheless I expect
> that ixp2000 will be converted sooner rather than later.
> 
> If you don't want people to use i2c-ixp2000 as an example for new
> code, maybe you can stick in a comment at the top of the file that
> says that what i2c-ixp2000 does isn't the Preferred(TM) way of doing
> it?

The driver is tagged DEPRECATED in Kconfig, that should be enough. Not
to mention that any new i2c bus driver has to go through me so I'd
intercept the offender anyway.

> (...)
> Sorry, I seem to have missed the addition of i2c-ixp2000 to
> feature-removal-schedule.txt.  I would have spoken up earlier if I
> had been aware of it being in there.
> 
> It seems odd to me to advocate removal of something which works fine
> and for which no functional replacement is available just because it
> doesn't conform to the Latest And Greatest API(TM).  That's like
> suggesting that Jan 1st 2009 we should rm -f all .c files that still
> reference the big kernel lock, or rm -f all ARM ports that don't
> use the clocksource/clockevent mechanism yet (which includes
> ixp2000 as well.)
> 
> Let's just remove i2c-ixp2000 when the GPIO API conversion happens, ok?

That's the plan, yes.

-- 
Jean Delvare

      reply	other threads:[~2007-11-28 19:07 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-24 16:25 [2.6 patch] some overdue I2C driver removal 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       ` [i2c] " Jon Smirl
2007-11-28 17:53         ` 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 [this message]

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=20071128200729.121dd793@hyperion.delvare \
    --to=khali@linux-fr.org \
    --cc=bunk@kernel.org \
    --cc=buytenh@wantstofly.org \
    --cc=i2c@lm-sensors.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