tty: serial: bcm63xx: Eliminate unnecessary request/release functions
authorKevin Cernekee <cernekee@gmail.com>
Tue, 21 Oct 2014 22:23:03 +0000 (15:23 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 6 Nov 2014 22:57:24 +0000 (14:57 -0800)
We don't really need to perform the ioremap "on demand" so it's simpler
just to do it from the probe function.  This also lets us eliminate the
UART_REG_SIZE constant and rely on the resource information passed in
from the DT or platform code.

Signed-off-by: Kevin Cernekee <cernekee@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/bcm63xx_uart.c
include/linux/serial_bcm63xx.h

index 109dea711318b5db0f7ddfda9ca0f1f70ae07582..e04e5805ae6e79c1ce88e3c4f18f288ed2031eba 100644 (file)
@@ -588,20 +588,7 @@ static void bcm_uart_set_termios(struct uart_port *port,
  */
 static int bcm_uart_request_port(struct uart_port *port)
 {
-       unsigned int size;
-
-       size = UART_REG_SIZE;
-       if (!request_mem_region(port->mapbase, size, "bcm63xx")) {
-               dev_err(port->dev, "Memory region busy\n");
-               return -EBUSY;
-       }
-
-       port->membase = ioremap(port->mapbase, size);
-       if (!port->membase) {
-               dev_err(port->dev, "Unable to map registers\n");
-               release_mem_region(port->mapbase, size);
-               return -EBUSY;
-       }
+       /* UARTs always present */
        return 0;
 }
 
@@ -610,8 +597,7 @@ static int bcm_uart_request_port(struct uart_port *port)
  */
 static void bcm_uart_release_port(struct uart_port *port)
 {
-       release_mem_region(port->mapbase, UART_REG_SIZE);
-       iounmap(port->membase);
+       /* Nothing to release ... */
 }
 
 /*
@@ -833,13 +819,20 @@ static int bcm_uart_probe(struct platform_device *pdev)
        if (pdev->id < 0 || pdev->id >= BCM63XX_NR_UARTS)
                return -EINVAL;
 
-       if (ports[pdev->id].membase)
+       port = &ports[pdev->id];
+       if (port->membase)
                return -EBUSY;
+       memset(port, 0, sizeof(*port));
 
        res_mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        if (!res_mem)
                return -ENODEV;
 
+       port->mapbase = res_mem->start;
+       port->membase = devm_ioremap_resource(&pdev->dev, res_mem);
+       if (IS_ERR(port->membase))
+               return PTR_ERR(port->membase);
+
        res_irq = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
        if (!res_irq)
                return -ENODEV;
@@ -849,10 +842,7 @@ static int bcm_uart_probe(struct platform_device *pdev)
        if (IS_ERR(clk))
                return -ENODEV;
 
-       port = &ports[pdev->id];
-       memset(port, 0, sizeof(*port));
        port->iotype = UPIO_MEM;
-       port->mapbase = res_mem->start;
        port->irq = res_irq->start;
        port->ops = &bcm_uart_ops;
        port->flags = UPF_BOOT_AUTOCONF;
index a80aa1a5bee2bebc53c87e7fde4e6e4881a38fee..570e964dc899c39928ec18dce0db1073838c986f 100644 (file)
                                        UART_FIFO_PARERR_MASK |         \
                                        UART_FIFO_BRKDET_MASK)
 
-#define UART_REG_SIZE                  24
-
 #endif /* _LINUX_SERIAL_BCM63XX_H */