Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Fri, 22 Feb 2019 22:44:04 +0100
From:   Andrew Lunn <andrew@lunn.ch>
To:     Parshuram Thombare <pthombar@cadence.com>
Cc:     nicolas.ferre@microchip.com, davem@davemloft.net,
        netdev@vger.kernel.org, f.fainelli@gmail.com, hkallweit1@gmail.com,
        linux-kernel@vger.kernel.org, rafalc@cadence.com,
        piotrs@cadence.com, jank@cadence.com
Subject: Re: [PATCH 3/3] net: ethernet: add support for high speed mac and
 usxgmii pcs
Message-ID: <20190222214404.GP5894@lunn.ch>
References: <20190222201309.GA23470@lvlogina.cadence.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20190222201309.GA23470@lvlogina.cadence.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

>  	if (macb_is_gem(bp)) {
> -		linkmode_copy(phydev->supported, PHY_GBIT_FEATURES);
> -		if (bp->caps & MACB_CAPS_TWO_PT_FIVE_GIG_SPEED)
> -			linkmode_set_bit(ETHTOOL_LINK_MODE_2500baseT_Full_BIT,
> -					 phydev->supported);
> +		if (bp->caps & MACB_CAPS_HIGH_SPEED) {
> +			linkmode_copy(phydev->supported, PHY_10GBIT_FEATURES);
> +		} else {
> +			u32 bitmask = ETHTOOL_LINK_MODE_2500baseT_Full_BIT;
> +
> +			linkmode_copy(phydev->supported, PHY_GBIT_FEATURES);
> +			if (bp->caps & MACB_CAPS_TWO_PT_FIVE_GIG_SPEED)
> +				linkmode_set_bit(bitmask, phydev->supported);
> +		}

Same issue again. Somebody could be using a 10G MAC with a 2.5G PHY.

     Andrew