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