[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