> -----Original Message-----
> From: Russell King - ARM Linux admin <[email protected]>
> Sent: Wednesday, January 27, 2021 4:06 PM
> To: Stefan Chulski <[email protected]>
> Cc: [email protected]; [email protected];
> [email protected]; Nadav Haklai <[email protected]>; Yan
> Markman <[email protected]>; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]
> Subject: [EXT] Re: [PATCH v4 net-next 19/19] net: mvpp2: add TX FC
> firmware check
> 
> External Email
> 
> ----------------------------------------------------------------------
> On Wed, Jan 27, 2021 at 01:43:35PM +0200, [email protected] wrote:
> >     if (priv->global_tx_fc && priv->hw_version != MVPP21) {
> > -           val = mvpp2_cm3_read(priv, MSS_FC_COM_REG);
> > -           val |= FLOW_CONTROL_ENABLE_BIT;
> > -           mvpp2_cm3_write(priv, MSS_FC_COM_REG, val);
> > +           err = mvpp2_enable_global_fc(priv);
> > +           if (err) {
> > +                   dev_warn(&pdev->dev, "CM3 firmware not running,
> version should be higher than 18.09\n");
> > +                   dev_warn(&pdev->dev, "Flow control not
> supported\n");
> > +           }
> 
> I've just booted this on my mcbin-ss, and I get:
> 
> mvpp2 f2000000.ethernet: CM3 firmware not running, version should be
> higher than 18.09
> mvpp2 f4000000.ethernet: CM3 firmware not running, version should be
> higher than 18.09
> 
> which is rather odd, because I believe I'm running the 18.12 firmware from
> git://github.com/MarvellEmbeddedProcessors/binaries-marvell
> branch binaries-marvell-armada-18.12.
> 
> Any ideas?

Your mcbin-ss is A8K AX or A8K B0? On AX revisions we do not have FC support in 
firmware.

Regards.


Reply via email to