From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Mon, 09 Feb 2009 12:28:57 +0100 Subject: [U-Boot] PATCH 0/8 Multi-adapter multi-bus I2C In-Reply-To: References: Message-ID: <49901379.2050809@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 ksi, ksi at koi8.net wrote: > This is the first attempt of multi-adapter multi-bus I2C subsystem rework > for U-Boot. > > This batch deals with fsl_i2c and soft_i2c drivers. All (hopefully :) > > relevant parts are changed -- config files for all boards affected, lib_xxx, > board/* etc. > > Please see illustrative section of MPC8548CDS.h config file for an example > for 2 soft_i2c, 2 fsl_i2c, 1 sm502_i2c adapters and 7 busses some of them > connected via I2C multiplexers. > > All comments, suggestions etc. are very welcome. > Common comments: - you mixed Coding Style cleanup with functional changes in one patch, please don't do this - I also think, we should have a u-boot-i2c.git or something like that for this big change. So your patches go first in u-boot-i2c.git, and everybody can try out your changes. And if all goes OK, then it is ready for mainline. What Do others think? > I'm going to move _ALL_ I2C drivers to drivers/i2c and rework all the > configs and other relevant stuff in next batches. Then I will do another run > to completely remove CONFIG_NEW_I2C and get rid of CONFIG_HARD_I2C. > > The patches are in next 8 emails. > It looks like to me, that this "8 patches" is only one big patch, that you just splitted in 8 EMails, don't do this. Please try to make real n patches from this big one (for example, the changes in fsl_i2c.c in one extra patch). > Signed-off-by: Sergey Kubushyn > Why here a Signed-off-by? For what? bye Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany