On Thu, Oct 24, 2024 at 06:36:06AM +0800, Inochi Amaoto wrote: > On Wed, Oct 23, 2024 at 02:42:16PM +0200, Andrew Lunn wrote: > > > Yes, this is what I have done at the beginning. At first I only > > > set up the phy setting and not set the config in the syscon. > > > But I got a weird thing: the phy lookback test is timeout. > > > Although the datasheet told it just adds a internal delay for > > > the phy, I suspect sophgo does something more to set this delay. > > > > You need to understand what is going on here. Just because it works > > does not mean it is correct. > > > > It seems like there is a missing info in the SG2044 doc: setting the > syscon internal delay bit is not enabling the internal mac delay, but > disable it. Now everything seems like normal: the mac adds no delay, > and the phy adds its delay. That makes a lot more sense. Thanks for digging into the details. Andrew
