Re: [PATCH net 1/2] net: ethernet: oa_tc6: Interrupt is active low, level triggered.

[Ciprian Regus]

>        ret = devm_request_irq(&tc6->spi->dev, tc6->spi->irq, oa_tc6_macphy_isr,
>-                              IRQF_TRIGGER_FALLING, dev_name(&tc6->spi->dev),
>+                              IRQF_TRIGGER_LOW, dev_name(&tc6->spi->dev),
>                               tc6);

oa_tc6_macphy_isr() doesn't clear the interrupt condition or disable the
interrupt. So it will be triggered several times until the oa tc6
thread gets to run. Should we use something like this instead?

devm_request_threaded_irq(..., IRQF_TRIGGER_LOW | IRQF_ONESHOT, ...)

(together with some rework to call oa_tc6_try_spi_transfer() from the
threaded irq part).