]> asedeno.scripts.mit.edu Git - linux.git/blobdiff - drivers/char/tpm/tpm_tis.c
tpm_tis: Consolidate the platform and acpi probe flow
[linux.git] / drivers / char / tpm / tpm_tis.c
index 1807b284326b88f94db762b8bd5d135360d0a16e..b14d4aa97af87afebcf9f84ed9941f8b993fa77c 100644 (file)
@@ -80,6 +80,8 @@ static int has_hid(struct acpi_device *dev, const char *hid)
 
 static inline int is_itpm(struct acpi_device *dev)
 {
+       if (!dev)
+               return 0;
        return has_hid(dev, "INTC0102");
 }
 #else
@@ -89,6 +91,47 @@ static inline int is_itpm(struct acpi_device *dev)
 }
 #endif
 
+#if defined(CONFIG_ACPI)
+#define DEVICE_IS_TPM2 1
+
+static const struct acpi_device_id tpm_acpi_tbl[] = {
+       {"MSFT0101", DEVICE_IS_TPM2},
+       {},
+};
+MODULE_DEVICE_TABLE(acpi, tpm_acpi_tbl);
+
+static int check_acpi_tpm2(struct device *dev)
+{
+       const struct acpi_device_id *aid = acpi_match_device(tpm_acpi_tbl, dev);
+       struct acpi_table_tpm2 *tbl;
+       acpi_status st;
+
+       if (!aid || aid->driver_data != DEVICE_IS_TPM2)
+               return 0;
+
+       /* If the ACPI TPM2 signature is matched then a global ACPI_SIG_TPM2
+        * table is mandatory
+        */
+       st =
+           acpi_get_table(ACPI_SIG_TPM2, 1, (struct acpi_table_header **)&tbl);
+       if (ACPI_FAILURE(st) || tbl->header.length < sizeof(*tbl)) {
+               dev_err(dev, FW_BUG "failed to get TPM2 ACPI table\n");
+               return -EINVAL;
+       }
+
+       /* The tpm2_crb driver handles this device */
+       if (tbl->start_method != ACPI_TPM2_MEMORY_MAPPED)
+               return -ENODEV;
+
+       return 0;
+}
+#else
+static int check_acpi_tpm2(struct device *dev)
+{
+       return 0;
+}
+#endif
+
 static int tpm_tcg_read_bytes(struct tpm_tis_data *data, u32 addr, u16 len,
                              u8 *result)
 {
@@ -141,11 +184,15 @@ static const struct tpm_tis_phy_ops tpm_tcg = {
        .write32 = tpm_tcg_write32,
 };
 
-static int tpm_tis_init(struct device *dev, struct tpm_info *tpm_info,
-                       acpi_handle acpi_dev_handle)
+static int tpm_tis_init(struct device *dev, struct tpm_info *tpm_info)
 {
        struct tpm_tis_tcg_phy *phy;
        int irq = -1;
+       int rc;
+
+       rc = check_acpi_tpm2(dev);
+       if (rc)
+               return rc;
 
        phy = devm_kzalloc(dev, sizeof(struct tpm_tis_tcg_phy), GFP_KERNEL);
        if (phy == NULL)
@@ -158,11 +205,11 @@ static int tpm_tis_init(struct device *dev, struct tpm_info *tpm_info,
        if (interrupts)
                irq = tpm_info->irq;
 
-       if (itpm)
+       if (itpm || is_itpm(ACPI_COMPANION(dev)))
                phy->priv.flags |= TPM_TIS_ITPM_WORKAROUND;
 
        return tpm_tis_core_init(dev, &phy->priv, irq, &tpm_tcg,
-                                acpi_dev_handle);
+                                ACPI_HANDLE(dev));
 }
 
 static SIMPLE_DEV_PM_OPS(tpm_tis_pm, tpm_pm_suspend, tpm_tis_resume);
@@ -171,7 +218,6 @@ static int tpm_tis_pnp_init(struct pnp_dev *pnp_dev,
                            const struct pnp_device_id *pnp_id)
 {
        struct tpm_info tpm_info = {};
-       acpi_handle acpi_dev_handle = NULL;
        struct resource *res;
 
        res = pnp_get_resource(pnp_dev, IORESOURCE_MEM, 0);
@@ -184,14 +230,7 @@ static int tpm_tis_pnp_init(struct pnp_dev *pnp_dev,
        else
                tpm_info.irq = -1;
 
-       if (pnp_acpi_device(pnp_dev)) {
-               if (is_itpm(pnp_acpi_device(pnp_dev)))
-                       itpm = true;
-
-               acpi_dev_handle = ACPI_HANDLE(&pnp_dev->dev);
-       }
-
-       return tpm_tis_init(&pnp_dev->dev, &tpm_info, acpi_dev_handle);
+       return tpm_tis_init(&pnp_dev->dev, &tpm_info);
 }
 
 static struct pnp_device_id tpm_pnp_tbl[] = {
@@ -231,93 +270,6 @@ module_param_string(hid, tpm_pnp_tbl[TIS_HID_USR_IDX].id,
                    sizeof(tpm_pnp_tbl[TIS_HID_USR_IDX].id), 0444);
 MODULE_PARM_DESC(hid, "Set additional specific HID for this driver to probe");
 
-#ifdef CONFIG_ACPI
-static int tpm_check_resource(struct acpi_resource *ares, void *data)
-{
-       struct tpm_info *tpm_info = (struct tpm_info *) data;
-       struct resource res;
-
-       if (acpi_dev_resource_interrupt(ares, 0, &res))
-               tpm_info->irq = res.start;
-       else if (acpi_dev_resource_memory(ares, &res)) {
-               tpm_info->res = res;
-               tpm_info->res.name = NULL;
-       }
-
-       return 1;
-}
-
-static int tpm_tis_acpi_init(struct acpi_device *acpi_dev)
-{
-       struct acpi_table_tpm2 *tbl;
-       acpi_status st;
-       struct list_head resources;
-       struct tpm_info tpm_info = {};
-       int ret;
-
-       st = acpi_get_table(ACPI_SIG_TPM2, 1,
-                           (struct acpi_table_header **) &tbl);
-       if (ACPI_FAILURE(st) || tbl->header.length < sizeof(*tbl)) {
-               dev_err(&acpi_dev->dev,
-                       FW_BUG "failed to get TPM2 ACPI table\n");
-               return -EINVAL;
-       }
-
-       if (tbl->start_method != ACPI_TPM2_MEMORY_MAPPED)
-               return -ENODEV;
-
-       INIT_LIST_HEAD(&resources);
-       tpm_info.irq = -1;
-       ret = acpi_dev_get_resources(acpi_dev, &resources, tpm_check_resource,
-                                    &tpm_info);
-       if (ret < 0)
-               return ret;
-
-       acpi_dev_free_resource_list(&resources);
-
-       if (resource_type(&tpm_info.res) != IORESOURCE_MEM) {
-               dev_err(&acpi_dev->dev,
-                       FW_BUG "TPM2 ACPI table does not define a memory resource\n");
-               return -EINVAL;
-       }
-
-       if (is_itpm(acpi_dev))
-               itpm = true;
-
-       return tpm_tis_init(&acpi_dev->dev, &tpm_info, acpi_dev->handle);
-}
-
-static int tpm_tis_acpi_remove(struct acpi_device *dev)
-{
-       struct tpm_chip *chip = dev_get_drvdata(&dev->dev);
-
-       tpm_chip_unregister(chip);
-       tpm_tis_remove(chip);
-
-       return 0;
-}
-
-static struct acpi_device_id tpm_acpi_tbl[] = {
-       {"MSFT0101", 0},        /* TPM 2.0 */
-       /* Add new here */
-       {"", 0},                /* User Specified */
-       {"", 0}                 /* Terminator */
-};
-MODULE_DEVICE_TABLE(acpi, tpm_acpi_tbl);
-
-static struct acpi_driver tis_acpi_driver = {
-       .name = "tpm_tis",
-       .ids = tpm_acpi_tbl,
-       .ops = {
-               .add = tpm_tis_acpi_init,
-               .remove = tpm_tis_acpi_remove,
-       },
-       .drv = {
-               .pm = &tpm_tis_pm,
-       },
-};
-#endif
-
 static struct platform_device *force_pdev;
 
 static int tpm_tis_plat_probe(struct platform_device *pdev)
@@ -341,7 +293,7 @@ static int tpm_tis_plat_probe(struct platform_device *pdev)
                        tpm_info.irq = 0;
        }
 
-       return tpm_tis_init(&pdev->dev, &tpm_info, NULL);
+       return tpm_tis_init(&pdev->dev, &tpm_info);
 }
 
 static int tpm_tis_plat_remove(struct platform_device *pdev)
@@ -369,6 +321,7 @@ static struct platform_driver tis_drv = {
                .name           = "tpm_tis",
                .pm             = &tpm_tis_pm,
                .of_match_table = of_match_ptr(tis_of_platform_match),
+               .acpi_match_table = ACPI_PTR(tpm_acpi_tbl),
        },
 };
 
@@ -411,11 +364,6 @@ static int __init init_tis(void)
        if (rc)
                goto err_platform;
 
-#ifdef CONFIG_ACPI
-       rc = acpi_bus_register_driver(&tis_acpi_driver);
-       if (rc)
-               goto err_acpi;
-#endif
 
        if (IS_ENABLED(CONFIG_PNP)) {
                rc = pnp_register_driver(&tis_pnp_driver);
@@ -426,10 +374,6 @@ static int __init init_tis(void)
        return 0;
 
 err_pnp:
-#ifdef CONFIG_ACPI
-       acpi_bus_unregister_driver(&tis_acpi_driver);
-err_acpi:
-#endif
        platform_driver_unregister(&tis_drv);
 err_platform:
        if (force_pdev)
@@ -441,9 +385,6 @@ static int __init init_tis(void)
 static void __exit cleanup_tis(void)
 {
        pnp_unregister_driver(&tis_pnp_driver);
-#ifdef CONFIG_ACPI
-       acpi_bus_unregister_driver(&tis_acpi_driver);
-#endif
        platform_driver_unregister(&tis_drv);
 
        if (force_pdev)