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

[Krzysztof Kozlowski]

On Fri, May 15, 2026 at 09:06:08PM +0200, Andrew Lunn wrote:
> On Fri, May 15, 2026 at 06:15:50PM +0200, Krzysztof Kozlowski wrote:
> > On 15/05/2026 18:13, Selvamani Rajagopal wrote:
> > > According to OPEN Alliance 10BASE-T1x MACPHY Serial Interface (TC6)
> > > specification, interrupt type is active low, level triggered interrupt.
> > > 
> > > Fixes: ac49b950bea9 ("dt-bindings: net: add Microchip's LAN865X 10BASE-T1S MACPHY")
> > 
> > Drop, there is no bug here. That is only an example. Otherwise explain
> > how does this bug affect anything.
> 
> There is a tendency for DT developers to just copy/paste the example,
> and not really engage brain. So if the example is wrong, the .dts file
> will be wrong.

Which still is not a fix here. The DTS will be wrong. The code here has
zero impact currently, it is just an example without impact on
dt_bindings_check or any other actual process, thus any fixes tag is not
applicable.

You can as well remove the flag or write there 0.

Best regards,
Krzysztof