Thread (1 message) 1 message, 1 author, 2014-10-08

Re: [tpmdd-devel] [PATCH 14/16] tpm/tpm_i2c_stm_st33: Fix potential bug in tpm_stm_i2c_send

From: Jason Gunthorpe <hidden>
Date: 2014-10-08 16:27:54

On Wed, Oct 08, 2014 at 09:38:39AM +0200, Christophe Ricard wrote:
   I would just add an additional comment to my previous message.
   After reviewing Documentation/stable_kernel_rules.txt, it looks like it
   is *not* necessary to send this patch marked as stable because of the
   following rule:
   "- It must fix a real bug that bothers people (not a, "This could be a
   problem..." type thing)."
   As argued previously this patch is more an algorithm fix luckily never
   seen with the current ST33 I2C TPM devices.
   I looks like a "This could be a problem..."
   Do you agree with this ?
Yes, please clarify your patch description with something like:

'This never happens in practice because the burst count is XXX bytes
and the largest TPM command is YYY bytes.'

Jason
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.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