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
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
