Skip to content

net: phy: Force master mode for RTL8211C on some boards

Hans de Goede u-boot 3 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v5 Latest1 of 3 patches 0 repliesNew
  2. v30 of 4 patches 0 repliesSuperseded
  3. v1 RFC1 patch 0 repliesSuperseded

1 of 3 patches is in this archive. The cover letter announces 3.

Patches in v5 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3net: phy: Optionally force master mode for RTL PHY+0 -01 reply

Reviews trailers per patch

The patch has a Reviewed-by or Acked-by.

PatchHans de Goede
1/3A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v5 thread

Cover letter [U-Boot] [PATCH v5 0/3] net: phy: Force master mode for RTL8211C on some boards

Hi,

Quoted 7 lines · Michael Haas

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.

Quoted 6 lines

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

Quoted 40 lines

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(-)

Read the cover letter in its thread