> Can you also explain how did you figure out that additional PHY properties > are needed? At first glance, based on description of motorcomm,keep-pll-enabled, > it's not immediately clear why it should be needed. Same goes for second > property. This is based on a search for 'motorcomm,clk-out-frequency-hz = <125000000>' in the kernel. Refer to device tree of other devices with the same properties. After a simple short test, it seems that it is unnecessary. I will delete these two properties in patch v2. > 1800 ps delay basically means that rgmii is not correct type and rgmii-rxid > should be used instead. Indeed, schematic confirms that's the case. With that, > allwinner,rx-delay-ps can be 0 and thus ommited from DT file. Thanks, rgmii-rxid works for me. Will be corrected in patch v2. Thanks, Chukun -- 2.25.1
