From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anton Vorontsov Subject: Re: [PATCH 2/2] i2c: MPC8349E-mITX Power Management and GPIO expander driver Date: Fri, 10 Oct 2008 19:16:25 +0400 Message-ID: <20081010151625.GA7462@oksana.dev.rtsoft.ru> References: <20080923141300.GB31262@oksana.dev.rtsoft.ru> <683C10AE-E5F2-4196-B2C7-1E1CC317EC24@kernel.crashing.org> <20081010144545.GA956@oksana.dev.rtsoft.ru> Reply-To: avorontsov@ru.mvista.com Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linuxppc-dev-bounces+glppd-linuxppc64-dev=m.gmane.org@ozlabs.org Errors-To: linuxppc-dev-bounces+glppd-linuxppc64-dev=m.gmane.org@ozlabs.org To: Kumar Gala Cc: Jean Delvare , i2c@lm-sensors.org, Ben Dooks , linuxppc-dev@ozlabs.org List-Id: linux-i2c@vger.kernel.org On Fri, Oct 10, 2008 at 09:58:44AM -0500, Kumar Gala wrote: [...] >>> is the plan to connect ppc_md.machine_shutdown() with this? >> >> You mean poweroff? It's already connected. >> >> + /* XXX: this is potentially racy, but there is no lock for >> ppc_md */ >> + if (!ppc_md.power_off) { >> + glob_mcu = mcu; >> + ppc_md.power_off = mcu_power_off; >> + dev_info(&client->dev, "will provide power-off >> service\n"); >> + } > > > uuh, yeah.. poweroff :) > > and look at that. Is this in Jean's queue? Nope. Here's what Jean wrote: On Wed, Oct 01, 2008 at 01:22:48PM +0200, Jean Delvare wrote: > I do not have the time to review these patches (and, honestly, have no > interest in them.) So I will not merge them but I have no objection to > them being merged by somebody else. So.. I think it could easily go through the powerpc tree. -- Anton Vorontsov email: cbouatmailru@gmail.com irc://irc.freenode.net/bd2