On Tue, Jun 21, 2011 at 11:27 AM, Stephen Warren <swarren@xxxxxxxxxx> wrote: > 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"? Yes, sounds appropriate to me. g. > > The code looks fine. > > -- > nvpublic > > -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. -- 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