Re: [PATCH] i2c: remove deprecated callbacks

From: Jean Delvare
Date: Fri Sep 02 2011 - 02:28:38 EST


On Fri, 02 Sep 2011 09:24:18 +1000, Benjamin Herrenschmidt wrote:
> On Thu, 2011-09-01 at 10:52 +0200, Jean Delvare wrote:
> > Furthermore, you didn't even check if the deprecated callbacks were
> > still in use. It turns out that there are 9 drivers still using one of
> > them (6 macintosh drivers and 3 sound drivers) so plain removing it
> > will cause build breakages. We obviously don't want to do that, so I am
> > not going to apply your patch (nor mine) for now.
> >
> > I know that Benjamin Herrenschmidt is working on converting (some of)
> > the macintosh drivers. I don't have any news for some time though. Ben,
> > are you done with the conversion by now? It would be great to get it
> > merged in kernel 3.2.
>
> I'm just bogged down by too many things, including quite a bit of travel
> lately. I have some mostly untested patches converting part of windfarm
> (not all of it yet), and therm_adt746x and a patch to add the generic
> code to register i2c devices from device-tree properly on powermacs.
>
> I'm missing a couple more drivers & bits of windfarm and therm_pm72
> which I'd like to use the opportunity to convert into a windfarm module.
>
> Then there's some sound drivers etc...
>
> I am travelling again tomorrow for 2 weeks, and after that a month in
> europe, tho I'll have 10 days in the middle where I _might_ be able to
> make progress on that. Else it will have to wait for november.

Thanks for the update, Ben. November is totally fine with me, the i2c
subsystem conversion started 5 years ago or so, we can wait a couple
additional months ;) I definitely prefer that you take the time to do
it properly and we can apply the same for the remaining drivers.

If you want, we may be able to spend half an hour face to face in
Prague in October to check what you've done and if there are blocking
issues remaining.

--
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/