Hi, On Tue, Dec 10, 2013 at 09:32:13AM +0000, Lee Jones wrote: > On Mon, 09 Dec 2013, Felipe Balbi wrote: > > > We don't need that extra workqueue when we have generic threaded irq > > handlers support. This patch just moves over to threaded irqs and > > deletes the unnecessary workqueue. > > > > Tested-by: Aaro Koskinen <aaro.koski...@iki.fi> > > Signed-off-by: Felipe Balbi <ba...@ti.com> > > --- > > drivers/mfd/menelaus.c | 29 ++++------------------------- > > 1 file changed, 4 insertions(+), 25 deletions(-) > > <snip> > > > - err = request_irq(client->irq, menelaus_irq, 0, > > - DRIVER_NAME, menelaus); > > + err = request_threaded_irq(client->irq, NULL, menelaus_irq, > > + IRQF_ONESHOT, DRIVER_NAME, menelaus); > > devm_* managed resources please.
that would be subject to another patch. One that converts to devm_* on the entire driver. -- balbi
signature.asc
Description: Digital signature
