Hi, On Wed, Jul 16, 2014 at 04:45:03PM +0200, Sebastian Andrzej Siewior wrote: > +static int omap_8250_startup(struct uart_port *port) > +{ > + struct uart_8250_port *up = > + container_of(port, struct uart_8250_port, port); > + struct omap8250_priv *priv = port->private_data; > + > + int ret; > + > + if (priv->wakeirq) { > + ret = request_irq(priv->wakeirq, omap_wake_irq, > + port->irqflags, "wakeup irq", port); > + if (ret) > + return ret; > + disable_irq(priv->wakeirq); > + } > + > + ret = serial8250_do_startup(port); > + if (ret) > + goto err; > + > + pm_runtime_get_sync(port->dev);
should this pm_runtime_get_sync() be placed above serial8250_do_startup() call ? -- balbi
signature.asc
Description: Digital signature