Hi,
On Thu, Apr 17, 2025, at 15:07, Andi Shyti wrote: > Hi Sven, Hector, > > ... > >> +/* >> + * The hardware (supposedly) has a 25ms timeout for clock stretching, thus >> + * use 100ms here which should be plenty. >> + */ >> +#define TRANSFER_TIMEOUT_MS 100 > > Please use the PASEMI prefix here. TRANSFER_TIMEOUT_MS it's not a > naming belonging to this driver. > > 100ms looks a bit too much to me, but if you say it works, then > it works. > The problem here is that we only have very outdated documentation for this hardware and no real idea what changed since Apple bought PASemi and continued using their i2c controller. We know that 10ms (which used to be the original timeout iirc) is not nearly enough and we also know that we need at least 25ms for clock strechting (assuming nothing changed in the past 10+ years). We just bumped it to 100ms to be safe after we very rarely got error reports which we tracked down to timeouts and haven't gotten any reports since. I've addressed all your other comments for v3 which I'll send out in a few minutes. Best, Sven
