Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754398Ab2H3LmF (ORCPT ); Thu, 30 Aug 2012 07:42:05 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:40973 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753018Ab2H3LmC convert rfc822-to-8bit (ORCPT ); Thu, 30 Aug 2012 07:42:02 -0400 From: "Kim, Milo" To: Anton Vorontsov CC: Anton Vorontsov , Axel Lin , David Woodhouse , "linux-kernel@vger.kernel.org" , Fengguang Wu Subject: RE: [PATCH] lp8727_charger: use IRQF_ONESHOT Thread-Topic: [PATCH] lp8727_charger: use IRQF_ONESHOT Thread-Index: AQHNgSRlnI1kj5ruAkSGxfb4PIUiRZdwEOgQgAIynaA= Date: Thu, 30 Aug 2012 11:41:54 +0000 Message-ID: References: <20120823114229.GA16789@localhost> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [157.87.185.123] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1328 Lines: 43 > > ERROR: Threaded IRQ with no primary handler requested without > > IRQF_ONESHOT > > > > Make sure threaded IRQs without a primary handler are always request > > with IRQF_ONESHOT > > > > Signed-off-by: Fengguang Wu > > --- > > > > Note: I don't really know much about the situation, feel free to > > ignore it if it's an false warning. > > > > cocci-output-25411-2ae2e0-lp8727_charger.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > --- a/drivers/power/lp8727_charger.c > > +++ b/drivers/power/lp8727_charger.c > > @@ -255,7 +255,7 @@ static int lp8727_intr_config(struct lp8 > > return request_threaded_irq(pchg->client->irq, > > NULL, > > lp8727_isr_func, > > - IRQF_TRIGGER_FALLING, > > + IRQF_TRIGGER_FALLING | IRQF_ONESHOT, > > "lp8727_irq", > > pchg); > > } > Thanks for the update. > > Acked-by: Milo(Woogyom) Kim > Please refer to the patch below. [PATCH 4/8] lp8727_charger: cleanup the interrupt handler code This includes previous Fengguang' patch. Best Regards, Milo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/