[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <e54083ee-ed3a-115f-0856-d943dac579a8@huawei.com>
Date: Thu, 14 May 2020 09:32:51 +0800
From: tanhuazhong <tanhuazhong@...wei.com>
To: Andrew Lunn <andrew@...n.ch>
CC: <f.fainelli@...il.com>, <hkallweit1@...il.com>,
<linux@...linux.org.uk>, <davem@...emloft.net>,
<netdev@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<linuxarm@...wei.com>, <kuba@...nel.org>,
Yufeng Mo <moyufeng@...wei.com>,
Jian Shen <shenjian15@...wei.com>
Subject: Re: [PATCH net-next] net: phy: realtek: add loopback support for
RTL8211F
On 2020/5/13 21:12, Andrew Lunn wrote:
> On Wed, May 13, 2020 at 04:25:44PM +0800, Huazhong Tan wrote:
>> From: Yufeng Mo <moyufeng@...wei.com>
>>
>> PHY loopback is already supported by genphy driver. This patch
>> adds the set_loopback interface to RTL8211F PHY driver, so the PHY
>> selftest can run properly on it.
>>
>> Signed-off-by: Yufeng Mo <moyufeng@...wei.com>
>> Signed-off-by: Jian Shen <shenjian15@...wei.com>
>> Signed-off-by: Huazhong Tan <tanhuazhong@...wei.com>
>
> It took three people to write a 1 line patch?
Yufeng Mo is the author of this patch, since he has not right to send mail
so I just help him to do than。 If necessary, Jian Shen could be deleted.
>
>> ---
>> drivers/net/phy/realtek.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/net/phy/realtek.c b/drivers/net/phy/realtek.c
>> index c7229d0..6c5918c 100644
>> --- a/drivers/net/phy/realtek.c
>> +++ b/drivers/net/phy/realtek.c
>> @@ -615,6 +615,7 @@ static struct phy_driver realtek_drvs[] = {
>> .resume = genphy_resume,
>> .read_page = rtl821x_read_page,
>> .write_page = rtl821x_write_page,
>> + .set_loopback = genphy_loopback,
>> }, {
>> .name = "Generic FE-GE Realtek PHY",
>> .match_phy_device = rtlgen_match_phy_device,
>
> Do you have access to the data sheets? Can you check if the other PHYs
> supported by this driver also support loopback in the standard way?
> They probably do.
>
> Andrew
>
We have checked the data sheet. Since we only have rtl8211f phy on our
hand, so only support loopback on this phy.
Thanks:)
> .
>
Powered by blists - more mailing lists