RE: [PATCH net-next v8 3/7] r8169: add support for phylink

[Javen]

>
>> +static void rtl_mac_link_up(struct phylink_config *config, struct phy_device
>*phydev,
>> +                         unsigned int mode, phy_interface_t interface,
>> +                         int speed, int duplex, bool tx_pause, bool
>> +rx_pause) {
>> +     struct rtl8169_private *tp = container_of(config, struct rtl8169_private,
>phylink_config);
>> +     struct device *d = tp_to_dev(tp);
>> +
>> +     tp->speed = speed;
>> +     rtl_link_chg_patch(tp, speed);
>> +
>> +     pm_request_resume(d);
>
>This does nothing with rx_pause and tx_pause. How is the MAC getting
>configured for flow control?

To actually generate pause frames, the MAC needs Rx FIFO "Near Full" and "Near Empty" thresholds. The driver configures these statically during hardware initialization (rtl_hw_start_xxx) by writing to ERI registers 0xCC (RFIFONFULL_TH) and 0xD0 (RFIFOEMPTY_TH) via rtl8168g_set_pause_thresholds().

Software enables/disables flow control by setting the Local PHY registers phy_set_asym_pause(). After auto-negotiation with the Link Partner, the PHY resolves the flow control capabilities. Once the link is up and flow control is negotiated, the integrated Realtek MAC hardware automatically reads the PHY result.

Therefore, we don't need to do anything with tx_pause and rx_pause in mac_link_up.

Thanks,
Javen
>
>    Andrew
>
>---
>pw-bot: cr