[Intel-wired-lan] [PATCH net v1 2/2] igc: Fix Supported Pause Frame Link Setting

[Dvora Fuxbrumer <dvorax.fuxbrumer@linux.intel.com>]

On 19/02/2021 18:36, Muhammad Husaini Zulkifli wrote:
> The Supported Pause Frame always display "No" eventhough the Advertised
> pause frame showing the correct setting based on the pause parameters via
> ethtool. Set bit in link_ksettings to "Supported" for Pause Frame.
> 
> Before output:
> Supported pause frame use: No
> 
> Expected output:
> Supported pause frame use: Symmetric
> 
> Fixes: 8c5ad0dae93c ("igc: Add ethtool support")
> Signed-off-by: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com>
> Reviewed-by: Malli C <mallikarjuna.chilakala@intel.com>
> ---
>   drivers/net/ethernet/intel/igc/igc_ethtool.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/net/ethernet/intel/igc/igc_ethtool.c b/drivers/net/ethernet/intel/igc/igc_ethtool.c
> index 67a4aed45fc2..8722294ab90c 100644
> --- a/drivers/net/ethernet/intel/igc/igc_ethtool.c
> +++ b/drivers/net/ethernet/intel/igc/igc_ethtool.c
> @@ -1711,6 +1711,9 @@ static int igc_ethtool_get_link_ksettings(struct net_device *netdev,
>   						     Autoneg);
>   	}
>   
> +	/* Set pause flow control settings */
> +	ethtool_link_ksettings_add_link_mode(cmd, supported, Pause);
> +
>   	switch (hw->fc.requested_mode) {
>   	case igc_fc_full:
>   		ethtool_link_ksettings_add_link_mode(cmd, advertising, Pause);
> 
Tested-by: Dvora Fuxbrumer <dvorax.fuxbrumer@linux.intel.com>