Revert "serial-uartlite: Do not use static struct uart_driver out of probe()"
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 13 Nov 2019 22:28:15 +0000 (06:28 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 13 Nov 2019 22:28:15 +0000 (06:28 +0800)
This reverts commit 3b209d253e7f8aa01fde0233d38a7239c8f7beb3.

As Johan says, this driver needs a lot more work and these changes are
only going in the wrong direction:
    https://lkml.kernel.org/r/20190523091839.GC568@localhost

Reported-by: Johan Hovold <johan@kernel.org>
Cc: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
Cc: Michal Simek <michal.simek@xilinx.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/uartlite.c

index a713080..2e93af7 100644 (file)
@@ -63,7 +63,6 @@ static struct uart_port *console_port;
 struct uartlite_data {
        const struct uartlite_reg_ops *reg_ops;
        struct clk *clk;
-       struct uart_driver *ulite_uart_driver;
 };
 
 struct uartlite_reg_ops {
@@ -695,9 +694,7 @@ static int ulite_release(struct device *dev)
        int rc = 0;
 
        if (port) {
-               struct uartlite_data *pdata = port->private_data;
-
-               rc = uart_remove_one_port(pdata->ulite_uart_driver, port);
+               rc = uart_remove_one_port(&ulite_uart_driver, port);
                dev_set_drvdata(dev, NULL);
                port->mapbase = 0;
        }
@@ -715,11 +712,8 @@ static int __maybe_unused ulite_suspend(struct device *dev)
 {
        struct uart_port *port = dev_get_drvdata(dev);
 
-       if (port) {
-               struct uartlite_data *pdata = port->private_data;
-
-               uart_suspend_port(pdata->ulite_uart_driver, port);
-       }
+       if (port)
+               uart_suspend_port(&ulite_uart_driver, port);
 
        return 0;
 }
@@ -734,11 +728,8 @@ static int __maybe_unused ulite_resume(struct device *dev)
 {
        struct uart_port *port = dev_get_drvdata(dev);
 
-       if (port) {
-               struct uartlite_data *pdata = port->private_data;
-
-               uart_resume_port(pdata->ulite_uart_driver, port);
-       }
+       if (port)
+               uart_resume_port(&ulite_uart_driver, port);
 
        return 0;
 }
@@ -813,7 +804,6 @@ static int ulite_probe(struct platform_device *pdev)
                pdata->clk = NULL;
        }
 
-       pdata->ulite_uart_driver = &ulite_uart_driver;
        ret = clk_prepare_enable(pdata->clk);
        if (ret) {
                dev_err(&pdev->dev, "Failed to prepare clock\n");