netdev - RE: [EXT] [PATCH RFC net-next 7/9] net: prestera: switch PCS driver to use phylink_pcs_neg

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: 
 <BN9PR18MB4251A2FD53EB8F560AE642A4DB419@BN9PR18MB4251.namprd18.prod.outlook.com>
Date: Wed, 24 May 2023 11:38:16 +0000
From: Elad Nachman <enachman@...vell.com>
To: "Russell King (Oracle)" <rmk+kernel@...linux.org.uk>,
        Andrew Lunn
	<andrew@...n.ch>, Heiner Kallweit <hkallweit1@...il.com>
CC: Daniel Machon <daniel.machon@...rochip.com>,
        "David S. Miller"
	<davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>,
        Florian Fainelli
	<f.fainelli@...il.com>,
        Horatiu Vultur <horatiu.vultur@...rochip.com>,
        Ioana
 Ciornei <ioana.ciornei@....com>,
        Jakub Kicinski <kuba@...nel.org>,
        Lars
 Povlsen <lars.povlsen@...rochip.com>,
        "linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>,
        Madalin Bucur
	<madalin.bucur@....com>,
        Marcin Wojtas <mw@...ihalf.com>, Michal Simek
	<michal.simek@....com>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        Paolo Abeni <pabeni@...hat.com>,
        Radhey Shyam Pandey
	<radhey.shyam.pandey@...inx.com>,
        Sean Anderson <sean.anderson@...o.com>,
        Steen Hegelund <Steen.Hegelund@...rochip.com>,
        Taras Chornyi
	<taras.chornyi@...ision.eu>,
        Thomas Petazzoni <thomas.petazzoni@...tlin.com>,
        "UNGLinuxDriver@...rochip.com" <UNGLinuxDriver@...rochip.com>,
        Vladimir
 Oltean <olteanv@...il.com>
Subject: RE: [EXT] [PATCH RFC net-next 7/9] net: prestera: switch PCS driver
 to use phylink_pcs_neg_mode()



> -----Original Message-----
> From: Russell King <rmk@...linux.org.uk> On Behalf Of Russell King (Oracle)
> Sent: Tuesday, May 23, 2023 6:56 PM
> To: Andrew Lunn <andrew@...n.ch>; Heiner Kallweit
> <hkallweit1@...il.com>
> Cc: Daniel Machon <daniel.machon@...rochip.com>; David S. Miller
> <davem@...emloft.net>; Eric Dumazet <edumazet@...gle.com>; Florian
> Fainelli <f.fainelli@...il.com>; Horatiu Vultur
> <horatiu.vultur@...rochip.com>; Ioana Ciornei <ioana.ciornei@....com>;
> Jakub Kicinski <kuba@...nel.org>; Lars Povlsen
> <lars.povlsen@...rochip.com>; linux-arm-kernel@...ts.infradead.org;
> Madalin Bucur <madalin.bucur@....com>; Marcin Wojtas
> <mw@...ihalf.com>; Michal Simek <michal.simek@....com>;
> netdev@...r.kernel.org; Paolo Abeni <pabeni@...hat.com>; Radhey Shyam
> Pandey <radhey.shyam.pandey@...inx.com>; Sean Anderson
> <sean.anderson@...o.com>; Steen Hegelund
> <Steen.Hegelund@...rochip.com>; Taras Chornyi
> <taras.chornyi@...ision.eu>; Thomas Petazzoni
> <thomas.petazzoni@...tlin.com>; UNGLinuxDriver@...rochip.com;
> Vladimir Oltean <olteanv@...il.com>
> Subject: [EXT] [PATCH RFC net-next 7/9] net: prestera: switch PCS driver to
> use phylink_pcs_neg_mode()
> 
> External Email
> 
> ----------------------------------------------------------------------
> Use the newly introduced phylink_pcs_neg_mode() to configure whether
> inband-AN should be used.
> 
> Signed-off-by: Russell King (Oracle) <rmk+kernel@...linux.org.uk>
> ---
>  drivers/net/ethernet/marvell/prestera/prestera_main.c | 10 ++++------
>  1 file changed, 4 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/net/ethernet/marvell/prestera/prestera_main.c
> b/drivers/net/ethernet/marvell/prestera/prestera_main.c
> index 9d504142e51a..2a26f96fbed2 100644
> --- a/drivers/net/ethernet/marvell/prestera/prestera_main.c
> +++ b/drivers/net/ethernet/marvell/prestera/prestera_main.c
> @@ -308,38 +308,36 @@ static int prestera_pcs_config(struct phylink_pcs
> *pcs,  {
>  	struct prestera_port *port = prestera_pcs_to_port(pcs);
>  	struct prestera_port_mac_config cfg_mac;
> +	unsigned int neg_mode;
>  	int err;
> 
> +	neg_mode = phylink_pcs_neg_mode(mode, interface, advertising);
> +
>  	err = prestera_port_cfg_mac_read(port, &cfg_mac);
>  	if (err)
>  		return err;
> 
>  	cfg_mac.admin = true;
>  	cfg_mac.fec = PRESTERA_PORT_FEC_OFF;
> +	cfg_mac.inband = neg_mode ==
> PHYLINK_PCS_NEG_INBAND_ENABLED;
> 
>  	switch (interface) {
>  	case PHY_INTERFACE_MODE_10GBASER:
>  		cfg_mac.speed = SPEED_10000;
> -		cfg_mac.inband = 0;
>  		cfg_mac.mode = PRESTERA_MAC_MODE_SR_LR;
>  		break;
>  	case PHY_INTERFACE_MODE_2500BASEX:
>  		cfg_mac.speed = SPEED_2500;
>  		cfg_mac.duplex = DUPLEX_FULL;
> -		cfg_mac.inband =
> test_bit(ETHTOOL_LINK_MODE_Autoneg_BIT,
> -					  advertising);
>  		cfg_mac.mode = PRESTERA_MAC_MODE_SGMII;
>  		break;
>  	case PHY_INTERFACE_MODE_SGMII:
> -		cfg_mac.inband = 1;
>  		cfg_mac.mode = PRESTERA_MAC_MODE_SGMII;
>  		break;
>  	case PHY_INTERFACE_MODE_1000BASEX:
>  	default:
>  		cfg_mac.speed = SPEED_1000;
>  		cfg_mac.duplex = DUPLEX_FULL;
> -		cfg_mac.inband =
> test_bit(ETHTOOL_LINK_MODE_Autoneg_BIT,
> -					  advertising);
>  		cfg_mac.mode = PRESTERA_MAC_MODE_1000BASE_X;
>  		break;
>  	}
> --
> 2.30.2
> 

Acked-by: Elad Nachman <enachman@...vell.com>