All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/3] phy: rockchip: emmc: Enable internal strobe pull-down by default
@ 2024-03-26 18:54 ` Folker Schwesinger via B4 Relay
  0 siblings, 0 replies; 74+ messages in thread
From: Folker Schwesinger @ 2024-03-26 18:54 UTC (permalink / raw)
  To: Vinod Koul, Kishon Vijay Abraham I, Heiko Stuebner, Chris Ruehl,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley
  Cc: Christopher Obbard, Alban Browaeys, Doug Anderson, Brian Norris,
	Jensen Huang, linux-phy, linux-arm-kernel, linux-rockchip,
	linux-kernel, devicetree, Folker Schwesinger

Various RK3399 boards, including the Rock Pi 4 series, the NanoPi 4
series and possibly others showed instabilities with some eMMC modules
when operating in HS400 mode [1]. Later, a missing pull-down resistor on
the eMMC strobe line was identified as the root cause of these
instabilities [2].

This series changes the Rockchip eMMC PHY driver to enable the
internal pull-down for the strobe line by default, effectively
restoring the behavior of the vendor kernel [3].
Addressing the problem in the driver avoids fix-ups in individual
board DTs.

As this change makes the optional DT property enable-strobe-pulldown
obsolete, it is replaced by a new property disable-strobe-pulldown,
that, when used, allows to disable the internal pull-down.

While at it, the enable-strobe-pulldown property was removed from the
NanoPi4 series DTS (which was the only user of the property in the kernel).

[1]: https://lore.kernel.org/all/20230705144255.115299-1-chris.obbard@collabora.com/
[2]: https://lore.kernel.org/all/20220822074139.3810-1-jensenhuang@friendlyarm.com/
[3]: https://github.com/rockchip-linux/kernel/blob/develop-4.4/drivers/phy/rockchip/phy-rockchip-emmc.c#L310

Signed-off-by: Folker Schwesinger <dev@folker-schwesinger.de>
---
Folker Schwesinger (3):
      phy: rockchip: emmc: Enable pulldown for strobe line
      devicetree: phy: rockchip-emmc: Document changed strobe-pulldown property
      arm64: dts: rockchip: Remove enable-strobe-pulldown for NanoPi4 boards

 Documentation/devicetree/bindings/phy/rockchip-emmc-phy.txt | 4 ++--
 arch/arm64/boot/dts/rockchip/rk3399-nanopi4.dtsi            | 1 -
 drivers/phy/rockchip/phy-rockchip-emmc.c                    | 6 +++---
 3 files changed, 5 insertions(+), 6 deletions(-)
---
base-commit: 4cece764965020c22cff7665b18a012006359095
change-id: 20240326-rk-default-enable-strobe-pulldown-3356c8cf1b4f

Best regards,
-- 
Folker Schwesinger <dev@folker-schwesinger.de>


^ permalink raw reply	[flat|nested] 74+ messages in thread

end of thread, other threads:[~2024-06-14 18:34 UTC | newest]

Thread overview: 74+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-26 18:54 [PATCH 0/3] phy: rockchip: emmc: Enable internal strobe pull-down by default Folker Schwesinger
2024-03-26 18:54 ` Folker Schwesinger via B4 Relay
2024-03-26 18:54 ` Folker Schwesinger via B4 Relay
2024-03-26 18:54 ` Folker Schwesinger via B4 Relay
2024-03-26 18:54 ` Folker Schwesinger via B4 Relay
2024-03-26 18:54 ` [PATCH 1/3] phy: rockchip: emmc: Enable pulldown for strobe line Folker Schwesinger
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 19:46   ` Conor Dooley
2024-03-26 19:46     ` Conor Dooley
2024-03-26 19:46     ` Conor Dooley
2024-03-26 19:46     ` Conor Dooley
2024-03-26 19:55     ` Dragan Simic
2024-03-26 19:55       ` Dragan Simic
2024-03-26 19:55       ` Dragan Simic
2024-03-26 19:55       ` Dragan Simic
2024-03-27 16:21       ` Folker Schwesinger
2024-03-27 16:21         ` Folker Schwesinger
2024-03-27 16:21         ` Folker Schwesinger
2024-03-27 16:21         ` Folker Schwesinger
2024-03-27 17:21         ` Conor Dooley
2024-03-27 17:21           ` Conor Dooley
2024-03-27 17:21           ` Conor Dooley
2024-03-27 17:21           ` Conor Dooley
2024-03-28 17:00     ` Alban Browaeys
2024-03-28 17:00       ` Alban Browaeys
2024-03-28 17:00       ` Alban Browaeys
2024-03-28 17:00       ` Alban Browaeys
2024-03-28 18:01       ` Conor Dooley
2024-03-28 18:01         ` Conor Dooley
2024-03-28 18:01         ` Conor Dooley
2024-03-28 18:01         ` Conor Dooley
2024-04-10 18:28         ` Alban Browaeys
2024-04-10 18:28           ` Alban Browaeys
2024-04-10 18:28           ` Alban Browaeys
2024-04-10 18:28           ` Alban Browaeys
2024-04-11 15:42           ` Conor Dooley
2024-04-11 15:42             ` Conor Dooley
2024-04-11 15:42             ` Conor Dooley
2024-04-11 15:42             ` Conor Dooley
2024-04-19 14:31             ` Heiko Stübner
2024-04-19 14:31               ` Heiko Stübner
2024-04-19 14:31               ` Heiko Stübner
2024-04-19 14:31               ` Heiko Stübner
2024-06-11 19:38               ` Alban Browaeys
2024-06-11 19:38                 ` Alban Browaeys
2024-06-11 19:38                 ` Alban Browaeys
2024-06-14 18:34                 ` Folker Schwesinger
2024-06-14 18:34                   ` Folker Schwesinger
2024-06-14 18:34                   ` Folker Schwesinger
2024-03-31 19:26       ` Dragan Simic
2024-03-31 19:26         ` Dragan Simic
2024-03-31 19:26         ` Dragan Simic
2024-03-31 19:26         ` Dragan Simic
2024-03-26 18:54 ` [PATCH 2/3] devicetree: phy: rockchip-emmc: Document changed strobe-pulldown property Folker Schwesinger
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 19:34   ` Conor Dooley
2024-03-26 19:34     ` Conor Dooley
2024-03-26 19:34     ` Conor Dooley
2024-03-26 19:34     ` Conor Dooley
2024-03-28  9:37   ` Krzysztof Kozlowski
2024-03-28  9:37     ` Krzysztof Kozlowski
2024-03-28  9:37     ` Krzysztof Kozlowski
2024-03-28  9:37     ` Krzysztof Kozlowski
2024-03-26 18:54 ` [PATCH 3/3] arm64: dts: rockchip: Remove enable-strobe-pulldown for NanoPi4 boards Folker Schwesinger
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay
2024-03-26 18:54   ` Folker Schwesinger via B4 Relay

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.