Re: [PATCH v2] tpm, tpm_tis: Workaround failed command reception on Infineon devices

2025-04-04 Thread Jonathan McDowell
Jarkko, I've realised I've somehow introduced a typo in the patch below that means it doesn't fire correctly; I'm not sure how this happened as my local copy I was testing on is definitely correct. Would you like a one line fix up patch, or can you manually fix it up in your tree? This hunk:

Re: [PATCH v2] tpm, tpm_tis: Workaround failed command reception on Infineon devices

2025-03-22 Thread Jarkko Sakkinen
On Fri, Mar 21, 2025 at 04:49:15PM +, Jonathan McDowell wrote: > Jarkko, I've realised I've somehow introduced a typo in the patch below that > means it doesn't fire correctly; I'm not sure how this happened as my local > copy I was testing on is definitely correct. Would you like a one line fi

Re: [PATCH v2] tpm, tpm_tis: Workaround failed command reception on Infineon devices

2025-03-11 Thread Paul Menzel
Dear Jonathan, Some nits, should you resend. Feel free to ignore. The verb *work around* is spelled with a space. Am 10.03.25 um 13:19 schrieb Jonathan McDowell: From: Jonathan McDowell Some Infineon devices have a issue where the status register will get stuck with a quick REQUEST_USE / C

Re: [PATCH v2] tpm, tpm_tis: Workaround failed command reception on Infineon devices

2025-03-11 Thread Jarkko Sakkinen
On Mon, Mar 10, 2025 at 12:19:55PM +, Jonathan McDowell wrote: > From: Jonathan McDowell > > Some Infineon devices have a issue where the status register will get > stuck with a quick REQUEST_USE / COMMAND_READY sequence. This is not > simply a matter of requiring a longer timeout; the work a

[PATCH v2] tpm, tpm_tis: Workaround failed command reception on Infineon devices

2025-03-10 Thread Jonathan McDowell
From: Jonathan McDowell Some Infineon devices have a issue where the status register will get stuck with a quick REQUEST_USE / COMMAND_READY sequence. This is not simply a matter of requiring a longer timeout; the work around is to retry the command submission. Add appropriate logic to do this i