]> git.itanic.dy.fi Git - linux-stable/commit
phy: marvell: phy-mvebu-a3700-comphy: Remove broken reset support
authorPali Rohár <pali@kernel.org>
Mon, 29 Aug 2022 08:30:46 +0000 (10:30 +0200)
committerVinod Koul <vkoul@kernel.org>
Tue, 30 Aug 2022 05:02:33 +0000 (10:32 +0530)
commit0a6fc70d76bddf98278af2ac000379c82aec8f11
treed19ae14aabdba8f6c79408fe6cf2b99e70d2c702
parent568035b01cfb107af8d2e4bd2fb9aea22cf5b868
phy: marvell: phy-mvebu-a3700-comphy: Remove broken reset support

Reset support for SATA PHY is somehow broken and after calling it, kernel
is not able to detect and initialize SATA disk Samsung SSD 850 EMT0 [1].

Reset support was introduced in commit 934337080c6c ("phy: marvell:
phy-mvebu-a3700-comphy: Add native kernel implementation") as part of
complete rewrite of this driver. v1 patch series of that commit [2] did
not contain reset support and was tested that is working fine with
Ethernet, SATA and USB PHYs without issues too.

So for now remove broken reset support and change implementation of
power_off callback to power off all functions on specified lane (and not
only selected function) because during startup kernel does not know which
function was selected and configured by bootloader. Same logic was used
also in v1 patch series of that commit.

This change fixes issues with initialization of SATA disk Samsung SSD 850
and disk is working again, like before mentioned commit.

Once problem with PHY reset callback is solved its functionality could be
re-introduced. But for now it is unknown why it does not work.

[1] - https://lore.kernel.org/r/20220531124159.3e4lgn2v462irbtz@shindev/
[2] - https://lore.kernel.org/r/20211028184242.22105-1-kabel@kernel.org/

Reported-by: Shinichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Fixes: 934337080c6c ("phy: marvell: phy-mvebu-a3700-comphy: Add native kernel implementation")
Cc: stable@vger.kernel.org # v5.18+
Signed-off-by: Pali Rohár <pali@kernel.org>
Tested-by: Shinichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Link: https://lore.kernel.org/r/20220829083046.15082-1-pali@kernel.org
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/marvell/phy-mvebu-a3700-comphy.c