From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:41062 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756340AbcB0RVc (ORCPT ); Sat, 27 Feb 2016 12:21:32 -0500 Subject: Re: [PATCH 7/7] iio: adc: palmas: Drop IRQF_EARLY_RESUME flag To: Grygorii Strashko , Lee Jones , Alessandro Zummo , MyungJoo Ham , Lars-Peter Clausen References: <1456501377-16871-1-git-send-email-grygorii.strashko@ti.com> <1456501377-16871-8-git-send-email-grygorii.strashko@ti.com> Cc: rtc-linux@googlegroups.com, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, nsekhar@ti.com, linux-omap@vger.kernel.org, Hartmut Knaack , Peter Meerwald , Nishanth Menon From: Jonathan Cameron Message-ID: <56D1DB19.5030504@kernel.org> Date: Sat, 27 Feb 2016 17:21:29 +0000 MIME-Version: 1.0 In-Reply-To: <1456501377-16871-8-git-send-email-grygorii.strashko@ti.com> Content-Type: text/plain; charset=windows-1252 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 26/02/16 15:42, Grygorii Strashko wrote: > Palmas gpadc IRQs are nested threaded and this flag is not required for nested > irqs anymore, since commit 3c646f2c6aa9 ("genirq: Don't suspend nested_thread > irqs over system suspend") was merged. > > Cc: Hartmut Knaack > Cc: Lars-Peter Clausen > Cc: Peter Meerwald > Cc: Lee Jones > Cc: Nishanth Menon > Signed-off-by: Grygorii Strashko Applied to the togreg branch of iio.git - initially pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > --- > drivers/iio/adc/palmas_gpadc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/iio/adc/palmas_gpadc.c b/drivers/iio/adc/palmas_gpadc.c > index f42eb8a..2bbf0c5 100644 > --- a/drivers/iio/adc/palmas_gpadc.c > +++ b/drivers/iio/adc/palmas_gpadc.c > @@ -534,7 +534,7 @@ static int palmas_gpadc_probe(struct platform_device *pdev) > } > ret = request_threaded_irq(adc->irq, NULL, > palmas_gpadc_irq, > - IRQF_ONESHOT | IRQF_EARLY_RESUME, dev_name(adc->dev), > + IRQF_ONESHOT, dev_name(adc->dev), > adc); > if (ret < 0) { > dev_err(adc->dev, > @@ -549,7 +549,7 @@ static int palmas_gpadc_probe(struct platform_device *pdev) > adc->irq_auto_0 = platform_get_irq(pdev, 1); > ret = request_threaded_irq(adc->irq_auto_0, NULL, > palmas_gpadc_irq_auto, > - IRQF_ONESHOT | IRQF_EARLY_RESUME, > + IRQF_ONESHOT, > "palmas-adc-auto-0", adc); > if (ret < 0) { > dev_err(adc->dev, "request auto0 irq %d failed: %d\n", > @@ -565,7 +565,7 @@ static int palmas_gpadc_probe(struct platform_device *pdev) > adc->irq_auto_1 = platform_get_irq(pdev, 2); > ret = request_threaded_irq(adc->irq_auto_1, NULL, > palmas_gpadc_irq_auto, > - IRQF_ONESHOT | IRQF_EARLY_RESUME, > + IRQF_ONESHOT, > "palmas-adc-auto-1", adc); > if (ret < 0) { > dev_err(adc->dev, "request auto1 irq %d failed: %d\n", > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Cameron Subject: Re: [PATCH 7/7] iio: adc: palmas: Drop IRQF_EARLY_RESUME flag Date: Sat, 27 Feb 2016 17:21:29 +0000 Message-ID: <56D1DB19.5030504@kernel.org> References: <1456501377-16871-1-git-send-email-grygorii.strashko@ti.com> <1456501377-16871-8-git-send-email-grygorii.strashko@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1456501377-16871-8-git-send-email-grygorii.strashko-l0cyMroinI0@public.gmane.org> Sender: linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Grygorii Strashko , Lee Jones , Alessandro Zummo , MyungJoo Ham , Lars-Peter Clausen Cc: rtc-linux-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, nsekhar-l0cyMroinI0@public.gmane.org, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Hartmut Knaack , Peter Meerwald , Nishanth Menon List-Id: linux-omap@vger.kernel.org On 26/02/16 15:42, Grygorii Strashko wrote: > Palmas gpadc IRQs are nested threaded and this flag is not required for nested > irqs anymore, since commit 3c646f2c6aa9 ("genirq: Don't suspend nested_thread > irqs over system suspend") was merged. > > Cc: Hartmut Knaack > Cc: Lars-Peter Clausen > Cc: Peter Meerwald > Cc: Lee Jones > Cc: Nishanth Menon > Signed-off-by: Grygorii Strashko Applied to the togreg branch of iio.git - initially pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > --- > drivers/iio/adc/palmas_gpadc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/iio/adc/palmas_gpadc.c b/drivers/iio/adc/palmas_gpadc.c > index f42eb8a..2bbf0c5 100644 > --- a/drivers/iio/adc/palmas_gpadc.c > +++ b/drivers/iio/adc/palmas_gpadc.c > @@ -534,7 +534,7 @@ static int palmas_gpadc_probe(struct platform_device *pdev) > } > ret = request_threaded_irq(adc->irq, NULL, > palmas_gpadc_irq, > - IRQF_ONESHOT | IRQF_EARLY_RESUME, dev_name(adc->dev), > + IRQF_ONESHOT, dev_name(adc->dev), > adc); > if (ret < 0) { > dev_err(adc->dev, > @@ -549,7 +549,7 @@ static int palmas_gpadc_probe(struct platform_device *pdev) > adc->irq_auto_0 = platform_get_irq(pdev, 1); > ret = request_threaded_irq(adc->irq_auto_0, NULL, > palmas_gpadc_irq_auto, > - IRQF_ONESHOT | IRQF_EARLY_RESUME, > + IRQF_ONESHOT, > "palmas-adc-auto-0", adc); > if (ret < 0) { > dev_err(adc->dev, "request auto0 irq %d failed: %d\n", > @@ -565,7 +565,7 @@ static int palmas_gpadc_probe(struct platform_device *pdev) > adc->irq_auto_1 = platform_get_irq(pdev, 2); > ret = request_threaded_irq(adc->irq_auto_1, NULL, > palmas_gpadc_irq_auto, > - IRQF_ONESHOT | IRQF_EARLY_RESUME, > + IRQF_ONESHOT, > "palmas-adc-auto-1", adc); > if (ret < 0) { > dev_err(adc->dev, "request auto1 irq %d failed: %d\n", > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk (saturn.retrosnub.co.uk. [2a01:8000:1ffa:f003:bc9d:1dff:fe9b:7466]) by gmr-mx.google.com with ESMTPS id c140si27969wmh.1.2016.02.27.09.21.31 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 27 Feb 2016 09:21:31 -0800 (PST) Subject: [rtc-linux] Re: [PATCH 7/7] iio: adc: palmas: Drop IRQF_EARLY_RESUME flag To: Grygorii Strashko , Lee Jones , Alessandro Zummo , MyungJoo Ham , Lars-Peter Clausen References: <1456501377-16871-1-git-send-email-grygorii.strashko@ti.com> <1456501377-16871-8-git-send-email-grygorii.strashko@ti.com> Cc: rtc-linux@googlegroups.com, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, nsekhar@ti.com, linux-omap@vger.kernel.org, Hartmut Knaack , Peter Meerwald , Nishanth Menon From: Jonathan Cameron Message-ID: <56D1DB19.5030504@kernel.org> Date: Sat, 27 Feb 2016 17:21:29 +0000 MIME-Version: 1.0 In-Reply-To: <1456501377-16871-8-git-send-email-grygorii.strashko@ti.com> Content-Type: text/plain; charset=UTF-8 Reply-To: rtc-linux@googlegroups.com List-ID: List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , On 26/02/16 15:42, Grygorii Strashko wrote: > Palmas gpadc IRQs are nested threaded and this flag is not required for nested > irqs anymore, since commit 3c646f2c6aa9 ("genirq: Don't suspend nested_thread > irqs over system suspend") was merged. > > Cc: Hartmut Knaack > Cc: Lars-Peter Clausen > Cc: Peter Meerwald > Cc: Lee Jones > Cc: Nishanth Menon > Signed-off-by: Grygorii Strashko Applied to the togreg branch of iio.git - initially pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > --- > drivers/iio/adc/palmas_gpadc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/iio/adc/palmas_gpadc.c b/drivers/iio/adc/palmas_gpadc.c > index f42eb8a..2bbf0c5 100644 > --- a/drivers/iio/adc/palmas_gpadc.c > +++ b/drivers/iio/adc/palmas_gpadc.c > @@ -534,7 +534,7 @@ static int palmas_gpadc_probe(struct platform_device *pdev) > } > ret = request_threaded_irq(adc->irq, NULL, > palmas_gpadc_irq, > - IRQF_ONESHOT | IRQF_EARLY_RESUME, dev_name(adc->dev), > + IRQF_ONESHOT, dev_name(adc->dev), > adc); > if (ret < 0) { > dev_err(adc->dev, > @@ -549,7 +549,7 @@ static int palmas_gpadc_probe(struct platform_device *pdev) > adc->irq_auto_0 = platform_get_irq(pdev, 1); > ret = request_threaded_irq(adc->irq_auto_0, NULL, > palmas_gpadc_irq_auto, > - IRQF_ONESHOT | IRQF_EARLY_RESUME, > + IRQF_ONESHOT, > "palmas-adc-auto-0", adc); > if (ret < 0) { > dev_err(adc->dev, "request auto0 irq %d failed: %d\n", > @@ -565,7 +565,7 @@ static int palmas_gpadc_probe(struct platform_device *pdev) > adc->irq_auto_1 = platform_get_irq(pdev, 2); > ret = request_threaded_irq(adc->irq_auto_1, NULL, > palmas_gpadc_irq_auto, > - IRQF_ONESHOT | IRQF_EARLY_RESUME, > + IRQF_ONESHOT, > "palmas-adc-auto-1", adc); > if (ret < 0) { > dev_err(adc->dev, "request auto1 irq %d failed: %d\n", > -- -- You received this message because you are subscribed to "rtc-linux". Membership options at http://groups.google.com/group/rtc-linux . Please read http://groups.google.com/group/rtc-linux/web/checklist before submitting a driver. --- You received this message because you are subscribed to the Google Groups "rtc-linux" group. To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com. For more options, visit https://groups.google.com/d/optout.