[PATCH] serial: omap: Add support for optional wake-up interrupt

Tony Lindgren tony at atomide.com
Fri Oct 18 12:45:35 EDT 2013


* Tony Lindgren <tony at atomide.com> [131018 09:38]:
> * Felipe Balbi <balbi at ti.com> [131018 09:19]:
> > > @@ -786,7 +813,10 @@ static void serial_omap_shutdown(struct uart_port *port)
> > >  
> > >  	pm_runtime_mark_last_busy(up->dev);
> > >  	pm_runtime_put_autosuspend(up->dev);
> > > -	free_irq(up->port.irq, up);
> > > +
> > > +	for (i = 0; i < ARRAY_SIZE(up->irqs); i++)
> > > +		if (up->irqs[i])
> > > +			devm_free_irq(up->port.dev, up->irqs[i], up);
> > 
> > do you need this at all if you're using devm_* ?
> 
> So it seems, startup and shutdown are managed by serial_core and
> that's what at least clps711x.c serial driver is doing.

And that means devm_* in this case does not really help us
here..

I guess we could keep the IRQ requested from probe, but
there's probably a reason why it's done in startup/shutdown.

So I'll just drop the devm_* changes for now.

Regards,

Tony



More information about the linux-arm-kernel mailing list