On Wed, Oct 08, 2014 at 11:14:40PM +0200, Christophe Henri RICARD wrote: > For your information, the lpcpd pin is named like this as per > PCClientTPMSpecification is also known as TPMSTB on this product. > The TPMSTB pin can be used in order to support D1 and D2 power > management state. I am actually sorry to refer to Microsoft website > here ;) : So, having the option to enter D1/D2 through that pin in the driver seems fine, and it should just be enabled if the driver is told what that pin is. No need for a module parameter, and no need for the driver to fail probe if it isn't told how to access that GPIO. However.. suspend/resume are called from the broader PM framework and a platform might arrange things so the TPM is put into a deeper sleep than D1/D2 (ie entirely powered off because the CPU was put into suspend-to-ram and the peripheral power converter was switched off) Thus the driver really must save the TPM state at suspend, and if the volatile state was wiped after resume then it must be restored from the saved state. Jason -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
