Re: [PATCH] tpm_infineon: Fix suspend/resume handler for pnp_driver

[Marcel Selhorst <m.selhorst@sirrix.com>]

Hi,

> I guess it might be tpm_infineon can't suspend/resume properly. However,
> with it, I don't know what happen actually. (it might be, tpm can't
> verify the security issue on suspend/resume process? or can't wakeup
> from tpm? or spend power?  or other?)

On suspend, the TPM framework sends a command "TPM_SaveState" to the chip to
inform it about the upcoming suspend. Therefore it is necessary to have the
function tpm_pm_suspend called (either directly or via pnp). On resume, the
state will automatically be restored by the TPM.

The patch looks good to me, but I haven't had a chance to test it, yet.
As soon as I did, I'll get back to you.

Thanks,
Marcel Selhorst
-- 
Sirrix AG security technologies -- http://www.sirrix.com
Dipl.-Ing. Marcel Selhorst  eMail: m.selhorst@sirrix.com
Tel: +49 (234) 610071-126   Fax: +49 (234) 610071-526
Tel: +49 (681)  95986-126   Fax: +49 (681)  95986-526
Get my public key from keyserver, KeyId: 0x7C9821CC
Fingerprint 4138 E617 E62E 79D3 E663 BE5A 14E7 1CD8 7C98 21CC

Vorstand: Ammar Alkassar (Vors.), Christian Stueble
Vorsitzender des Aufsichtsrates: Prof. Dr. Kai Rannenberg
Sitz der Gesellschaft: Homburg/Saar, HRB 3857 Amtsgericht Saarbruecken

This message may contain confidential and/or privileged information.
If you are not the addressee, you must not use, copy, disclose or
take any action based on this message or any information herein.
If you have received this message in error, please advise the sender
immediately by reply e-mail and delete this message.