power: max8903_charger: set IRQF_ONESHOT if no primary handler is specified
If no primary handler is specified for threaded_irq then a default one is assigned which always returns IRQ_WAKE_THREAD. This handler requires the IRQF_ONESHOT, because the source of interrupt is not disabled. Signed-off-by: Saurabh Sengar <saurabh.truth@gmail.com> Signed-off-by: Sebastian Reichel <sre@kernel.org>
This commit is contained in:
parent
c4c0edfbf8
commit
d5fdfedc0e
|
@ -293,7 +293,7 @@ static int max8903_probe(struct platform_device *pdev)
|
|||
ret = devm_request_threaded_irq(dev, gpio_to_irq(pdata->dok),
|
||||
NULL, max8903_dcin,
|
||||
IRQF_TRIGGER_FALLING |
|
||||
IRQF_TRIGGER_RISING,
|
||||
IRQF_TRIGGER_RISING | IRQF_ONESHOT,
|
||||
"MAX8903 DC IN", data);
|
||||
if (ret) {
|
||||
dev_err(dev, "Cannot request irq %d for DC (%d)\n",
|
||||
|
@ -306,7 +306,7 @@ static int max8903_probe(struct platform_device *pdev)
|
|||
ret = devm_request_threaded_irq(dev, gpio_to_irq(pdata->uok),
|
||||
NULL, max8903_usbin,
|
||||
IRQF_TRIGGER_FALLING |
|
||||
IRQF_TRIGGER_RISING,
|
||||
IRQF_TRIGGER_RISING | IRQF_ONESHOT,
|
||||
"MAX8903 USB IN", data);
|
||||
if (ret) {
|
||||
dev_err(dev, "Cannot request irq %d for USB (%d)\n",
|
||||
|
@ -319,7 +319,7 @@ static int max8903_probe(struct platform_device *pdev)
|
|||
ret = devm_request_threaded_irq(dev, gpio_to_irq(pdata->flt),
|
||||
NULL, max8903_fault,
|
||||
IRQF_TRIGGER_FALLING |
|
||||
IRQF_TRIGGER_RISING,
|
||||
IRQF_TRIGGER_RISING | IRQF_ONESHOT,
|
||||
"MAX8903 Fault", data);
|
||||
if (ret) {
|
||||
dev_err(dev, "Cannot request irq %d for Fault (%d)\n",
|
||||
|
|
Loading…
Reference in a new issue