Thread (9 messages) flat view 9 messages, 3 authors, 2021-10-09

Re: [net PATCH 1/2] drivers: net: phy: at803x: fix resume for QCA8327 phy

From: Ansuel Smith <ansuelsmth@gmail.com>
Date: 2021-10-08 23:50:29
Also in: lkml

On Fri, Oct 08, 2021 at 04:47:50PM -0700, Jakub Kicinski wrote:
On Sat,  9 Oct 2021 01:34:25 +0200 Ansuel Smith wrote:
quoted
From Documentation phy resume triggers phy reset and restart
auto-negotiation. Add a dedicated function to wait reset to finish as
it was notice a regression where port sometime are not reliable after a
suspend/resume session. The reset wait logic is copied from phy_poll_reset.
Add dedicated suspend function to use genphy_suspend only with QCA8337
phy and set only additional debug settings for QCA8327. With more test
it was reported that QCA8327 doesn't proprely support this mode and
using this cause the unreliability of the switch ports, especially the
malfunction of the port0.

Fixes: 15b9df4ece17 ("net: phy: at803x: add resume/suspend function to qca83xx phy")
Hm, there's some confusion here. This commit does not exist in net,
and neither does the one from patch 2.

We should be fine with these going into net-next, right Andrew?
Took the hash from linux-next. Think this is the reason they are not in
net?

--
	Ansuel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help