Subject: Re: [PATCH net-next v11 4/9] dt-bindings: net: convert hisi-femac.txt to YAML

From: Jakub Kicinski <[email-masked]>
Date: Mon, 11 Mar 2024 09:03:41 -0700

On Sat, 9 Mar 2024 20:30:21 +0800 Yang Xiwen wrote:
> > +  clock-names:
> > +    items:
> > +      - const: mac
> > +      - const: macif
> > +      - const: phy  
> 
> Still not very correct here. In downstream the core can also have an 
> external PHY. The internal phy is also optional. So maybe this clock 
> should be optional.

You are responding to yourself 4 min after posting?
What is the purpose of your comments?