Re: [PATCH] arm64: dts: ti: k3-am62d2-evm: Fix missing RX delay for DP83867 PHY

[Siddharth Vadapalli <s-vadapalli@ti.com>]

On Tue, 2026-01-20 at 18:33 +0530, Hari Prasath wrote:
> On 20/01/26 11:43 am, Siddharth Vadapalli wrote:
> > MAC Ports 1 and 2 of the CPSW3G Ethernet Switch in the AM62D2 SoC are 
> > both connected to different instances of the DP83867 Ethernet PHY on the 
> > AM62D2 EVM, with the 'phy-mode' set to 'rgmii-id'. The DP83867 Ethernet 
> > PHY has to add a 2 nanosecond
> > ZjQcmQRYFpfptBannerStart
> > This message was sent from outside of Texas Instruments.
> > Do not click links or open attachments unless you recognize the source 
> > of this email and know the content is safe.
> > Report Suspicious
> > <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK! 
> > uRdqXRfPtm07agZk_PPjvDYD9oe_mpoIkkjINUyRhGSu--0mQdy1pi4MPng- 
> > ix3RK2L1V3y4DrnTdaCm8zj7QGx0QX2i$>
> > ZjQcmQRYFpfptBannerEnd
> > 
> > MAC Ports 1 and 2 of the CPSW3G Ethernet Switch in the AM62D2 SoC are both
> > connected to different instances of the DP83867 Ethernet PHY on the AM62D2
> > EVM, with the 'phy-mode' set to 'rgmii-id'. The DP83867 Ethernet PHY has to
> > add a 2 nanosecond delay on receive (from wire) based on the EVM design.
> > 
> > Since the device driver for the DP83867 Ethernet PHY coincidentally assumes
> > that the a 2 nanosecond receive delay has to be added in the absence of the
> 
> 'the' can be removed here.Verified this claim on default value in the 
> driver without this DT property.

Thank you for pointing this out. While rephrasing the sentence, I forgot to
remove 'the'.

> 
> Reviewed-by: Hari Prasath Gujulan Elango <gehariprasath@ti.com>

Thank you for reviewing the patch. I will fix the commit message and
include your tag in the v2 patch.

Regards,
Siddharth.