From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5803EC43218 for ; Sat, 27 Apr 2019 12:53:42 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 1E4C920C01 for ; Sat, 27 Apr 2019 12:53:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727448AbfD0Mxl (ORCPT ); Sat, 27 Apr 2019 08:53:41 -0400 Received: from mout.kundenserver.de ([212.227.126.133]:41279 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727383AbfD0Mxk (ORCPT ); Sat, 27 Apr 2019 08:53:40 -0400 Received: from orion.localdomain ([77.2.90.210]) by mrelayeu.kundenserver.de (mreue009 [212.227.15.167]) with ESMTPSA (Nemesis) id 1Mwwhv-1gb3fT0nff-00yOwS; Sat, 27 Apr 2019 14:53:08 +0200 From: "Enrico Weigelt, metux IT consult" To: linux-kernel@vger.kernel.org Cc: gregkh@linuxfoundation.org, andrew@aj.id.au, andriy.shevchenko@linux.intel.com, macro@linux-mips.org, vz@mleia.com, slemieux.tyco@gmail.com, khilman@baylibre.com, liviu.dudau@arm.com, sudeep.holla@arm.com, lorenzo.pieralisi@arm.com, davem@davemloft.net, jacmet@sunsite.dk, linux@prisktech.co.nz, matthias.bgg@gmail.com, linux-mips@vger.kernel.org, linux-serial@vger.kernel.org, linux-ia64@vger.kernel.org, linux-amlogic@lists.infradead.org, linuxppc-dev@lists.ozlabs.org, sparclinux@vger.kernel.org Subject: [PATCH 40/41] drivers: tty: serial: helper for setting mmio range Date: Sat, 27 Apr 2019 14:52:21 +0200 Message-Id: <1556369542-13247-41-git-send-email-info@metux.net> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1556369542-13247-1-git-send-email-info@metux.net> References: <1556369542-13247-1-git-send-email-info@metux.net> X-Provags-ID: V03:K1:vc6wGTaQT5uupsAbTreZS+sSZdK+XaJlhvT04A4iLubNRzD5IPf YPAFQc5vGV+Pk+jx4AQ39iE2rlAHtPW/QsqfgV74Nxmys/AYDobtR0hEBlUQHBmvYoLL0xR XQxKhJsbfgTox/my2891L8/IyJQwk8mMDnx+dEs89rrf/NOEyfUPIMcBkPiwhE0bwqAARyQ PkO/tEXnAuNfYoxPG2X+g== X-UI-Out-Filterresults: notjunk:1;V03:K0:0fn3jPl5GLI=:8ufW2QbxtS+gVFB+NQW1u1 8JHjeYbllrWa5uU4cxUCotmvjIkNGdis89u+uPSuyFgBGmyJBwFdo5yHePu3iPdLG2B1BFOJO foXQ4x0fmbpCh25EffQqOEXIYG7vY3k358LIG2TLIQxadpd4bq2sIwZy7FqVpgQc+u/GEXTd3 Wn/zipp8OdR0jcjLWHyJlof/nIf5+6Z3EhkCE2xQD7oI0Japl61roo6dtokJD59AAdclz2VGl k9X+Ihlopl17s9Br+Bt1SNrS55FFDMpC6YyGOrsrLeIE20ZhIi2NAOG6m9Lj3V/cMZgL4iOuy d88rL7VIy4KysIP0yyoyCRcRxAdyP1OILLtqVCv6ertt6H9/tOvw63iiH3lt+nZ2ioY48WCEZ LCzxc82PTqydfWAXzVyscNyW2Tan2TzmeaC24czKtOkY9uxcT4GFjW0Dd9zUTr55I5z5r/o3+ fqwId2nx+Biz1Eh4HbBPQKK5rIr7Hvxp2wFhFYKr38/yYgSn+HK3a+rX36YJjwgZYS4xZuRPb KeYISc7a/rvtbgvsGsZnc3NQqQN8CbJdf6lRHljqczP2S7HJNYkr5lF+U/qC728wf318smVGo /YvoOMsH57w8LufDHEDU7PtF4FDzpum3jlVa9LpQMuq5pmKWYSl7yYL/rrF461IjDVFgbBIl9 O6zfXAVhU+2Bqicr2w4+ZN8lWh/Yf5Qf++rqvs/lzinWz4LQiHaDmy7DIs3xXa18IgAyexQxr X7p9yVRCoZOh5E1CW23U5VUql4QErQjhEQIXAA== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Introduce a little helpers for settings the mmio range from an struct resource or start/len parameters with less code. (also setting iotype to UPIO_MEM) Also converting drivers to use these new helpers as well as fetching mapsize field instead of using hardcoded values. (the runtime overhead of that should be negligible) The idea is moving to a consistent scheme, so later common calls like request+ioremap combination can be done by generic helpers. Signed-off-by: Enrico Weigelt --- drivers/tty/serial/8250/8250_acorn.c | 5 ++-- drivers/tty/serial/8250/8250_aspeed_vuart.c | 4 +-- drivers/tty/serial/8250/8250_dw.c | 3 +-- drivers/tty/serial/8250/8250_em.c | 2 +- drivers/tty/serial/8250/8250_exar.c | 6 +++-- drivers/tty/serial/8250/8250_hp300.c | 11 +++++--- drivers/tty/serial/8250/8250_of.c | 9 ++----- drivers/tty/serial/meson_uart.c | 5 ++-- drivers/tty/serial/mps2-uart.c | 5 ++-- drivers/tty/serial/pmac_zilog.c | 8 ++---- drivers/tty/serial/vt8500_serial.c | 4 +-- drivers/tty/serial/xilinx_uartps.c | 3 +-- drivers/tty/serial/zs.c | 10 +++++--- include/linux/serial_core.h | 40 +++++++++++++++++++++++++++++ 14 files changed, 76 insertions(+), 39 deletions(-) diff --git a/drivers/tty/serial/8250/8250_acorn.c b/drivers/tty/serial/8250/8250_acorn.c index 758c4aa..359171b 100644 --- a/drivers/tty/serial/8250/8250_acorn.c +++ b/drivers/tty/serial/8250/8250_acorn.c @@ -63,14 +63,15 @@ struct serial_card_info { uart.port.irq = ec->irq; uart.port.flags = UPF_BOOT_AUTOCONF | UPF_SHARE_IRQ; uart.port.uartclk = type->uartclk; - uart.port.iotype = UPIO_MEM; uart.port.regshift = 2; uart.port.dev = &ec->dev; for (i = 0; i < info->num_ports; i++) { uart.port.membase = info->vaddr + type->offset[i]; - uart.port.mapbase = bus_addr + type->offset[i]; + /* mapsize is computed by serial8250_register_8250_port() */ + uart_memres_set_start_len(&uart.port, + bus_addr + type->offset[i], 0); info->ports[i] = serial8250_register_8250_port(&uart); } diff --git a/drivers/tty/serial/8250/8250_aspeed_vuart.c b/drivers/tty/serial/8250/8250_aspeed_vuart.c index 0438d9a..0e06391 100644 --- a/drivers/tty/serial/8250/8250_aspeed_vuart.c +++ b/drivers/tty/serial/8250/8250_aspeed_vuart.c @@ -328,8 +328,6 @@ static int aspeed_vuart_probe(struct platform_device *pdev) memset(&port, 0, sizeof(port)); port.port.private_data = vuart; port.port.membase = vuart->regs; - port.port.mapbase = res->start; - port.port.mapsize = resource_size(res); port.port.startup = aspeed_vuart_startup; port.port.shutdown = aspeed_vuart_shutdown; port.port.throttle = aspeed_vuart_throttle; @@ -337,6 +335,8 @@ static int aspeed_vuart_probe(struct platform_device *pdev) port.port.status = UPSTAT_SYNC_FIFO; port.port.dev = &pdev->dev; + uart_memres_set_res(&port.port, res); + rc = sysfs_create_group(&vuart->dev->kobj, &aspeed_vuart_attr_group); if (rc < 0) return rc; diff --git a/drivers/tty/serial/8250/8250_dw.c b/drivers/tty/serial/8250/8250_dw.c index d31b975..cb65256 100644 --- a/drivers/tty/serial/8250/8250_dw.c +++ b/drivers/tty/serial/8250/8250_dw.c @@ -513,18 +513,17 @@ static int dw8250_probe(struct platform_device *pdev) } spin_lock_init(&p->lock); - p->mapbase = regs->start; p->irq = irq; p->handle_irq = dw8250_handle_irq; p->pm = dw8250_do_pm; p->type = PORT_8250; p->flags = UPF_SHARE_IRQ | UPF_FIXED_PORT; p->dev = dev; - p->iotype = UPIO_MEM; p->serial_in = dw8250_serial_in; p->serial_out = dw8250_serial_out; p->set_ldisc = dw8250_set_ldisc; p->set_termios = dw8250_set_termios; + uart_memres_set_res(p, regs); p->membase = devm_ioremap(dev, regs->start, resource_size(regs)); if (!p->membase) diff --git a/drivers/tty/serial/8250/8250_em.c b/drivers/tty/serial/8250/8250_em.c index 2a76e22..7610441 100644 --- a/drivers/tty/serial/8250/8250_em.c +++ b/drivers/tty/serial/8250/8250_em.c @@ -100,12 +100,12 @@ static int serial8250_em_probe(struct platform_device *pdev) } memset(&up, 0, sizeof(up)); - up.port.mapbase = regs->start; up.port.irq = irq->start; up.port.type = PORT_UNKNOWN; up.port.flags = UPF_BOOT_AUTOCONF | UPF_FIXED_PORT | UPF_IOREMAP; up.port.dev = &pdev->dev; up.port.private_data = priv; + uart_memres_set_res(&up.port, res); clk_prepare_enable(priv->sclk); up.port.uartclk = clk_get_rate(priv->sclk); diff --git a/drivers/tty/serial/8250/8250_exar.c b/drivers/tty/serial/8250/8250_exar.c index 0089aa3..f4c1289 100644 --- a/drivers/tty/serial/8250/8250_exar.c +++ b/drivers/tty/serial/8250/8250_exar.c @@ -134,8 +134,10 @@ static int default_setup(struct exar8250 *priv, struct pci_dev *pcidev, const struct exar8250_board *board = priv->board; unsigned int bar = 0; - port->port.iotype = UPIO_MEM; - port->port.mapbase = pci_resource_start(pcidev, bar) + offset; + uart_memres_set_start_len(&port->port, + pci_resource_start(pcidev, bar) + offset, + pci_resource_len(pcidev, bar)); + port->port.membase = priv->virt + offset; port->port.regshift = board->reg_shift; diff --git a/drivers/tty/serial/8250/8250_hp300.c b/drivers/tty/serial/8250/8250_hp300.c index 3012ea0..09f9dd3 100644 --- a/drivers/tty/serial/8250/8250_hp300.c +++ b/drivers/tty/serial/8250/8250_hp300.c @@ -114,7 +114,9 @@ int __init hp300_setup_serial_console(void) pr_info("Serial console is HP APCI 1\n"); port.uartclk = HPAPCI_BAUD_BASE * 16; - port.mapbase = (FRODO_BASE + FRODO_APCI_OFFSET(1)); + uart_memres_set_start_len( + &port, + FRODO_BASE + FRODO_APCI_OFFSET(1), 0); port.membase = (char *)(port.mapbase + DIO_VIRADDRBASE); port.regshift = 2; add_preferred_console("ttyS", port.line, "9600n8"); @@ -131,7 +133,8 @@ int __init hp300_setup_serial_console(void) pr_info("Serial console is HP DCA at select code %d\n", scode); port.uartclk = HPDCA_BAUD_BASE * 16; - port.mapbase = (pa + UART_OFFSET); + + uart_memres_set_start_len(&port, (pa + UART_OFFSET)); port.membase = (char *)(port.mapbase + DIO_VIRADDRBASE); port.regshift = 1; port.irq = DIO_IPL(pa + DIO_VIRADDRBASE); @@ -173,7 +176,9 @@ static int hpdca_init_one(struct dio_dev *d, uart.port.flags = UPF_SKIP_TEST | UPF_SHARE_IRQ | UPF_BOOT_AUTOCONF; uart.port.irq = d->ipl; uart.port.uartclk = HPDCA_BAUD_BASE * 16; - uart.port.mapbase = (d->resource.start + UART_OFFSET); + uart_memres_set_start_len(&uart.port, + (d->resource.start + UART_OFFSET), + resource_size(&d->resource)); uart.port.membase = (char *)(uart.port.mapbase + DIO_VIRADDRBASE); uart.port.regshift = 1; uart.port.dev = &d->dev; diff --git a/drivers/tty/serial/8250/8250_of.c b/drivers/tty/serial/8250/8250_of.c index 08157a1..1c5896c 100644 --- a/drivers/tty/serial/8250/8250_of.c +++ b/drivers/tty/serial/8250/8250_of.c @@ -96,18 +96,13 @@ static int of_platform_serial_setup(struct platform_device *ofdev, UPF_FIXED_TYPE; spin_lock_init(&port->lock); - if (resource_type(&resource) == IORESOURCE_IO) { - port->iotype = UPIO_PORT; - port->iobase = resource.start; - } else { - port->mapbase = resource.start; - port->mapsize = resource_size(&resource); + uart_memres_set_res(port, &resource); + if (resource_type(&resource) == IORESOURCE_MEM) { /* Check for shifted address mapping */ if (of_property_read_u32(np, "reg-offset", &prop) == 0) port->mapbase += prop; - port->iotype = UPIO_MEM; if (of_property_read_u32(np, "reg-io-width", &prop) == 0) { switch (prop) { case 1: diff --git a/drivers/tty/serial/meson_uart.c b/drivers/tty/serial/meson_uart.c index fbc5bc0..bfcaa2f 100644 --- a/drivers/tty/serial/meson_uart.c +++ b/drivers/tty/serial/meson_uart.c @@ -698,9 +698,8 @@ static int meson_uart_probe(struct platform_device *pdev) if (ret) return ret; - port->iotype = UPIO_MEM; - port->mapbase = res_mem->start; - port->mapsize = resource_size(res_mem); + uart_memset_set_res(port, res_mem); + port->irq = res_irq->start; port->flags = UPF_BOOT_AUTOCONF | UPF_LOW_LATENCY; port->dev = &pdev->dev; diff --git a/drivers/tty/serial/mps2-uart.c b/drivers/tty/serial/mps2-uart.c index 587b42f..a4fd1de 100644 --- a/drivers/tty/serial/mps2-uart.c +++ b/drivers/tty/serial/mps2-uart.c @@ -562,9 +562,8 @@ static int mps2_init_port(struct platform_device *pdev, if (IS_ERR(mps_port->port.membase)) return PTR_ERR(mps_port->port.membase); - mps_port->port.mapbase = res->start; - mps_port->port.mapsize = resource_size(res); - mps_port->port.iotype = UPIO_MEM; + uart_memres_set_res(&mps_port->port, res); + mps_port->port.flags = UPF_BOOT_AUTOCONF; mps_port->port.fifosize = 1; mps_port->port.ops = &mps2_uart_pops; diff --git a/drivers/tty/serial/pmac_zilog.c b/drivers/tty/serial/pmac_zilog.c index 1fef014..7017dd2 100644 --- a/drivers/tty/serial/pmac_zilog.c +++ b/drivers/tty/serial/pmac_zilog.c @@ -1412,8 +1412,7 @@ static int __init pmz_init_port(struct uart_pmac_port *uap) */ if (of_address_to_resource(np, 0, &r_ports)) return -ENODEV; - uap->port.mapbase = r_ports.start; - uap->port.mapsize = PMZ_MAPSIZE; + uart_memres_set_start_len(&uap->port, r_ports.start, PMZ_MAPSIZE); uap->port.membase = ioremap(uap->port.mapbase, uap->port.mapsize); uap->control_reg = uap->port.membase; @@ -1493,7 +1492,6 @@ static int __init pmz_init_port(struct uart_pmac_port *uap) /* * Init remaining bits of "port" structure */ - uap->port.iotype = UPIO_MEM; uap->port.irq = irq_of_parse_and_map(np, 0); uap->port.uartclk = ZS_CLOCK; uap->port.fifosize = 1; @@ -1711,10 +1709,8 @@ static int __init pmz_init_port(struct uart_pmac_port *uap) if (!r_ports || irq <= 0) return -ENODEV; - uap->port.mapbase = r_ports->start; - uap->port.mapsize = PMZ_MAPSIZE; + uart_memres_set_start_len(&uap->port, r_ports->start, PMZ_MAPSIZE); uap->port.membase = (unsigned char __iomem *) r_ports->start; - uap->port.iotype = UPIO_MEM; uap->port.irq = irq; uap->port.uartclk = ZS_CLOCK; uap->port.fifosize = 1; diff --git a/drivers/tty/serial/vt8500_serial.c b/drivers/tty/serial/vt8500_serial.c index 3d58e9b..b303de4 100644 --- a/drivers/tty/serial/vt8500_serial.c +++ b/drivers/tty/serial/vt8500_serial.c @@ -695,8 +695,6 @@ static int vt8500_serial_probe(struct platform_device *pdev) VT8500_RECOMMENDED_CLK ); vt8500_port->uart.type = PORT_VT8500; - vt8500_port->uart.iotype = UPIO_MEM; - vt8500_port->uart.mapbase = mmres->start; vt8500_port->uart.irq = irqres->start; vt8500_port->uart.fifosize = 16; vt8500_port->uart.ops = &vt8500_uart_pops; @@ -704,6 +702,8 @@ static int vt8500_serial_probe(struct platform_device *pdev) vt8500_port->uart.dev = &pdev->dev; vt8500_port->uart.flags = UPF_IOREMAP | UPF_BOOT_AUTOCONF; + uart_memres_set_res(&vt8500_port->uart, mmres); + /* Serial core uses the magic "16" everywhere - adjust for it */ vt8500_port->uart.uartclk = 16 * clk_get_rate(vt8500_port->clk) / vt8500_port->clk_predivisor / diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c index cf8ca66..895c90c 100644 --- a/drivers/tty/serial/xilinx_uartps.c +++ b/drivers/tty/serial/xilinx_uartps.c @@ -1626,8 +1626,7 @@ static int cdns_uart_probe(struct platform_device *pdev) * This function also registers this device with the tty layer * and triggers invocation of the config_port() entry point. */ - port->mapbase = res->start; - port->mapsize = CDNS_UART_REGISTER_SPACE; + uart_memres_set_start_len(res->start, CDNS_UART_REGISTER_SPACE); port->irq = irq; port->dev = &pdev->dev; port->uartclk = clk_get_rate(cdns_uart_data->uartclk); diff --git a/drivers/tty/serial/zs.c b/drivers/tty/serial/zs.c index ab432ba..87990b1 100644 --- a/drivers/tty/serial/zs.c +++ b/drivers/tty/serial/zs.c @@ -1113,10 +1113,12 @@ static int __init zs_probe_sccs(void) uport->flags = UPF_BOOT_AUTOCONF; uport->ops = &zs_ops; uport->line = chip * ZS_NUM_CHAN + side; - uport->mapsize = ZS_CHAN_IO_SIZE; - uport->mapbase = dec_kn_slot_base + - zs_parms.scc[chip] + - (side ^ ZS_CHAN_B) * ZS_CHAN_IO_SIZE; + + uart_memres_set_start_len(dec_kn_slot_base + + zs_parms.scc[chip] + + (side ^ ZS_CHAN_B) * + ZS_CHAN_IO_SIZE, + ZS_CHAN_IO_SIZE); for (i = 0; i < ZS_NUM_REGS; i++) zport->regs[i] = zs_init_regs[i]; diff --git a/include/linux/serial_core.h b/include/linux/serial_core.h index 5fe2b03..d891c8d 100644 --- a/include/linux/serial_core.h +++ b/include/linux/serial_core.h @@ -427,6 +427,46 @@ void uart_console_write(struct uart_port *port, const char *s, int uart_match_port(struct uart_port *port1, struct uart_port *port2); /* + * set physical io range from struct resource + * if resource is NULL, clear the fields + * also set the iotype to UPIO_MEM + */ +static inline void uart_memres_set_res(struct uart_port *port, + struct resource *res) +{ + if (!res) { + port->mapsize = 0; + port->mapbase = 0; + port->iobase = 0; + return; + } + + if (resource_type(res) == IORESOURCE_IO) { + port->iotype = UPIO_PORT; + port->iobase = resource->start; + return; + } + + uart->mapbase = res->start; + uart->mapsize = resource_size(res); + uart->iotype = UPIO_MEM; +} + +/* + * set physical io range by start address and length + * if resource is NULL, clear the fields + * also set the iotype to UPIO_MEM + */ +static inline void uart_memres_set_start_len(struct uart_driver *uart, + resource_size_t start, + resource_size_t len) +{ + uart->mapbase = start; + uart->mapsize = len; + uart->iotype = UPIO_MEM; +} + +/* * Power Management */ int uart_suspend_port(struct uart_driver *reg, struct uart_port *port); -- 1.9.1