Hi Rob, On 7/21/22 7:35 PM, Sean Anderson wrote: > What about > > phys = <&serdes1_lane1>; > > and then under the serdes node do something like > > serdes1: phy@foo { > ... > > serdes1_lane1 { > first-lane = <1>; > > sgmii { > fsl,pccr = <0x8>; > fsl,idx = <2>; > fsl,cfg = <1>; > fsl,proto = "sgmii"; > // or PHY_TYPE_SGMII > }; > > qsgmii { > ... > }; > > xfi { > ... > }; > }; > }; > > and this way you could have something like a fsl,reserved property to > deal with not-yet-supported lanes. And this could be added piecemeal by > board configs. Does this sound good? I would like to start working on v4 of this series, and reworking the binding will be a big part of that. Am I heading in the right direction? This seems to be a more common approach (e.g. mediatek,tphy). --Sean