<div data-ntes="ntes_mail_body_root" style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div id="spnEditorContent"><div style="margin: 0;">Hi, </div><div style="margin: 0;"><br></div><div style="margin: 0;">thx, </div><div style="margin: 0;"><br></div><div style="margin: 0;"><span style="color: rgb(17, 17, 51); font-family: -apple-system, blinkmacsystemfont, "Helvetica Neue", helvetica, "segoe ui", arial, roboto, "PingFang SC", miui, "Hiragino Sans GB", "Microsoft Yahei", sans-serif, "Source Han Serif"; font-size: 16px; letter-spacing: 0.064px; white-space-collapse: preserve;">i will modify this patch.</span></div><div style="margin: 0;"><br></div><div style="margin: 0;"><br></div></div><div style="position:relative;zoom:1"></div><div id="divNeteaseMailCard"></div><div style="margin: 0;"><br></div><pre><br>At 2025-10-15 16:59:18, "Simon Horman" <horms@kernel.org> wrote:
>On Tue, Oct 14, 2025 at 09:08:47PM -0700, Lizhe wrote:
>> 'phy_power_on' is a local scope one within the driver, since the return
>> value of the phy_power_on() function is always 0, checking its return
>> value is redundant.
>> 
>> the function name 'phy_power_on()' conflicts with the existing
>> phy_power_on() function in the PHY subsystem. a suitable alternative
>> name would be rk_phy_power_set(), particularly since when the second
>> argument is false, this function actually powers off the PHY
>> 
>> Signed-off-by: Lizhe <sensor1010@163.com>
>> ---
>>  drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c | 17 ++++-------------
>>  1 file changed, 4 insertions(+), 13 deletions(-)
>> 
>> diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c
>> index 51ea0caf16c1..9d296bfab013 100644
>> --- a/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c
>> +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c
>> @@ -1461,23 +1461,18 @@ static int gmac_clk_enable(struct rk_priv_data *bsp_priv, bool enable)
>>        return 0;
>>  }
>>  
>> -static int phy_power_on(struct rk_priv_data *bsp_priv, bool enable)
>> +static void rk_phy_power_on(struct rk_priv_data *bsp_priv, bool enable)
>
>Hi Lizhe,
>
>This introduces a compilation error because phy_power_on()
>is still used on line 1670.
>
>Perhaps the hunk to update that line got lost somewhere.
>
>-- 
>pw-bot: changes-requested
>
>...
</pre></div>