tpm/tpm_tis: Add missing ifdef CONFIG_ACPI for pnp_acpi_device
authorPeter Huewe <peterhuewe@gmx.de>
Wed, 21 Jan 2015 23:42:50 +0000 (00:42 +0100)
committerJames Morris <james.l.morris@oracle.com>
Thu, 22 Jan 2015 02:37:33 +0000 (13:37 +1100)
This fixes a build failure if CONFIG_PNP is set but CONFIG_ACPI is not:
drivers/char/tpm/tpm_tis.c: In function ?tpm_tis_pnp_init?:
drivers/char/tpm/tpm_tis.c:912:45: error: invalid type argument of
?->? (have ?int?)
   acpi_dev_handle = pnp_acpi_device(pnp_dev)->handle;

If CONFIG_PNPACPI is not set pnp_acpi_device is defined as 0 and thus
accesing the handle is not possible.

Fixes: 0dc553652102 ("tpm: fix raciness of PPI interface lookup")
Reported-by: Jim Davis <jim.epost@gmail.com>
Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
Signed-off-by: James Morris <james.l.morris@oracle.com>
drivers/char/tpm/tpm_tis.c

index 20a61bc98db8733d37d74308a59c5510100cd964..6725bef7cb969108d638fc90258c0f5256b3f81e 100644 (file)
@@ -908,8 +908,10 @@ static int tpm_tis_pnp_init(struct pnp_dev *pnp_dev,
        if (is_itpm(pnp_dev))
                itpm = true;
 
+#ifdef CONFIG_ACPI
        if (pnp_acpi_device(pnp_dev))
                acpi_dev_handle = pnp_acpi_device(pnp_dev)->handle;
+#endif
 
        return tpm_tis_init(&pnp_dev->dev, acpi_dev_handle, start, len, irq);
 }