Re: [PATCH v2] tpm: Apply a sane minimum adapterlimit value for retransmission.

From: Andrew Lunn
Date: Wed Mar 01 2017 - 19:28:25 EST


On Wed, Mar 01, 2017 at 04:36:17PM +0100, Enric Balletbo i Serra wrote:
> From: Bryan Freed <bfreed@xxxxxxxxxxxx>
>
> When the I2C Infineon part is attached to an I2C adapter that imposes
> a size limitation, large requests will fail with -EOPNOTSUPP. Retry
> them with a sane minimum size without re-issuing the 0x05 command
> as this appears to occasionally put the TPM in a bad state.
>
> Signed-off-by: Bryan Freed <bfreed@xxxxxxxxxxxx>
> [rework the patch to adapt to the feedback received]
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx>
> ---
> This is a reworked version of the original patch based on the
> suggestion made by Wolfram Sang to simply fall back to a sane minimum
> when the maximum fails.

Hi Enric

Maybe you should remember that you need to use smaller transfers? If
you don't remember, but use the full size message every time and only
drop back on error, the i2c core is going to log rate limited
messages. By remembering, there will only be one such message in the
log.

Andrew