mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-27 19:29:37 +00:00
serial: 8250_aspeed_vuart: use UPF_IOREMAP to set up register mapping
Previously this driver's use of devm_ioremap_resource() led to duplicated calls to __release_region() when unbinding it (one from serial8250_release_std_resource() and one from devres_release_all()), the second of which resulted in a warning message: # echo 1e787000.serial > /sys/bus/platform/drivers/aspeed-vuart/unbind [33091.774200] Trying to free nonexistent resource <000000001e787000-000000001e78703f> With this change the driver uses the generic serial8250 code's UPF_IOREMAP to take care of the register mapping automatically instead of doing its own devm_ioremap_resource(), thus avoiding the duplicate __release_region() on unbind. In doing this we eliminate vuart->regs, since it merely duplicates vuart->port->port.membase, which we now use for our I/O accesses. Reported-by: Andrew Jeffery <andrew@aj.id.au> Signed-off-by: Zev Weiss <zev@bewilderbeest.net> Link: https://lore.kernel.org/r/20210510014231.647-4-zev@bewilderbeest.net Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
c9805fbf9d
commit
54da3e381c
1 changed files with 3 additions and 8 deletions
|
@ -34,7 +34,6 @@
|
|||
|
||||
struct aspeed_vuart {
|
||||
struct device *dev;
|
||||
void __iomem *regs;
|
||||
struct clk *clk;
|
||||
int line;
|
||||
struct timer_list unthrottle_timer;
|
||||
|
@ -66,12 +65,12 @@ static const int unthrottle_timeout = HZ/10;
|
|||
|
||||
static inline u8 aspeed_vuart_readb(struct aspeed_vuart *vuart, u8 reg)
|
||||
{
|
||||
return readb(vuart->regs + reg);
|
||||
return readb(vuart->port->port.membase + reg);
|
||||
}
|
||||
|
||||
static inline void aspeed_vuart_writeb(struct aspeed_vuart *vuart, u8 val, u8 reg)
|
||||
{
|
||||
writeb(val, vuart->regs + reg);
|
||||
writeb(val, vuart->port->port.membase + reg);
|
||||
}
|
||||
|
||||
static ssize_t lpc_address_show(struct device *dev,
|
||||
|
@ -429,13 +428,9 @@ static int aspeed_vuart_probe(struct platform_device *pdev)
|
|||
timer_setup(&vuart->unthrottle_timer, aspeed_vuart_unthrottle_exp, 0);
|
||||
|
||||
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
|
||||
vuart->regs = devm_ioremap_resource(&pdev->dev, res);
|
||||
if (IS_ERR(vuart->regs))
|
||||
return PTR_ERR(vuart->regs);
|
||||
|
||||
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;
|
||||
|
@ -492,7 +487,7 @@ static int aspeed_vuart_probe(struct platform_device *pdev)
|
|||
port.port.iotype = UPIO_MEM;
|
||||
port.port.type = PORT_16550A;
|
||||
port.port.uartclk = clk;
|
||||
port.port.flags = UPF_SHARE_IRQ | UPF_BOOT_AUTOCONF
|
||||
port.port.flags = UPF_SHARE_IRQ | UPF_BOOT_AUTOCONF | UPF_IOREMAP
|
||||
| UPF_FIXED_PORT | UPF_FIXED_TYPE | UPF_NO_THRE_TEST;
|
||||
|
||||
if (of_property_read_bool(np, "no-loopback-test"))
|
||||
|
|
Loading…
Reference in a new issue