[PATCH 0/2] net: phy: realtek: fix RTL8211F TX-delay handling
Neil Armstrong
narmstrong at baylibre.com
Mon Nov 28 05:25:46 PST 2016
On 11/25/2016 02:11 PM, Martin Blumenstingl wrote:
> The RTL8211F PHY driver currently enables the TX-delay only when the
> phy-mode is PHY_INTERFACE_MODE_RGMII. This is incorrect, because there
> are three RGMII variations of the phy-mode which explicitly request the
> PHY to enable the RX and/or TX delay, while PHY_INTERFACE_MODE_RGMII
> specifies that the PHY should disable the RX and/or TX delays.
>
> Additionally to the RTL8211F PHY driver change this contains a small
> update to the phy-mode documentation to clarify the purpose of the
> RGMII phy-modes.
> While this may not be perfect yet it's at least a start. Please feel
> free to drop this patch from this series and send an improved version
> yourself.
>
> These patches are the results of recent discussions, see [0]
>
> [0] http://lists.infradead.org/pipermail/linux-amlogic/2016-November/001688.html
>
> Martin Blumenstingl (2):
> Documentation: devicetree: clarify usage of the RGMII phy-modes
> net: phy: realtek: fix enabling of the TX-delay for RTL8211F
>
> Documentation/devicetree/bindings/net/ethernet.txt | 24 ++++++++++++++++++----
> drivers/net/phy/realtek.c | 20 ++++++++++--------
> 2 files changed, 32 insertions(+), 12 deletions(-)
>
Along the "stmmac: dwmac-meson8b: configurable RGMII TX delay" v2 patchset,
fixes the link issue on my Nexbox A1 board and does not break my Amlogic P230 board.
Tested-by: Neil Armstrong <narmstrong at baylibre.com>
More information about the linux-amlogic
mailing list