Re: [PATCH] i2c: remove deprecated callbacks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 1 Sep 2011 17:12:00 +0800, Américo Wang wrote:
> On Thu, Sep 1, 2011 at 4:52 PM, Jean Delvare <khali@xxxxxxxxxxxx> 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.
> 
> Hmm, sorry that I didn't know this, maybe we can defer the removal
> of these callbacks?

We have to, obviously. And given that this is 50% of the patch, there's
little point in applying the rest now.

> > (...)
> > If you want to help, then instead of sending a patch which I already
> > have and virtually everybody could have written, please get the
> > remaining 3 drivers converted to the current i2c binding model:
> >  sound/ppc/keywest.c
> >  sound/aoa/codecs/onyx.c
> >  sound/aoa/codecs/tas.c
> 
> Sure, if I can... so how do we convert them? Any examples?

If it was trivial, someone would have done it already... I converted
most drivers a couple years ago, but the remaining ones only run on
systems I don't have and converting them properly would take intimate
knowledge of the systems in question and preferably actual hardware for
testing.

You can read the possible replacements for the current deprecated
implementations:
  Documentation/i2c/instantiating-devices

At the moment I am essentially waiting to see when Ben is headed, as
hopefully the sound drivers can follow the same path. If you want to
help Ben with code review or testing (if you can) this would be great.

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


[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux