Skip to content

Commit

Permalink
hwmon: (abx500) drop the use of IRQF_NO_SUSPEND
Browse files Browse the repository at this point in the history
The description in the driver states: "ABX500 does not provide auto ADC,
so to monitor the required temperatures, a periodic work is used. It is
more important to not wake up the CPU... If the chip gets too hot during
a sleep state it's most likely due to external factors, such as the
surrounding temperature and nothing can be done in S/W."

So it makes no sense to keep IRQs enabled as it need not be wakeup
source. This patch removes the use of IRQF_NO_SUSPEND flag

Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
  • Loading branch information
Sudeep Holla authored and Guenter Roeck committed Oct 14, 2015
1 parent 791432c commit 47e4b5e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/hwmon/abx500.c
Original file line number Diff line number Diff line change
Expand Up @@ -377,7 +377,7 @@ static int setup_irqs(struct platform_device *pdev)
}

ret = devm_request_threaded_irq(&pdev->dev, irq, NULL,
abx500_temp_irq_handler, IRQF_NO_SUSPEND, "abx500-temp", pdev);
abx500_temp_irq_handler, 0, "abx500-temp", pdev);
if (ret < 0)
dev_err(&pdev->dev, "Request threaded irq failed (%d)\n", ret);

Expand Down

0 comments on commit 47e4b5e

Please sign in to comment.