All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: WANG Cong <xiyou.wangcong@gmail.com>,
	linux-kernel@vger.kernel.org, akpm@linux-foundation.org,
	Randy Dunlap <rdunlap@xenotime.net>,
	Ben Dooks <ben-linux@fluff.org>,
	linux-doc@vger.kernel.org, linux-i2c@vger.kernel.org
Subject: Re: [PATCH] i2c: remove deprecated callbacks
Date: Fri, 2 Sep 2011 08:28:13 +0200	[thread overview]
Message-ID: <20110902082813.310454af@endymion.delvare> (raw)
In-Reply-To: <1314919458.4145.23.camel@pasglop>

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

  parent reply	other threads:[~2011-09-02  6:28 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-01  4:40 [PATCH] i2c: remove deprecated callbacks WANG Cong
2011-09-01  4:40 ` WANG Cong
2011-09-01  8:52 ` Jean Delvare
2011-09-01  9:12   ` Américo Wang
2011-09-01 13:01     ` Jean Delvare
2011-09-01 23:24   ` Benjamin Herrenschmidt
2011-09-01 23:32     ` Benjamin Herrenschmidt
2011-09-01 23:32       ` Benjamin Herrenschmidt
2011-09-02  6:28     ` Jean Delvare [this message]
2011-09-02 20:15       ` Benjamin Herrenschmidt
2011-09-02 20:15         ` Benjamin Herrenschmidt

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=20110902082813.310454af@endymion.delvare \
    --to=khali@linux-fr.org \
    --cc=akpm@linux-foundation.org \
    --cc=ben-linux@fluff.org \
    --cc=benh@kernel.crashing.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rdunlap@xenotime.net \
    --cc=xiyou.wangcong@gmail.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.