Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1344349 > unrolled thread

[PATCH 0/7] mfd: Drop IRQF_EARLY_RESUME flag

Started byGrygorii Strashko <grygorii.strashko@ti.com>
First post2016-02-26 16:50 +0100
Last post2016-02-29 00:50 +0100
Articles 11 — 4 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/7] mfd: Drop IRQF_EARLY_RESUME flag Grygorii Strashko <grygorii.strashko@ti.com> - 2016-02-26 16:50 +0100
    [PATCH 4/7] rtc: tps65910: Drop IRQF_EARLY_RESUME flag Grygorii Strashko <grygorii.strashko@ti.com> - 2016-02-26 16:50 +0100
    [PATCH 5/7] rtc: tps80031: Drop IRQF_EARLY_RESUME flag Grygorii Strashko <grygorii.strashko@ti.com> - 2016-02-26 16:50 +0100
    [PATCH 1/7] extcon: palmas: Drop IRQF_EARLY_RESUME flag Grygorii Strashko <grygorii.strashko@ti.com> - 2016-02-26 16:50 +0100
      Re: [PATCH 1/7] extcon: palmas: Drop IRQF_EARLY_RESUME flag Chanwoo Choi <cw00.choi@samsung.com> - 2016-02-29 03:30 +0100
    [PATCH 7/7] iio: adc: palmas: Drop IRQF_EARLY_RESUME flag Grygorii Strashko <grygorii.strashko@ti.com> - 2016-02-26 16:50 +0100
      Re: [PATCH 7/7] iio: adc: palmas: Drop IRQF_EARLY_RESUME flag Jonathan Cameron <jic23@kernel.org> - 2016-02-27 18:30 +0100
    [PATCH 6/7] rtc: as3722: Drop IRQF_EARLY_RESUME flag Grygorii Strashko <grygorii.strashko@ti.com> - 2016-02-26 16:50 +0100
    [PATCH 2/7] rtc: palmas: Drop IRQF_EARLY_RESUME flag Grygorii Strashko <grygorii.strashko@ti.com> - 2016-02-26 16:50 +0100
    [PATCH 3/7] rtc: tps6586x: Drop IRQF_EARLY_RESUME flag Grygorii Strashko <grygorii.strashko@ti.com> - 2016-02-26 16:50 +0100
    Re: [PATCH 0/7] mfd: Drop IRQF_EARLY_RESUME flag Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2016-02-29 00:50 +0100

#1344349 — [PATCH 0/7] mfd: Drop IRQF_EARLY_RESUME flag

