Hi, thanks for your patch > static struct pnp_device_id tpm_pnp_tbl[] = { > {"PNP0C31", 0}, /* TPM */ > {"ATM1200", 0}, /* Atmel */ > @@ -835,9 +834,12 @@ static struct pnp_driver tis_pnp_driver = { > .name = "tpm_tis", > .id_table = tpm_pnp_tbl, > .probe = tpm_tis_pnp_init, > - .suspend = tpm_tis_pnp_suspend, > - .resume = tpm_tis_pnp_resume, > .remove = tpm_tis_pnp_remove, > +#ifdef CONFIG_PM_SLEEP > + .driver = { > + .pm = &tpm_tis_pm, > + }, > +#endif > }; I don't think the #if CONFIG_PM_SLEEP is required here. Thanks, Peter -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html