Re: Input: ep93xx_keypad: Checking for a failed platform_get_irq()call in ep93xx_keypad_probe()

From: Tang Bin
Date: Thu Apr 09 2020 - 22:51:48 EST


Hi Dmitry:

On 2020/4/10 4:48, Dmitry Torokhov wrote:
Platform code historically allowed creating IRQ resources with IRQ
number 0 to indicate "no interrupt assigned", so this driver tries to
filter out such conditions. The negative IRQs (errors) will be rejected
by request_irq() but I guess we can lose -EPROBE_DEFER. We could do

if (keypad->irq <= 0) {
err = keypad->irq ?: -ENXIO : keypad->irq;
goto failed_free;
}


I have been aware of this problem for a few days, and by doing experiments on the hardware, I have found the following ways that maybe suitableï

ÂÂÂ if (keypad->irq <= 0) {
ÂÂ ÂÂÂ Âerr = keypad->irq ? : -ENXIO;
ÂÂ ÂÂÂ Âgoto failed_free;
ÂÂ Â}
ÂÂ Âor
ÂÂ Âif (keypad->irq <= 0) {
ÂÂ ÂÂÂ Âerr = keypad->irq < 0 ? keypad->irq : -ENXIO;
ÂÂ ÂÂÂ Âgoto failed_free;
ÂÂ Â}

If you think it's usefull, I will send this patch to fix it.

Thanks

Tang Bin