FromGrygorii Strashko <grygorii.strashko@ti.com>
Date2016-02-26 16:50 +0100
Subject[PATCH 0/7] mfd: Drop IRQF_EARLY_RESUME flag
Message-ID<r6sOt-5pp-1@gated-at.bofh.it>
The IRQF_EARLY_RESUME flag is not required for nested irqs anymore,
since commit 3c646f2c6aa9 ("genirq: Don't suspend nested_thread irqs
over system suspend") was merged.
Hence, this flag can be dropped for MFD cell's devices where MFD interrupt
controller defines nested IRQs as nested threaded IRQs.

Cc: Lee Jones <lee.jones@linaro.org>
Cc: Alessandro Zummo <a.zummo@towertech.it>
Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: MyungJoo Ham <myungjoo.ham@samsung.com>
Cc: Chanwoo Choi <cw00.choi@samsung.com>
Cc: Hartmut Knaack <knaack.h@gmx.de>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Peter Meerwald <pmeerw@pmeerw.net>
Cc: Tony Lindgren <tony@atomide.com>
Cc: Nishanth Menon <nm@ti.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>

Grygorii Strashko (7):
  extcon: palmas: Drop IRQF_EARLY_RESUME flag
  rtc: palmas: Drop IRQF_EARLY_RESUME flag
  rtc: tps6586x: Drop IRQF_EARLY_RESUME flag
  rtc: tps65910: Drop IRQF_EARLY_RESUME flag
  rtc: tps80031: Drop IRQF_EARLY_RESUME flag
  rtc: as3722: Drop IRQF_EARLY_RESUME flag
  iio: adc: palmas: Drop IRQF_EARLY_RESUME flag

 drivers/extcon/extcon-palmas.c | 4 ++--
 drivers/iio/adc/palmas_gpadc.c | 6 +++---
 drivers/rtc/rtc-as3722.c       | 2 +-
 drivers/rtc/rtc-palmas.c       | 3 +--
 drivers/rtc/rtc-tps6586x.c     | 2 +-
 drivers/rtc/rtc-tps65910.c     | 2 +-
 drivers/rtc/rtc-tps80031.c     | 2 +-
 7 files changed, 10 insertions(+), 11 deletions(-)

-- 
2.7.2

[toc] | [next] | [standalone]

#1344350 — [PATCH 4/7] rtc: tps65910: Drop IRQF_EARLY_RESUME flag

FromGrygorii Strashko <grygorii.strashko@ti.com>
Date2016-02-26 16:50 +0100
Subject[PATCH 4/7] rtc: tps65910: Drop IRQF_EARLY_RESUME flag
Message-ID<r6sOt-5pp-3@gated-at.bofh.it>
In reply to#1344349
tps65910 RTC IRQ is nested threaded and wired to the tps65910 inerrupt
controller. So, 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: Alessandro Zummo <a.zummo@towertech.it>
Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Tony Lindgren <tony@atomide.com>
Cc: Nishanth Menon <nm@ti.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
 drivers/rtc/rtc-tps65910.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-tps65910.c b/drivers/rtc/rtc-tps65910.c
index f42aa2b..5a3d53c 100644
--- a/drivers/rtc/rtc-tps65910.c
+++ b/drivers/rtc/rtc-tps65910.c
@@ -268,7 +268,7 @@ static int tps65910_rtc_probe(struct platform_device *pdev)
 	}
 
 	ret = devm_request_threaded_irq(&pdev->dev, irq, NULL,
-		tps65910_rtc_interrupt, IRQF_TRIGGER_LOW | IRQF_EARLY_RESUME,
+		tps65910_rtc_interrupt, IRQF_TRIGGER_LOW,
 		dev_name(&pdev->dev), &pdev->dev);
 	if (ret < 0) {
 		dev_err(&pdev->dev, "IRQ is not free.\n");
-- 
2.7.2

[toc] | [prev] | [next] | [standalone]

#1344355 — [PATCH 5/7] rtc: tps80031: Drop IRQF_EARLY_RESUME flag

FromGrygorii Strashko <grygorii.strashko@ti.com>
Date2016-02-26 16:50 +0100
Subject[PATCH 5/7] rtc: tps80031: Drop IRQF_EARLY_RESUME flag
Message-ID<r6sOu-5pp-21@gated-at.bofh.it>
In reply to#1344349
tps80031 RTC IRQ is nested threaded and wired to the tps80031
inerrupt controller. So, 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: Alessandro Zummo <a.zummo@towertech.it>
Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Nishanth Menon <nm@ti.com>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
 drivers/rtc/rtc-tps80031.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-tps80031.c b/drivers/rtc/rtc-tps80031.c
index 27e254c..737f26e 100644
--- a/drivers/rtc/rtc-tps80031.c
+++ b/drivers/rtc/rtc-tps80031.c
@@ -287,7 +287,7 @@ static int tps80031_rtc_probe(struct platform_device *pdev)
 
 	ret = devm_request_threaded_irq(&pdev->dev, rtc->irq, NULL,
 			tps80031_rtc_irq,
-			IRQF_ONESHOT | IRQF_EARLY_RESUME,
+			IRQF_ONESHOT,
 			dev_name(&pdev->dev), rtc);
 	if (ret < 0) {
 		dev_err(&pdev->dev, "request IRQ:%d failed, err = %d\n",
-- 
2.7.2

[toc] | [prev] | [next] | [standalone]

#1344356 — [PATCH 1/7] extcon: palmas: Drop IRQF_EARLY_RESUME flag

FromGrygorii Strashko <grygorii.strashko@ti.com>
Date2016-02-26 16:50 +0100
Subject[PATCH 1/7] extcon: palmas: Drop IRQF_EARLY_RESUME flag
Message-ID<r6sOu-5pp-23@gated-at.bofh.it>
In reply to#1344349
Palams extcon IRQs are nested threaded and wired to the Palmas
inerrupt controller. So, 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: MyungJoo Ham <myungjoo.ham@samsung.com>
Cc: Chanwoo Choi <cw00.choi@samsung.com>
Cc: Tony Lindgren <tony@atomide.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Roger Quadros <rogerq@ti.com>
Cc: Nishanth Menon <nm@ti.com>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
 drivers/extcon/extcon-palmas.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/extcon/extcon-palmas.c b/drivers/extcon/extcon-palmas.c
index 93c30a8..0a861b3 100644
--- a/drivers/extcon/extcon-palmas.c
+++ b/drivers/extcon/extcon-palmas.c
@@ -266,7 +266,7 @@ static int palmas_usb_probe(struct platform_device *pdev)
 				palmas_usb->id_irq,
 				NULL, palmas_id_irq_handler,
 				IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING |
-				IRQF_ONESHOT | IRQF_EARLY_RESUME,
+				IRQF_ONESHOT,
 				"palmas_usb_id", palmas_usb);
 		if (status < 0) {
 			dev_err(&pdev->dev, "can't get IRQ %d, err %d\n",
@@ -304,7 +304,7 @@ static int palmas_usb_probe(struct platform_device *pdev)
 				palmas_usb->vbus_irq, NULL,
 				palmas_vbus_irq_handler,
 				IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING |
-				IRQF_ONESHOT | IRQF_EARLY_RESUME,
+				IRQF_ONESHOT,
 				"palmas_usb_vbus", palmas_usb);
 		if (status < 0) {
 			dev_err(&pdev->dev, "can't get IRQ %d, err %d\n",
-- 
2.7.2

[toc] | [prev] | [next] | [standalone]

#1345438 — Re: [PATCH 1/7] extcon: palmas: Drop IRQF_EARLY_RESUME flag

FromChanwoo Choi <cw00.choi@samsung.com>
Date2016-02-29 03:30 +0100
SubjectRe: [PATCH 1/7] extcon: palmas: Drop IRQF_EARLY_RESUME flag
Message-ID<r7lKW-3F2-1@gated-at.bofh.it>
In reply to#1344356
Hi Grygorii,

On 2016년 02월 27일 00:42, Grygorii Strashko wrote:
> Palams extcon IRQs are nested threaded and wired to the Palmas
> inerrupt controller. So, 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: MyungJoo Ham <myungjoo.ham@samsung.com>
> Cc: Chanwoo Choi <cw00.choi@samsung.com>
> Cc: Tony Lindgren <tony@atomide.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Roger Quadros <rogerq@ti.com>
> Cc: Nishanth Menon <nm@ti.com>
> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
> ---
>  drivers/extcon/extcon-palmas.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/extcon/extcon-palmas.c b/drivers/extcon/extcon-palmas.c
> index 93c30a8..0a861b3 100644
> --- a/drivers/extcon/extcon-palmas.c
> +++ b/drivers/extcon/extcon-palmas.c
> @@ -266,7 +266,7 @@ static int palmas_usb_probe(struct platform_device *pdev)
>  				palmas_usb->id_irq,
>  				NULL, palmas_id_irq_handler,
>  				IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING |
> -				IRQF_ONESHOT | IRQF_EARLY_RESUME,
> +				IRQF_ONESHOT,
>  				"palmas_usb_id", palmas_usb);
>  		if (status < 0) {
>  			dev_err(&pdev->dev, "can't get IRQ %d, err %d\n",
> @@ -304,7 +304,7 @@ static int palmas_usb_probe(struct platform_device *pdev)
>  				palmas_usb->vbus_irq, NULL,
>  				palmas_vbus_irq_handler,
>  				IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING |
> -				IRQF_ONESHOT | IRQF_EARLY_RESUME,
> +				IRQF_ONESHOT,
>  				"palmas_usb_vbus", palmas_usb);
>  		if (status < 0) {
>  			dev_err(&pdev->dev, "can't get IRQ %d, err %d\n",
> 

Applied it on extcon git.

Thanks,
Chanwoo Choi

[toc] | [prev] | [next] | [standalone]

#1344357 — [PATCH 7/7] iio: adc: palmas: Drop IRQF_EARLY_RESUME flag

FromGrygorii Strashko <grygorii.strashko@ti.com>
Date2016-02-26 16:50 +0100
Subject[PATCH 7/7] iio: adc: palmas: Drop IRQF_EARLY_RESUME flag
Message-ID<r6sOu-5pp-25@gated-at.bofh.it>
In reply to#1344349
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 <knaack.h@gmx.de>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Peter Meerwald <pmeerw@pmeerw.net>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Nishanth Menon <nm@ti.com>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
 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",
-- 
2.7.2

[toc] | [prev] | [next] | [standalone]

#1345050 — Re: [PATCH 7/7] iio: adc: palmas: Drop IRQF_EARLY_RESUME flag

FromJonathan Cameron <jic23@kernel.org>
Date2016-02-27 18:30 +0100
SubjectRe: [PATCH 7/7] iio: adc: palmas: Drop IRQF_EARLY_RESUME flag
Message-ID<r6QQO-6qw-1@gated-at.bofh.it>
In reply to#1344357
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 <knaack.h@gmx.de>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Peter Meerwald <pmeerw@pmeerw.net>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Nishanth Menon <nm@ti.com>
> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
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",
>

[toc] | [prev] | [next] | [standalone]

#1344358 — [PATCH 6/7] rtc: as3722: Drop IRQF_EARLY_RESUME flag

FromGrygorii Strashko <grygorii.strashko@ti.com>
Date2016-02-26 16:50 +0100
Subject[PATCH 6/7] rtc: as3722: Drop IRQF_EARLY_RESUME flag
Message-ID<r6sOu-5pp-29@gated-at.bofh.it>
In reply to#1344349
as3722 RTC IRQ is nested threaded and wired to the as3722 inerrupt
controller. So, 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: Alessandro Zummo <a.zummo@towertech.it>
Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Nishanth Menon <nm@ti.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
 drivers/rtc/rtc-as3722.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-as3722.c b/drivers/rtc/rtc-as3722.c
index 56cc582..6ef0c88 100644
--- a/drivers/rtc/rtc-as3722.c
+++ b/drivers/rtc/rtc-as3722.c
@@ -210,7 +210,7 @@ static int as3722_rtc_probe(struct platform_device *pdev)
 	dev_info(&pdev->dev, "RTC interrupt %d\n", as3722_rtc->alarm_irq);
 
 	ret = devm_request_threaded_irq(&pdev->dev, as3722_rtc->alarm_irq, NULL,
-			as3722_alarm_irq, IRQF_ONESHOT | IRQF_EARLY_RESUME,
+			as3722_alarm_irq, IRQF_ONESHOT,
 			"rtc-alarm", as3722_rtc);
 	if (ret < 0) {
 		dev_err(&pdev->dev, "Failed to request alarm IRQ %d: %d\n",
-- 
2.7.2

[toc] | [prev] | [next] | [standalone]

#1344360 — [PATCH 2/7] rtc: palmas: Drop IRQF_EARLY_RESUME flag

FromGrygorii Strashko <grygorii.strashko@ti.com>
Date2016-02-26 16:50 +0100
Subject[PATCH 2/7] rtc: palmas: Drop IRQF_EARLY_RESUME flag
Message-ID<r6sOu-5pp-33@gated-at.bofh.it>
In reply to#1344349
Palams RTC IRQ is nested threaded and wired to the Palmas inerrupt
controller. So, 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: Alessandro Zummo <a.zummo@towertech.it>
Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: Tony Lindgren <tony@atomide.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Nishanth Menon <nm@ti.com>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
 drivers/rtc/rtc-palmas.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/rtc/rtc-palmas.c b/drivers/rtc/rtc-palmas.c
index 7ea2c47..6080e0e 100644
--- a/drivers/rtc/rtc-palmas.c
+++ b/drivers/rtc/rtc-palmas.c
@@ -311,8 +311,7 @@ static int palmas_rtc_probe(struct platform_device *pdev)
 
 	ret = devm_request_threaded_irq(&pdev->dev, palmas_rtc->irq, NULL,
 			palmas_rtc_interrupt,
-			IRQF_TRIGGER_LOW | IRQF_ONESHOT |
-			IRQF_EARLY_RESUME,
+			IRQF_TRIGGER_LOW | IRQF_ONESHOT,
 			dev_name(&pdev->dev), palmas_rtc);
 	if (ret < 0) {
 		dev_err(&pdev->dev, "IRQ request failed, err = %d\n", ret);
-- 
2.7.2

[toc] | [prev] | [next] | [standalone]

#1344366 — [PATCH 3/7] rtc: tps6586x: Drop IRQF_EARLY_RESUME flag

FromGrygorii Strashko <grygorii.strashko@ti.com>
Date2016-02-26 16:50 +0100
Subject[PATCH 3/7] rtc: tps6586x: Drop IRQF_EARLY_RESUME flag
Message-ID<r6sOv-5pp-51@gated-at.bofh.it>
In reply to#1344349
tps6586x RTC IRQ is nested threaded and wired to the tps6586x inerrupt
controller. So, 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: Alessandro Zummo <a.zummo@towertech.it>
Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Nishanth Menon <nm@ti.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
 drivers/rtc/rtc-tps6586x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/rtc/rtc-tps6586x.c b/drivers/rtc/rtc-tps6586x.c
index 3b6ce80..e404faa 100644
--- a/drivers/rtc/rtc-tps6586x.c
+++ b/drivers/rtc/rtc-tps6586x.c
@@ -286,7 +286,7 @@ static int tps6586x_rtc_probe(struct platform_device *pdev)
 
 	ret = devm_request_threaded_irq(&pdev->dev, rtc->irq, NULL,
 				tps6586x_rtc_irq,
-				IRQF_ONESHOT | IRQF_EARLY_RESUME,
+				IRQF_ONESHOT,
 				dev_name(&pdev->dev), rtc);
 	if (ret < 0) {
 		dev_err(&pdev->dev, "request IRQ(%d) failed with ret %d\n",
-- 
2.7.2

[toc] | [prev] | [next] | [standalone]

#1345422

FromAlexandre Belloni <alexandre.belloni@free-electrons.com>
Date2016-02-29 00:50 +0100
Message-ID<r7jg5-1IK-1@gated-at.bofh.it>
In reply to#1344349
Hi,

On 26/02/2016 at 17:42:50 +0200, Grygorii Strashko wrote :
> The IRQF_EARLY_RESUME flag is not required for nested irqs anymore,
> since commit 3c646f2c6aa9 ("genirq: Don't suspend nested_thread irqs
> over system suspend") was merged.
> Hence, this flag can be dropped for MFD cell's devices where MFD interrupt
> controller defines nested IRQs as nested threaded IRQs.
> 
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Alessandro Zummo <a.zummo@towertech.it>
> Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> Cc: MyungJoo Ham <myungjoo.ham@samsung.com>
> Cc: Chanwoo Choi <cw00.choi@samsung.com>
> Cc: Hartmut Knaack <knaack.h@gmx.de>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Peter Meerwald <pmeerw@pmeerw.net>
> Cc: Tony Lindgren <tony@atomide.com>
> Cc: Nishanth Menon <nm@ti.com>
> Cc: Laxman Dewangan <ldewangan@nvidia.com>
> 
> Grygorii Strashko (7):
>   extcon: palmas: Drop IRQF_EARLY_RESUME flag
>   rtc: palmas: Drop IRQF_EARLY_RESUME flag
>   rtc: tps6586x: Drop IRQF_EARLY_RESUME flag
>   rtc: tps65910: Drop IRQF_EARLY_RESUME flag
>   rtc: tps80031: Drop IRQF_EARLY_RESUME flag
>   rtc: as3722: Drop IRQF_EARLY_RESUME flag
>   iio: adc: palmas: Drop IRQF_EARLY_RESUME flag
> 
>  drivers/extcon/extcon-palmas.c | 4 ++--
>  drivers/iio/adc/palmas_gpadc.c | 6 +++---
>  drivers/rtc/rtc-as3722.c       | 2 +-
>  drivers/rtc/rtc-palmas.c       | 3 +--
>  drivers/rtc/rtc-tps6586x.c     | 2 +-
>  drivers/rtc/rtc-tps65910.c     | 2 +-
>  drivers/rtc/rtc-tps80031.c     | 2 +-
>  7 files changed, 10 insertions(+), 11 deletions(-)
> 

I applied all the rtc related patches, thanks!

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web