From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Wed, 08 May 2013 06:11:33 +0200 Subject: [U-Boot] [PATCH v3 8/9] tegra: i2c: Enable new CONFIG_SYS_I2Cframework In-Reply-To: <518915EF.2080808@wwwdotorg.org> References: <1367668903-29653-1-git-send-email-hs@denx.de> <1367668903-29653-9-git-send-email-hs@denx.de> <5187FFAF.6000802@wwwdotorg.org> <2292893.q39nOzQG2d@fb07-iapwap2.physik.uni-giessen.de> <518915EF.2080808@wwwdotorg.org> Message-ID: <5189D075.9060407@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Stephen, Am 07.05.2013 16:55, schrieb Stephen Warren: > On 05/07/2013 02:01 AM, Marc Dietrich wrote: >> Hi, >> >> Am Montag, 6. Mai 2013, 13:08:31 schrieb Stephen Warren: >>> On 05/04/2013 06:01 AM, Heiko Schocher wrote: >>>> From: Simon Glass >>>> >>>> This enables CONFIG_SYS_I2C on Tegra, updating existing boards and the >>>> Tegra i2c driver to support this. >>>> >>>> include/configs/beaver.h | 5 ++- >>>> include/configs/cardhu.h | 3 +- >>>> include/configs/dalmore.h | 3 +- >>>> include/configs/seaboard.h | 5 ++- >>>> include/configs/trimslice.h | 5 ++- >>>> include/configs/whistler.h | 5 ++- >>> >>> There are a lot more Tegra boards than just those. Shouldn't they all be >>> updated? You also didn't Cc the Tegra maintainer - I have done on this mail. >> >> not all boards use I2C up to now, to the patch seems to change only the boards >> which do. So this is ok for now. The question is if CONFIG_SYS_I2C could be >> moved to tegra_common or so because the device-tree disables I2C be default. > > Yes, we should do that for all the devices that are now > instantiated/configured using device tree. But, we can do that as a > separate patch series after this. Yes, that makes sense. bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany