RE: [PATCH net-next] dt-bindings: net: Remove redundant ethernet-phy-ieee802.3-c22 fallback

[Biju Das]

Hi Jakub Kicinski,

Thanks for the feedback.

> -----Original Message-----
> From: Jakub Kicinski <kuba@xxxxxxxxxx>
> Sent: 19 March 2026 00:05
> Subject: Re: [PATCH net-next] dt-bindings: net: Remove redundant ethernet-phy-ieee802.3-c22 fallback
> 
> On Fri, 13 Mar 2026 13:06:21 +0000 Biju wrote:
> > From: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> >
> > Drop the ethernet-phy-ieee802.3-c22 compatible string from ethernet
> > PHY bindings and their examples. The c22 fallback is implicitly
> > assumed for PHY ID-based compatible strings and does not need to be
> > stated explicitly, unlike c45 which requires opt-in.
> >
> > Remove the c22 pattern from the ethernet-phy.yaml schema and update
> > examples in amlogic,g12a-mdio-mux, mscc-phy-vsc8531, renesas,ether,
> > and renesas,etheravb bindings accordingly.
> 
> Maybe a noob question but why are you not updating all the other ones?
> 
> $ git grep --files-with-matches 'ethernet-phy-ieee802.3-c22' -- \
> 	Documentation/devicetree/bindings/net/ | \
> 	wc -l
> 16

The other files have only 'compatible = "ethernet-phy-ieee802.3-c22"'

Whereas this patch removes the fallback pattern based on
Rob's suggestion [1]

compatible = "ethernet-phy-id0022.1622",                     
		 "ethernet-phy-ieee802.3-c22"; 


Please let me know, should I drop all 'ethernet-phy-ieee802.3-c22' compatibles? 

[1] https://lore.kernel.org/all/CAL_JsqJK3brXxSeD-U42PBCeHoxUhWJYho3ZWj__t5C1LL-n-Q@xxxxxxxxxxxxxx/

Cheers,
Biju