From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 05C48DDDED for ; Mon, 20 Oct 2008 16:22:25 +1100 (EST) Subject: Re: [PATCH] Add the of_find_i2c_device_by_node function From: Benjamin Herrenschmidt To: avorontsov@ru.mvista.com In-Reply-To: <20081019220352.GA19539@oksana.dev.rtsoft.ru> References: <20081019140040.32281.65209.stgit@terra> <20081019212034.GA8224@oksana.dev.rtsoft.ru> <9e4733910810191450t36cbe120y106d18f71cb3163b@mail.gmail.com> <20081019220352.GA19539@oksana.dev.rtsoft.ru> Content-Type: text/plain Date: Mon, 20 Oct 2008 16:19:56 +1100 Message-Id: <1224479996.7654.130.camel@pasglop> Mime-Version: 1.0 Cc: linuxppc-dev@ozlabs.org, i2c@lm-sensors.org Reply-To: benh@kernel.crashing.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, 2008-10-20 at 02:03 +0400, Anton Vorontsov wrote: > > This is the original mail.... > > http://lkml.org/lkml/2008/6/13/290 > > http://lkml.org/lkml/2008/6/12/8 > > I can't find the ones patching i2c. > > Well, I didn't disagree with the change. I think this is good change > (and good catch, btw). > > Just mention it in the patch description (or better yet, you could send > a separate patch for this particular issue, it seems quite serious). Yes, it should most definitely be a separate patch. Cheers, Ben.