RE: [PATCH net 2/2] dt-bindings: net: updated interrupt type to be active low, level triggered
From: Selvamani Rajagopal
Date: Fri May 15 2026 - 12:55:23 EST
>
>
> This Message Is From an External Sender
> This message came from outside your organization.
>
> 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.
Couple of reasons:
1) Agree. This is just an example. The reason I changed the YAML file is, due to the change in corresponding the source code change patch 1/2. I thought this entry should match with what is used in the source code.
2) We have our own TC6 compliant driver that is under review net-next repo that is going to have these changes. We were told to submit the fixes (specifically patch 1/2) to the existing code in net repo, not net-next repo.
>
> Also, did you consider the board layout?
I believe standard (OPEN Alliance) calls for active low, level triggered interrupt. That is the reason for these two patches.
BTW, Just saw Andrew's response on this.
>
> Best regards,
> Krzysztof