Hi Andi,
On 10.02.2017 03:17, Andi Shyti wrote:
(...)
> +
> + err = devm_request_threaded_irq(&client->dev, client->irq,
> + NULL, stmfts_irq_handler,
> + IRQF_ONESHOT | IRQF_TRIGGER_LOW,
> + "stmfts_irq", sdata);
> + if (err)
> + return err;
> +
> + /*
> + * Disable irq they, they are not needed at this stage.
> + * One possible case when an IRQ can be already rased is e.g. if the
> + * regulator is set as always on and the stmfts device sends an IRQ as
> + * soon as it gets powered, de-synchronizing the power on sequence.
> + * During power on, the device will be reset and all the initialization
> + * IRQ will be resent.
> + */
> + disable_irq(sdata->client->irq);
It is safer to set:
irq_set_status_flags(client->irq, IRQ_NOAUTOEN);
before devm_request_threaded_irq
Regards
Andrzej
