Re: [PATCH v4 01/13] dt-bindings: net: mediatek,net: update for mt7988

[Frank Wunderlich]

Am 18. Juni 2025 19:38:07 MESZ schrieb "Rob Herring (Arm)" <robh at kernel.org>:
>
>On Mon, 16 Jun 2025 11:58:11 +0200, Frank Wunderlich wrote:
>> From: Frank Wunderlich <frank-w at public-files.de>
>> 
>> Update binding for mt7988 which has 3 gmac and 2 reg items.
>> 
>> With RSS-IRQs the interrupt max-items is now 6. Add interrupt-names
>> to make them accessible by name.
>> 
>> Signed-off-by: Frank Wunderlich <frank-w at public-files.de>
>> ---
>> v4:
>> - increase max interrupts to 8 because of RSS/LRO interrupts
>> - dropped Robs RB due to this change
>> - allow interrupt names
>> - add interrupt-names without reserved IRQs on mt7988
>>   this requires mtk driver patch:
>>   https://patchwork.kernel.org/project/netdevbpf/patch/20250616080738.117993-2-linux@fw-web.de/
>> 
>> v2:
>> - change reg to list of items
>> ---
>>  .../devicetree/bindings/net/mediatek,net.yaml | 28 ++++++++++++++++---
>>  1 file changed, 24 insertions(+), 4 deletions(-)
>> 
>
>Reviewed-by: Rob Herring (Arm) <robh at kernel.org>

Thank you rob.

Have you seen my reponse to coverletter? Got info from mtk that reserved irqs are not unusable (only currently unused) and imho they should be upstreamed too but without a special meaning.

What do you think?

I would increase the interupts count to 8 and name the reserved irqs fe0..fe3. can i keep your RB or should i drop it again?


regards Frank