From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Thu, 20 Jan 2011 11:27:01 +0100 Subject: [U-Boot] [PATCH V2 05/11] I2C: mxc_i2c: address failure with mx35 processor In-Reply-To: <20110120093029.E03F4D301D7@gemini.denx.de> References: <1295513194-16158-1-git-send-email-sbabic@denx.de> <1295513194-16158-6-git-send-email-sbabic@denx.de> <20110120093029.E03F4D301D7@gemini.denx.de> Message-ID: <4D380DF5.5090504@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 On 01/20/2011 10:30 AM, Wolfgang Denk wrote: > Dear Stefano Babic, > > In message <1295513194-16158-6-git-send-email-sbabic@denx.de> you wrote: >> There is sporadic failures when more as one I2C slave >> is on the bus and the processor tries to communicate >> with more as one slave. >> The problem was seen on a mx35pdk (two I2C slaves, >> PMIC controller and CAN/RTC chip). > > Please also mention which approach the patch takes to fix this > problem. Ok > > Checkpatch says: > > total: 1 errors, 1 warnings, 166 lines checked I know, but I found the construct ok. checkpatch complains against using assignment in if function, even if it is large used. Again, as we have a tool to check patches, I will change to remove this issue. > > Please have a look. > >> -#ifdef DEBUG >> -#define DPRINTF(args...) printf(args) >> -#else >> -#define DPRINTF(args...) >> -#endif > > Ah. Please forget the related comment in my previous message. ...and you please forget my comment to your comment... > > But please mention this change in the commit message. I will do it. Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de =====================================================================