[PATCH v2 02/21] of: extra: Introduce ofnode_phy_is_fixed_link() API

[Bin Meng]

Hi Vladimir,

On Sat, Mar 13, 2021 at 9:03 PM Vladimir Oltean <olteanv at gmail.com> wrote:
>
> On Fri, Mar 12, 2021 at 09:35:43PM +0800, Bin Meng wrote:
> > +bool ofnode_phy_is_fixed_link(ofnode eth_node, ofnode *phy_node)
> > +{
> > +     bool found = false;
> > +     ofnode node, subnode;
> > +     int len;
> > +
> > +     /* new binding */
> > +     subnode = ofnode_find_subnode(eth_node, "fixed-link");
> > +     if (ofnode_valid(subnode)) {
> > +             node = subnode;
> > +             found = true;
> > +     }
> > +
> > +     /* old binding */
> > +     if (ofnode_get_property(eth_node, "fixed-link", &len) &&
>
> Maybe "else if" here? If an old-style and a new-style binding exist, we
> should prefer looking at the new one.
>

Agree. Will do in v3.

> And with that "else if", we could remove the "found" variable:

Regards,
Bin