* [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