Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757210Ab1FUR2L (ORCPT ); Tue, 21 Jun 2011 13:28:11 -0400 Received: from hqemgate04.nvidia.com ([216.228.121.35]:19868 "EHLO hqemgate04.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757055Ab1FUR2J (ORCPT ); Tue, 21 Jun 2011 13:28:09 -0400 X-PGP-Universal: processed; by hqnvupgp05.nvidia.com on Tue, 21 Jun 2011 10:27:59 -0700 From: Stephen Warren To: John Bonesio , Ken Radtke , "lucas.demarchi@profusion.mobi" , "devicetree-discuss@lists.ozlabs.org" , "linux-kernel@vger.kernel.org" , "grant.likely@secretlab.ca" , Rakesh Iyer , "linux-i2c@vger.kernel.org" , "ben-linux@fluff.org" , "khali@linux-fr.org" Date: Tue, 21 Jun 2011 10:27:57 -0700 Subject: RE: [PATCH] ARM: Tegra: i2c driver Thread-Topic: [PATCH] ARM: Tegra: i2c driver Thread-Index: AcwvqXAoR1eTTdk5ShmLzw20RAjhtwAjrsCw Message-ID: <74CDBE0F657A3D45AFBB94109FB122FF049D2EB8AD@HQMAIL01.nvidia.com> References: <20110621002318.8086.48367.stgit@riker> In-Reply-To: <20110621002318.8086.48367.stgit@riker> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id p5LHSKZI018258 Content-Length: 588 Lines: 14 John Bonesio wrote at Monday, June 20, 2011 6:23 PM: > This patch modifies the tegra i2c driver so that it can be initiailized > using the device tree along with the devices connected to the i2c bus. The patch title "ARM: Tegra: i2c driver" doesn't really describe what the patch does and has the wrong tag; It sounds like adding a new driver. Perhaps "i2c: Tegra: Add DeviceTree support"? The code looks fine. -- nvpublic ????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?