From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludovic.desroches Subject: Re: [PATCH v11 3/4] drivers/i2c/busses/i2c-at91.c: add new driver Date: Mon, 27 Aug 2012 14:04:12 +0200 Message-ID: <503B623C.4030304@atmel.com> References: <4f3dcfbf1e8eab3e251ab3558e9e1e9123643928.1345618760.git.n.voss@weinmann.de>, <503B4919.6080306@atmel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1"; Format="flowed" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Yanes Martinez Adrian Cc: "rmallon@gmail.com" , Nikolaus Voss , "nicolas.ferre@atmel.com" , "linux-kernel@vger.kernel.org" , "balbi@ti.com" , "h.feurstein@gmail.com" , "ludovic.desroches" , "linux-i2c@vger.kernel.org" , "ben-linux@fluff.org" , "linux-arm-kernel@lists.infradead.org" , "carsten.behling@garz-fricke.com" List-Id: linux-i2c@vger.kernel.org Hi Adrian, Le 08/27/2012 12:40 PM, Yanes Martinez Adrian a =E9crit : > Hi Ludovic, > >> Tested on at91rm9200, at91sam9261, at91sam9g20 and at91sam9m10 with >> eeprom, temperature sensor and rtc devices (and wolfson chip for >> sam9g20). > > concerning the at91rm9200, what was the amount of bytes you transmitted? > I have only checked that I can communicate with i2c devices so the = amount is very low (1-4 bytes). > Some months ago I performed some testing[1] with the previous version of > the patch and the main conclusion was that the absence of PDC in the > AT91RM9200 made this driver unusable in such platform. > Does the new driver break something? Was it working with the old one? There are some issues with which are not related to the driver but to = the design of the IP. With old I2C versions, using GPIO I2C is recommended. > Please double check this. > > Thanks in advance, > > Adrian. > > 1 - http://thread.gmane.org/gmane.linux.drivers.i2c/10367/focus=3D1287445 > Regards Ludovic