On 03/01/2018 02:52 PM, Jarkko Sakkinen wrote:
On Wed, Feb 28, 2018 at 02:18:27PM -0500, Nayna Jain wrote:
In tpm_transmit, after send(), the code checks for status in a loop
Maybe cutting hairs now but please just use the actual function name
instead of send(). Just makes the commit log more useful asset.
Sure, will do.

-               tpm_msleep(TPM_TIMEOUT);
+               tpm_msleep(TPM_TIMEOUT_POLL);
What about just calling schedule()?
I'm not sure what you mean by "schedule()".  Are you suggesting instead of using usleep_range(),  using something with an even finer grain construct?

Thanks & Regards,
     - Nayna


/Jarkko


Reply via email to