Re: [PATCH v2 net-next 08/14] net: enetc: remove invalid code from enetc4_pl_mac_link_up()

[Maxime Chevallier]

On 7/2/26 04:57, wei.f...@oss.nxp.com wrote:
> From: Wei Fang <wei.f...@nxp.com>
> 
> When adding phylink MAC operations support to the NETC switch driver,
> Russell King pointed out several pieces of invalid logic in the
> .mac_link_up() implementation (see [1] and [2]):
> 
> 1) Half-duplex backpressure is not supported by the kernel, Ethernet
>    relies on packet dropping for congestion management.
> 
> 2) phylink_autoneg_inband() is unnecessary, as RGMII in-band status is
>    not supported.
> 
> 3) TX and RX pause are disabled in half-duplex mode, so there is no
>    need to override them in .mac_link_up().
> 
> The same invalid logic is also present in enetc4_pl_mac_link_up(), so
> remove the invalid code from it.
> 
> Link: https://lore.kernel.org/imx/aceiqqi-_oycy...@shell.armlinux.org.uk/ # 1
> Link: https://lore.kernel.org/imx/acefwqmavwls_...@shell.armlinux.org.uk/ # 2
> Signed-off-by: Wei Fang <wei.f...@nxp.com>

Reviewed-by: Maxime Chevallier <maxime.chevall...@bootlin.com>

Maxime