From mboxrd@z Thu Jan  1 00:00:00 1970
From: Kostya Porotchkin <kostap@marvell.com>
Date: Tue, 27 Apr 2021 13:48:23 +0000
Subject: [EXT] [PATCH v1 05/10] net: mvpp2: Fix 2.5G GMII_SPEED
 configurations
In-Reply-To: <20210427152713.v1.5.Iffb0ea1939f77c16abfcdf4a426fa87b06db87da@changeid>
References: <20210427132718.645043-1-sr@denx.de>
 <20210427152713.v1.5.Iffb0ea1939f77c16abfcdf4a426fa87b06db87da@changeid>
Message-ID: <CO6PR18MB441712B4E89400AA64151EE2CA419@CO6PR18MB4417.namprd18.prod.outlook.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi, Stefan,

> -----Original Message-----
> From: Stefan Roese <sr@denx.de>
> Sent: Tuesday, April 27, 2021 16:27
> To: u-boot at lists.denx.de
> Cc: Stefan Chulski <stefanc@marvell.com>; Marcin Wojtas
> <mw@semihalf.com>; Nadav Haklai <nadavh@marvell.com>; Marek Behun
> <marek.behun@nic.cz>; Joe Hershberger <joe.hershberger@ni.com>; Kostya
> Porotchkin <kostap@marvell.com>; Yan Markman
> <ymarkman@marvell.com>; sa_ip-sw-jenkins <sa_ip-sw-
> jenkins at marvell.com>
> Subject: [EXT] [PATCH v1 05/10] net: mvpp2: Fix 2.5G GMII_SPEED
> configurations
> 
> External Email
> 
> ----------------------------------------------------------------------
> From: Stefan Chulski <stefanc@marvell.com>
> 
> GMII_SPEED should be enabled for 2.5G speed
> 
> Signed-off-by: Stefan Chulski <stefanc@marvell.com>
> Reviewed-by: Yan Markman <ymarkman@marvell.com>
> Reviewed-by: Kostya Porotchkin <kostap@marvell.com>
> Tested-by: sa_ip-sw-jenkins <sa_ip-sw-jenkins@marvell.com>
> Signed-off-by: Stefan Roese <sr@denx.de>
> ---
> 
>  drivers/net/mvpp2.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/net/mvpp2.c b/drivers/net/mvpp2.c index
> 2043bdf10aa4..ec7cb89a94c8 100644
> --- a/drivers/net/mvpp2.c
> +++ b/drivers/net/mvpp2.c
> @@ -4445,7 +4445,8 @@ static void mvpp2_link_event(struct mvpp2_port
> *port)
>  			if (phydev->duplex)
>  				val |= MVPP2_GMAC_CONFIG_FULL_DUPLEX;
> 
> -			if (phydev->speed == SPEED_1000)
> +			if (phydev->speed == SPEED_1000 ||
> +			    phydev->speed == 2500)
[KP] Shouldn't it be "SPEED_2500"?

>  				val |= MVPP2_GMAC_CONFIG_GMII_SPEED;
>  			else if (phydev->speed == SPEED_100)
>  				val |= MVPP2_GMAC_CONFIG_MII_SPEED;
> --
> 2.31.1