Re: [PATCH net v3 2/2] dt-bindings: net: updated interrupt type to be active low, level triggered

[Parthiban.Veerasooran]

On 04/06/26 10:30 pm, Andrew Lunn wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
>> I just made the assumption that what was in the example worked, and
>> wanted clarification that what the example was changed to actually
>> reflected something else that worked on the hardware in question before
>> I gave an ack.
> 
> That is the problem with edge interrupts when level should be
> used. They work %99.9 of the time, so many developers don't notice
> they have it wrong. And then one day it all stops because an interrupt
> got lost.
I agree with this point. But simply changing the interrupt type in the 
existing logic is not sufficient. Level‑triggered interrupts require 
careful handling in the driver, and with the current implementation I am 
observing issues under system load when using level‑triggered IRQs, as 
also discussed in another thread.

Best regards,
Parthiban V
> 
>      Andrew