Thread (21 messages) 21 messages, 3 authors, 2024-12-18

Re: [PATCH v2 2/2] arm64: dts: qcom: qcs615-ride: Enable ethernet node

From: Andrew Lunn <andrew@lunn.ch>
Date: 2024-12-18 16:57:54
Also in: linux-arm-msm, linux-devicetree, lkml

Okay, I will follow your instructions:
1. Change the phy-mode to 'rgmii-id'.
2. Add the delay in the MAC driver.
3. Mask the phy-mode before passing it to the PHY.
Good.
quoted
But i assume Qualcomm RDKs always make use of a Qualcomm PHY, there is
special pricing if you use the combination, so there is probably
little incentive to use somebody elses PHY. And i assume you can
quickly check all Qualcomm PHYs support RGMII delays. PHYs which don't
support RGMII delays are very rare, it just happened that one vendors
RDK happened to use one, so they ended up with delays in the MAC being
standard for their boards.
Most Qualcomm MAC applications are actually paired with a third-party PHY.
I'm not sure the QCA8K, IPC and old Atheros people would agree with
you.

But since you don't have any behaviour change, you are not asking the
PHY to insert the delays, you should be safe in your part of the
Qualcomm world.

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