public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v5 0/3] net: phy: Force master mode for RTL8211C on some boards
Date: Thu, 31 Mar 2016 14:31:22 +0200	[thread overview]
Message-ID: <56FD189A.7010300@redhat.com> (raw)
In-Reply-To: <1458926572-21236-1-git-send-email-haas@computerlinguist.org>

Hi,

On 25-03-16 18:22, Michael Haas wrote:
> This patch is required to get reliable 1000BASE-T operation on some
> boards using the RTL8211C(L) PHY.
>
> Following discussions on v2 of this patch, I have removed the incorrect
> check for the RTL8211C(L). Affected boards now have to define
> CONFIG_RTL8211X_PHY_FORCE_MASTER to benefit from the fix.
>
> Note that this patch requires Karsten Merkers '[PATCH] net: phy: Realtek
> RTL8211B/C PHY ID fix' as well as Hans de Goede's recent u-boot-sunxi
> pull request, specifically 1eae8f66ff749409eb96e2f3f3387c56232d0b8a and
> fc8991c61c393ce6a9d3dfc97cb56dbbd9e8cbba.
>
> Michael

Thanks, I've applied this series to my tree, and it will be part of
the next u-boot-sunxi pull-req.

Regards,

Hans


>
>
> Changes in v5:
>   - Improve formatting of Kconfig help text. No content change. Change
>     suggested by Karsten Merker.
>   - Fix order of defconfig entry (suggested by Karsten Marker)
> Series-changes: 4
>   - Changed commit summary according to Chen-Yu Tsai's suggestion,
>     modified to fit the 70 character limit
>   - Fix order of defconfig entry (suggested by Karsten Marker)
>
> Changes in v4:
>   - Squashed previously separate commit introducing KCONFIG variable
>     into commit containing main code change (Hans de Goede's suggestion)
>   - Changed KCONFIG description according to Karsten Merker's suggestions
>     plus some clarification of my own
>   - Changed commit message according to Karsten Merker's suggestions
>   - Changed commit summary according to Chen-Yu Tsai's suggestion,
>     modified to fit the 70 character limit
>
> Changes in v3:
>   - Remove incorrect detection of RTL8211CL and use #ifdef instead
>     (thanks to Karsten Merker)
>   - Introduced constants for register bits
>
> Changes in v2:
>   - Removed accidental inclusion of Karsten's patch in my first submission of this series.
>   - Fix a typo in the code: 6 -> &
>
> Michael Haas (3):
>    net: phy: Optionally force master mode for RTL PHY
>    sunxi: A20-Olimex-SOM-EVB: Force 8211CL to master
>    sunxi: A20-OLinuXino-Lime2: Force 8211CL to master
>
>   configs/A20-OLinuXino-Lime2_defconfig |  1 +
>   configs/A20-Olimex-SOM-EVB_defconfig  |  1 +
>   drivers/net/Kconfig                   | 21 +++++++++++++++++++++
>   drivers/net/phy/realtek.c             | 13 ++++++++++++-
>   4 files changed, 35 insertions(+), 1 deletion(-)
>

      parent reply	other threads:[~2016-03-31 12:31 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-25 17:22 [U-Boot] [PATCH v5 0/3] net: phy: Force master mode for RTL8211C on some boards Michael Haas
2016-03-25 17:22 ` [U-Boot] [PATCH v5 1/3] net: phy: Optionally force master mode for RTL PHY Michael Haas
2016-03-25 21:11   ` Joe Hershberger
2016-03-28 12:54     ` Hans de Goede
2016-03-28 16:12       ` Joe Hershberger
2016-03-25 17:22 ` [U-Boot] [PATCH v5 2/3] sunxi: A20-Olimex-SOM-EVB: Force 8211CL to master Michael Haas
2016-03-25 17:22 ` [U-Boot] [PATCH v5 3/3] sunxi: A20-OLinuXino-Lime2: " Michael Haas
2016-03-31 12:31 ` Hans de Goede [this message]

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=56FD189A.7010300@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=u-boot@lists.denx.de \
    /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