Hi Rob, Thanks for the review and sorry I din't reply earlier. <snip> > > +- vsc8531,rx-delay : RGMII RX delay. Allowed values are defined in > > 'vsc8531' is not a vendor. The form is <vendor>,<propname>. I was just aligning it with existing optional property names in this document. Could you please let me know if mscc,rx-delay is more appropriate? Regards, Harini