* [PATCH v10 3/4] drivers/i2c/busses/i2c-at91.c: add new driver [not found] ` <D99582E5322435468A77E74BB0039E7B1D22784E8B@SRV02.hamburg.garz-fricke.de> @ 2012-04-27 8:04 ` Voss, Nikolaus 2012-04-27 8:36 ` AW: " Carsten Behling 0 siblings, 1 reply; 3+ messages in thread From: Voss, Nikolaus @ 2012-04-27 8:04 UTC (permalink / raw) To: linux-arm-kernel Hi Carsten, Carsten Behling wrote on 2012-04-27: >> INIT_COMPLETION(dev->cmd_complete); >> + dev->transfer_status = 0; >> if (dev->msg->flags & I2C_M_RD) { >> unsigned start_flags = AT91_TWI_START; > > this patch will not work, because you reset 'dev->transfer_status' > before it is evaluated on errors (AT91_TWI_NACK, AT91_TWI_OVRE). it should, because transfer_status is updated by the ISR between wait_for_completion_interruptible_timeout() and the error evaluation. Please try it out. > > P.S. Are you using a RM9200? Seems that this SOC has some problems... > > Yes, our ECO920 uses the RM9200, but we support it with an very old kernel > (2.6.21). I remember there were many problems with I2C. Ok, but good to hear the driver basically works with the RM9200. Niko ^ permalink raw reply [flat|nested] 3+ messages in thread
* AW: [PATCH v10 3/4] drivers/i2c/busses/i2c-at91.c: add new driver 2012-04-27 8:04 ` [PATCH v10 3/4] drivers/i2c/busses/i2c-at91.c: add new driver Voss, Nikolaus @ 2012-04-27 8:36 ` Carsten Behling 2012-05-09 19:07 ` Adrian Yanes 0 siblings, 1 reply; 3+ messages in thread From: Carsten Behling @ 2012-04-27 8:36 UTC (permalink / raw) To: linux-arm-kernel Hi Niko, you a right. Driver works with that patch. Mit freundlichen Gr??en / Best regards Carsten Behling Development Engineer Garz & Fricke GmbH Tempowerkring 2, 21079 Hamburg - Germany Amtsgericht Hamburg HRB 60514 Gesch?ftsf?hrer: Manfred Garz, Matthias Fricke Phone: +49 (0) 40 791 899 - 56 Fax: +49 40 / 791 899 - 39 www.garz-fricke.com -----Urspr?ngliche Nachricht----- Von: Voss, Nikolaus [mailto:N.Voss at weinmann.de] Gesendet: Freitag, 27. April 2012 10:05 An: Carsten Behling Cc: Marc-Oliver Westerburg; 'devel at ayanes.com'; 'Wolfram Sang'; 'linux-arm-kernel at lists.infradead.org'; 'linux-kernel at vger.kernel.org'; 'linux-i2c at vger.kernel.org'; 'Hubert Feurstein' Betreff: RE: [PATCH v10 3/4] drivers/i2c/busses/i2c-at91.c: add new driver Hi Carsten, Carsten Behling wrote on 2012-04-27: >> INIT_COMPLETION(dev->cmd_complete); >> + dev->transfer_status = 0; >> if (dev->msg->flags & I2C_M_RD) { >> unsigned start_flags = AT91_TWI_START; > > this patch will not work, because you reset 'dev->transfer_status' > before it is evaluated on errors (AT91_TWI_NACK, AT91_TWI_OVRE). it should, because transfer_status is updated by the ISR between wait_for_completion_interruptible_timeout() and the error evaluation. Please try it out. > > P.S. Are you using a RM9200? Seems that this SOC has some problems... > > Yes, our ECO920 uses the RM9200, but we support it with an very old kernel > (2.6.21). I remember there were many problems with I2C. Ok, but good to hear the driver basically works with the RM9200. Niko ^ permalink raw reply [flat|nested] 3+ messages in thread
* AW: [PATCH v10 3/4] drivers/i2c/busses/i2c-at91.c: add new driver 2012-04-27 8:36 ` AW: " Carsten Behling @ 2012-05-09 19:07 ` Adrian Yanes 0 siblings, 0 replies; 3+ messages in thread From: Adrian Yanes @ 2012-05-09 19:07 UTC (permalink / raw) To: linux-arm-kernel >> Yes, our ECO920 uses the RM9200, but we support it with an very old kernel >> (2.6.21). I remember there were many problems with I2C. I really doubt this, as I indicated in a previous email[1], the RM9200 has a hardware "bug". So either you performed the testing with less than 2 bytes of data and in a really low frequency (<50kHz) or we are missing something. Can you provide more details about which kind of testing you performed? Thanks 1 - http://article.gmane.org/gmane.linux.kernel/1287445/match= ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-05-09 19:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <D99582E5322435468A77E74BB0039E7B1D22784CF0@SRV02.hamburg.garz-fricke.de>
[not found] ` <alpine.DEB.2.00.1204261035060.16439@gatekeeper.vosshq.de>
[not found] ` <D99582E5322435468A77E74BB0039E7B1D22784E8B@SRV02.hamburg.garz-fricke.de>
2012-04-27 8:04 ` [PATCH v10 3/4] drivers/i2c/busses/i2c-at91.c: add new driver Voss, Nikolaus
2012-04-27 8:36 ` AW: " Carsten Behling
2012-05-09 19:07 ` Adrian Yanes
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox