From: Jerome Brunet <jbrunet@baylibre.com>
To: Anand Moon <linux.amoon@gmail.com>
Cc: netdev@vger.kernel.org, devicetree <devicetree@vger.kernel.org>,
Florian Fainelli <f.fainelli@gmail.com>,
Alexandre TORGUE <alexandre.torgue@st.com>,
Neil Armstrong <narmstrong@baylibre.com>,
Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
Kevin Hilman <khilman@baylibre.com>,
Linux Kernel <linux-kernel@vger.kernel.org>,
Andre Roth <neolynx@gmail.com>,
linux-amlogic@lists.infradead.org,
Carlo Caione <carlo@caione.org>,
Giuseppe Cavallaro <peppe.cavallaro@st.com>,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH net 1/3] net: phy: realtek: add eee advertisement disable options
Date: Thu, 17 Nov 2016 11:20:55 +0100 [thread overview]
Message-ID: <1479378055.17538.57.camel@baylibre.com> (raw)
In-Reply-To: <CANAwSgTjG8G0U+A1p7hOKND9rjY4BzZfPgyWWHAEryYkHj_UOw@mail.gmail.com>
On Wed, 2016-11-16 at 22:36 +0530, Anand Moon wrote:
> Hi Jerome.
>
> On 15 November 2016 at 19:59, Jerome Brunet <jbrunet@baylibre.com>
> wrote:
> >
> > On some platforms, energy efficient ethernet with rtl8211 devices
> > is
> > causing issue, like throughput drop or broken link.
> >
> > This was reported on the OdroidC2 (DWMAC + RTL8211F). While the
> > issue root
> > cause is not fully understood yet, disabling EEE advertisement
> > prevent auto
> > negotiation from enabling EEE.
> >
> > This patch provides options to disable 1000T and 100TX EEE
> > advertisement
> > individually for the realtek phys supporting this feature.
> >
> > Reported-by: Martin Blumenstingl <martin.blumenstingl@googlemail.co
> > m>
> > Cc: Giuseppe Cavallaro <peppe.cavallaro@st.com>
> > Cc: Alexandre TORGUE <alexandre.torgue@st.com>
> > Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
> > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> > Tested-by: Andre Roth <neolynx@gmail.com>
> > ---
> > drivers/net/phy/realtek.c | 65
> > ++++++++++++++++++++++++++++++++++++++++++++++-
> > 1 file changed, 64 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/phy/realtek.c b/drivers/net/phy/realtek.c
> > index aadd6e9f54ad..77235fd5faaf 100644
> > --- a/drivers/net/phy/realtek.c
> > +++ b/drivers/net/phy/realtek.c
> > @@ -15,6 +15,12 @@
> > */
> > #include <linux/phy.h>
> > #include <linux/module.h>
> > +#include <linux/of.h>
> > +
> > +struct rtl8211x_phy_priv {
> > + bool eee_1000t_disable;
> > + bool eee_100tx_disable;
> > +};
> >
> > #define RTL821x_PHYSR 0x11
> > #define RTL821x_PHYSR_DUPLEX 0x2000
> > @@ -93,12 +99,44 @@ static int rtl8211f_config_intr(struct
> > phy_device *phydev)
> > return err;
> > }
> >
> > +static void rtl8211x_clear_eee_adv(struct phy_device *phydev)
> > +{
> > + struct rtl8211x_phy_priv *priv = phydev->priv;
> > + u16 val;
> > +
> > + if (priv->eee_1000t_disable || priv->eee_100tx_disable) {
> > + val = phy_read_mmd_indirect(phydev,
> > MDIO_AN_EEE_ADV,
> > + MDIO_MMD_AN);
> > +
> > + if (priv->eee_1000t_disable)
> > + val &= ~MDIO_AN_EEE_ADV_1000T;
> > + if (priv->eee_100tx_disable)
> > + val &= ~MDIO_AN_EEE_ADV_100TX;
> > +
> > + phy_write_mmd_indirect(phydev, MDIO_AN_EEE_ADV,
> > + MDIO_MMD_AN, val);
> > + }
> > +}
> > +
> > +static int rtl8211x_config_init(struct phy_device *phydev)
> > +{
> > + int ret;
> > +
> > + ret = genphy_config_init(phydev);
> > + if (ret < 0)
> > + return ret;
> > +
> > + rtl8211x_clear_eee_adv(phydev);
> > +
> > + return 0;
> > +}
> > +
> > static int rtl8211f_config_init(struct phy_device *phydev)
> > {
> > int ret;
> > u16 reg;
> >
> > - ret = genphy_config_init(phydev);
> > + ret = rtl8211x_config_init(phydev);
> > if (ret < 0)
> > return ret;
> >
> > @@ -115,6 +153,26 @@ static int rtl8211f_config_init(struct
> > phy_device *phydev)
> > return 0;
> > }
> >
> > +static int rtl8211x_phy_probe(struct phy_device *phydev)
> > +{
> > + struct device *dev = &phydev->mdio.dev;
> > + struct device_node *of_node = dev->of_node;
> > + struct rtl8211x_phy_priv *priv;
> > +
> > + priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
> > + if (!priv)
> > + return -ENOMEM;
> > +
> > + priv->eee_1000t_disable =
> > + of_property_read_bool(of_node, "realtek,disable-
> > eee-1000t");
> > + priv->eee_100tx_disable =
> > + of_property_read_bool(of_node, "realtek,disable-
> > eee-100tx");
> > +
> > + phydev->priv = priv;
> > +
> > + return 0;
> > +}
> > +
> > static struct phy_driver realtek_drvs[] = {
> > {
> > .phy_id = 0x00008201,
> > @@ -140,7 +198,9 @@ static struct phy_driver realtek_drvs[] = {
> > .phy_id_mask = 0x001fffff,
> > .features = PHY_GBIT_FEATURES,
> > .flags = PHY_HAS_INTERRUPT,
> > + .probe = &rtl8211x_phy_probe,
> > .config_aneg = genphy_config_aneg,
> > + .config_init = &rtl8211x_config_init,
> > .read_status = genphy_read_status,
> > .ack_interrupt = rtl821x_ack_interrupt,
> > .config_intr = rtl8211e_config_intr,
> > @@ -152,7 +212,9 @@ static struct phy_driver realtek_drvs[] = {
> > .phy_id_mask = 0x001fffff,
> > .features = PHY_GBIT_FEATURES,
> > .flags = PHY_HAS_INTERRUPT,
> > + .probe = &rtl8211x_phy_probe,
> > .config_aneg = &genphy_config_aneg,
> > + .config_init = &rtl8211x_config_init,
> > .read_status = &genphy_read_status,
> > .ack_interrupt = &rtl821x_ack_interrupt,
> > .config_intr = &rtl8211e_config_intr,
> > @@ -164,6 +226,7 @@ static struct phy_driver realtek_drvs[] = {
> > .phy_id_mask = 0x001fffff,
> > .features = PHY_GBIT_FEATURES,
> > .flags = PHY_HAS_INTERRUPT,
> > + .probe = &rtl8211x_phy_probe,
> > .config_aneg = &genphy_config_aneg,
> > .config_init = &rtl8211f_config_init,
> > .read_status = &genphy_read_status,
> > --
> > 2.7.4
> >
>
> How about adding callback functionality for .soft_reset to handle
> BMCR
> where we update the Auto-Negotiation for the phy,
> as per the datasheet of the rtl8211f.
I'm not sure I understand how this would help with our issue (and EEE).
Am I missing something or is it something unrelated that you would like
to see happening on this driver ?
>
> -Best Regard
> Anand Moon
>
> >
> >
> > _______________________________________________
> > linux-amlogic mailing list
> > linux-amlogic@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-amlogic
next prev parent reply other threads:[~2016-11-17 10:21 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-15 14:29 [PATCH net 0/3] Fix OdroidC2 Gigabit Tx link issue Jerome Brunet
2016-11-15 14:29 ` [PATCH net 1/3] net: phy: realtek: add eee advertisement disable options Jerome Brunet
2016-11-15 16:30 ` Andrew Lunn
2016-11-15 17:03 ` Florian Fainelli
2016-11-16 9:56 ` Jerome Brunet
2016-11-16 13:23 ` Andrew Lunn
2016-11-16 14:51 ` Jerome Brunet
2016-11-16 15:06 ` Andrew Lunn
2016-11-16 15:38 ` Jerome Brunet
2016-11-16 17:01 ` Florian Fainelli
2016-11-16 17:06 ` Anand Moon
2016-11-17 10:20 ` Jerome Brunet [this message]
2016-11-17 18:00 ` Anand Moon
2016-11-17 21:48 ` Jerome Brunet
2016-11-15 14:29 ` [PATCH net 2/3] dt-bindings: net: add DT bindings for realtek phys Jerome Brunet
2016-11-16 15:11 ` Rob Herring
2016-11-16 15:20 ` Jerome Brunet
2016-11-15 14:29 ` [PATCH net 3/3] ARM64: dts: meson: odroidc2: disable 1000t-eee advertisement Jerome Brunet
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1479378055.17538.57.camel@baylibre.com \
--to=jbrunet@baylibre.com \
--cc=alexandre.torgue@st.com \
--cc=carlo@caione.org \
--cc=devicetree@vger.kernel.org \
--cc=f.fainelli@gmail.com \
--cc=khilman@baylibre.com \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux.amoon@gmail.com \
--cc=martin.blumenstingl@googlemail.com \
--cc=narmstrong@baylibre.com \
--cc=neolynx@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=peppe.cavallaro@st.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).