Thread (10 messages) 10 messages, 5 authors, 2012-09-11
STALE5026d

[PATCH v11 3/4] drivers/i2c/busses/i2c-at91.c: add new driver

From: ludovic.desroches <hidden>
Date: 2012-08-27 14:00:14
Also in: linux-i2c

Le 08/27/2012 03:32 PM, Voss, Nikolaus a ?crit :
Hi Adian,

Yanes Martinez Adrian wrote on Monday, August 27, 2012 2:46 PM:
quoted
Le 08/27/2012 12:40 PM, Yanes Martinez Adrian a ?crit :
quoted
Hi Ludovic,
quoted
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?
quoted
quoted
I have only checked that I can communicate with i2c devices so the
amount is very low (1-4 bytes).
IIRC it was from >7 bytes when the issues appeared.
quoted
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.
I didn't get your last post  in that thread. In my opinion RM9200 should really
be excluded from being configured with this driver, what's your opinion
Ludovic?
Yes, the use of the TWI controller may be removed for some chips since 
there are issues that are dependent on the controller itself. RM9200 is 
not the only one.

For the moment, I don't have the necessary knowledge about the TWI 
controller to take a decision. I would say it can be done in an other 
patch in order to have your driver included in the mainline quickly 
since it doesn't introduce a breakage.

Regards

Ludovic
Regards,
Niko

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help