lkml.org 
[lkml]   [2021]   [Nov]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v2 2/2] net: ethernet: Add driver for Sunplus SP7021
Date
Hi Andrew,

I set phy-id registers to 30 and 31 and found the read-back
values of mdio read commands from CPU are all 0x0000.

I consulted with an ASIC engineer. She confirmed that if
phy-id of a mdio command from CPU does not match any
phy-id registers, the mdio command will not be sent out.

She explained if phy-id of a mdio command does not match
any phy-id registers (represent addresses of external PHYs),
why MAC needs to send a command to non-existing PHY?


Best regards,

Wells Lu
呂芳騰

智能運算專案/Smart Computing Program
家庭平台事業群/Home Entertainment Business Unit
凌陽科技/Sunplus Technology Co., Ltd.
地址:300新竹科學園區創新一路19號
19, Innovation 1st Road,
Science-based Industrial Park
Hsin-Chu, Taiwan 300
TEL:886-3-5786005 ext. 2580

> -----Original Message-----
> From: Andrew Lunn <andrew@lunn.ch>
> Sent: Thursday, November 25, 2021 11:21 PM
> To: Wells Lu 呂芳騰 <wells.lu@sunplus.com>
> Cc: Wells Lu <wellslutw@gmail.com>; davem@davemloft.net; kuba@kernel.org;
> robh+dt@kernel.org; netdev@vger.kernel.org; devicetree@vger.kernel.org;
> linux-kernel@vger.kernel.org; p.zabel@pengutronix.de; Vincent Shih 施錕鴻
> <vincent.shih@sunplus.com>
> Subject: Re: [PATCH v2 2/2] net: ethernet: Add driver for Sunplus SP7021
>
> > Due to hardware design, we still need to set PHY address, because MDIO
> > controller of SP7021 only sends out MDIO commands with the same
> > address listed in PHY address registers. The function below needs to
> > be kept.
>
> I suggest you actually set it to some other address. One of the good/bad things about MDIO
> is you have no idea if the device is there. A read to a device which does not exist just
> returns 0xffff, not an error. So i would set the address of 0x1f. I've never seen a PHY
> actually use that address.
>
> Andrew
\
 
 \ /
  Last update: 2021-11-26 04:59    [W:1.986 / U:0.580 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site