On 04/21/2017 04:24 PM, Icenowy Zheng wrote: > From: Icenowy Zheng <icenowy@xxxxxxxx> > > Some RTL8211E Ethernet PHY have an issue that needs a workaround > indicated with device tree. > > Add the binding for a property that indicates this workaround. > > Signed-off-by: Icenowy Zheng <icenowy@xxxxxxxx> > --- > .../devicetree/bindings/net/realtek,rtl8211e.txt | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/realtek,rtl8211e.txt > > diff --git a/Documentation/devicetree/bindings/net/realtek,rtl8211e.txt b/Documentation/devicetree/bindings/net/realtek,rtl8211e.txt > new file mode 100644 > index 000000000000..c1913301bfe8 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/realtek,rtl8211e.txt > @@ -0,0 +1,22 @@ > +Realtek RTL8211E Ethernet PHY > + > +One batch of RTL8211E is slight broken, that needs some special (and > +full of magic numbers) tweaking in order to make GbE to operate properly. > +The only well-known board that used the broken batch is Pine64+. > +Configure it through an Ethernet OF device node. > + > +Optional properties: > + > +- realtek,disable-rx-delay: > + If set, RX delay will be completely disabled (according to Realtek). This > + will affect the performance on non-broken boards. > + default: do not disable RX delay. Please don't introduce custom properties to do that, instead correct specify the "phy-mode" such that it is e.g: "rgmii-txid" which indicates that there should be no RX internal delay, but a TX internal delay added by the PHY. -- Florian -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html