Re: [i2c] [2.6 patch] some overdue I2C driver removal

[Jean Delvare]

On Wed, 28 Nov 2007 12:43:00 -0500, Jon Smirl wrote:
> On 11/28/07, Lennart Sorensen <lsorense@xxxxxxxxxxxxxxxxxxx> 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.

There's no "i2c web site". And no, I'm not breaking people's setup for
fun. The drivers in question are small, I don't care that much if they
stick around. If we can remove them (as is the case for i2c-ixp4xx),
great, let's go, but if not, there's really no reason to make
gratuitous trouble.

The patch I have in my tree now removes i2c-ixp4xx and leaves
i2c-ixp2000 and scx200_i2c alone. They can always be removed later if
the respective platforms are updated to use the generic GPIO layer.

-- 
Jean Delvare
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/