Re: [PATCH net-next v7 1/3] dt-bindings: net: wiznet,w5100: convert to DT schema

From: Arthur Crepin Leblond

Date: Fri Sep 11 2026 - 11:41:58 EST


On Wed, Sep 09, 2026 at 12:38:58PM +0000, netdev-bot+sashiko@xxxxxxxxxx wrote:
Would it make sense to either describe/constrain the type as level-low in
the schema, or have the driver pass IRQF_TRIGGER_NONE and honour the DT
type?

I'll change the falling edge to low.

Would an allOf/if:properties:compatible:contains: block per part be
better here, or a description explaining why 80 MHz is safe for all
three?

I would maybe rephrase the description then but a allOf sounds
overkill, w5500 and w5200 seem to support theoretically up to 80 MHz
but going that high will probably cause some signal distortion. I
would consider this a user mistake to use an unstable frequency.

Arthur