Add pvpanic driver framework. Split the original pvpanic acpi/of
driver as the two seperate files and modify code for adaptation framework
in follow-up patches.

Signed-off-by: Peng Hao <peng.h...@zte.com.cn>
---
 drivers/misc/pvpanic/pvpanic.c | 171 ++++++++++-------------------------------
 1 file changed, 39 insertions(+), 132 deletions(-)

diff --git a/drivers/misc/pvpanic/pvpanic.c b/drivers/misc/pvpanic/pvpanic.c
index 595ac06..6380540 100644
--- a/drivers/misc/pvpanic/pvpanic.c
+++ b/drivers/misc/pvpanic/pvpanic.c
@@ -8,15 +8,20 @@
 
 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 
-#include <linux/acpi.h>
+#include <linux/io.h>
 #include <linux/kernel.h>
 #include <linux/module.h>
-#include <linux/of.h>
-#include <linux/of_address.h>
 #include <linux/platform_device.h>
 #include <linux/types.h>
 
-static void __iomem *base;
+static struct {
+       struct platform_device *pdev;
+       void __iomem *base;
+       bool is_ioport;
+} pvpanic_data = {
+       .pdev = NULL,
+       .is_ioport = false,
+};
 
 #define PVPANIC_PANICKED        (1 << 0)
 
@@ -27,7 +32,7 @@
 static void
 pvpanic_send_event(unsigned int event)
 {
-       iowrite8(event, base);
+       iowrite8(event, pvpanic_data.base);
 }
 
 static int
@@ -43,150 +48,52 @@
        .priority = 1, /* let this called before broken drm_fb_helper */
 };
 
-#ifdef CONFIG_ACPI
-static int pvpanic_add(struct acpi_device *device);
-static int pvpanic_remove(struct acpi_device *device);
-
-static const struct acpi_device_id pvpanic_device_ids[] = {
-       { "QEMU0001", 0 },
-       { "", 0 }
-};
-MODULE_DEVICE_TABLE(acpi, pvpanic_device_ids);
-
-static struct acpi_driver pvpanic_driver = {
-       .name =         "pvpanic",
-       .class =        "QEMU",
-       .ids =          pvpanic_device_ids,
-       .ops =          {
-                               .add =          pvpanic_add,
-                               .remove =       pvpanic_remove,
-                       },
-       .owner =        THIS_MODULE,
-};
-
-static acpi_status
-pvpanic_walk_resources(struct acpi_resource *res, void *context)
+static int pvpanic_platform_probe(struct platform_device *pdev)
 {
-       struct resource r;
-
-       if (acpi_dev_resource_io(res, &r)) {
-               base = ioport_map(r.start, resource_size(&r));
-               return AE_OK;
-       } else if (acpi_dev_resource_memory(res, &r)) {
-               base = ioremap(r.start, resource_size(&r));
-               return AE_OK;
+       struct device *dev = &pdev->dev;
+       struct resource *res;
+       void __iomem *base;
+
+       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+       if (res) {
+               base = devm_ioremap_resource(dev, res);
+               if (IS_ERR(base))
+                       return -ENODEV;
+       } else {
+               res = platform_get_resource(pdev, IORESOURCE_IO, 0);
+               if (!res)
+                       return -ENODEV;
+
+               base = ioport_map(res->start, resource_size(res));
+               if (!base)
+                       return -ENODEV;
+               pvpanic_data.is_ioport = true;
        }
 
-       return AE_ERROR;
-}
-
-static int pvpanic_add(struct acpi_device *device)
-{
-       int ret;
-
-       ret = acpi_bus_get_status(device);
-       if (ret < 0)
-               return ret;
-
-       if (!device->status.enabled || !device->status.functional)
-               return -ENODEV;
-
-       acpi_walk_resources(device->handle, METHOD_NAME__CRS,
-                           pvpanic_walk_resources, NULL);
-
-       if (!base)
-               return -ENODEV;
-
+       pvpanic_data.base = base;
        atomic_notifier_chain_register(&panic_notifier_list,
                                       &pvpanic_panic_nb);
 
        return 0;
 }
 
-static int pvpanic_remove(struct acpi_device *device)
+static int pvpanic_platform_remove(struct platform_device *pdev)
 {
-
        atomic_notifier_chain_unregister(&panic_notifier_list,
                                         &pvpanic_panic_nb);
-       iounmap(base);
-
-       return 0;
-}
-
-static int pvpanic_register_acpi_driver(void)
-{
-       return acpi_bus_register_driver(&pvpanic_driver);
-}
-
-static void pvpanic_unregister_acpi_driver(void)
-{
-       acpi_bus_unregister_driver(&pvpanic_driver);
-}
-#else
-static int pvpanic_register_acpi_driver(void)
-{
-       return -ENODEV;
-}
 
-static void pvpanic_unregister_acpi_driver(void) {}
-#endif
-
-static int pvpanic_mmio_probe(struct platform_device *pdev)
-{
-       struct resource *mem;
-
-       mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       if (!mem)
-               return -EINVAL;
-
-       base = devm_ioremap_resource(&pdev->dev, mem);
-       if (IS_ERR(base))
-               return PTR_ERR(base);
-
-       atomic_notifier_chain_register(&panic_notifier_list,
-                                      &pvpanic_panic_nb);
-
-       return 0;
-}
-
-static int pvpanic_mmio_remove(struct platform_device *pdev)
-{
-
-       atomic_notifier_chain_unregister(&panic_notifier_list,
-                                        &pvpanic_panic_nb);
+       if (pvpanic_data.is_ioport)
+               iounmap(pvpanic_data.base);
 
        return 0;
 }
 
-static const struct of_device_id pvpanic_mmio_match[] = {
-       { .compatible = "qemu,pvpanic-mmio", },
-       {}
-};
-
-static struct platform_driver pvpanic_mmio_driver = {
+static struct platform_driver pvpanic_driver = {
+       .probe = pvpanic_platform_probe,
+       .remove = pvpanic_platform_remove,
        .driver = {
-               .name = "pvpanic-mmio",
-               .of_match_table = pvpanic_mmio_match,
-       },
-       .probe = pvpanic_mmio_probe,
-       .remove = pvpanic_mmio_remove,
+               .name = "pvpanic",
+       }
 };
 
-static int __init pvpanic_mmio_init(void)
-{
-       if (acpi_disabled)
-               return platform_driver_register(&pvpanic_mmio_driver);
-       else
-               return pvpanic_register_acpi_driver();
-}
-
-static void __exit pvpanic_mmio_exit(void)
-{
-       if (acpi_disabled)
-               platform_driver_unregister(&pvpanic_mmio_driver);
-       else
-               pvpanic_unregister_acpi_driver();
-}
-
-module_init(pvpanic_mmio_init);
-module_exit(pvpanic_mmio_exit);
+module_platform_driver(pvpanic_driver);
-- 
1.8.3.1

Reply via email to