All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Marek Behún" <kabel@kernel.org>
To: Vladimir Oltean <olteanv@gmail.com>
Cc: "Russell King (Oracle)" <linux@armlinux.org.uk>,
	Andrew Lunn <andrew@lunn.ch>,
	Heiner Kallweit <hkallweit1@gmail.com>,
	Alexandre Belloni <alexandre.belloni@bootlin.com>,
	Alvin __ipraga <alsi@bang-olufsen.dk>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	Claudiu Manoil <claudiu.manoil@nxp.com>,
	Daniel Scally <djrscally@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	DENG Qingfang <dqfext@gmail.com>,
	Eric Dumazet <edumazet@google.com>,
	Florian Fainelli <f.fainelli@gmail.com>,
	George McCollister <george.mccollister@gmail.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Hauke Mehrtens <hauke@hauke-m.de>,
	Heikki Krogerus <heikki.krogerus@linux.intel.com>,
	Jakub Kicinski <kuba@kernel.org>,
	Kurt Kanzenbach <kurt@linutronix.de>,
	Landen Chao <Landen.Chao@mediatek.com>,
	Linus Walleij <linus.walleij@linaro.org>,
	linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org,
	Matthias Brugger <matthias.bgg@gmail.com>,
	netdev@vger.kernel.org, Paolo Abeni <pabeni@redhat.com>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Sakari Ailus <sakari.ailus@linux.intel.com>,
	Sean Wang <sean.wang@mediatek.com>,
	UNGLinuxDriver@microchip.com,
	Vivien Didelot <vivien.didelot@gmail.com>,
	Woojung Huh <woojung.huh@microchip.com>
Subject: Re: [PATCH net-next 3/6] net: dsa: add support for retrieving the interface mode
Date: Fri, 22 Jul 2022 14:59:36 +0200	[thread overview]
Message-ID: <20220722145936.497ac73f@dellmb> (raw)
In-Reply-To: <20220721182216.z4vdaj4zfb6w3emo@skbuf>

On Thu, 21 Jul 2022 21:22:16 +0300
Vladimir Oltean <olteanv@gmail.com> wrote:

> On Thu, Jul 21, 2022 at 07:21:45PM +0200, Marek Behún wrote:
> > Marvell documentation says that 2500base-x does not implement inband
> > AN.  
> 
> Does Marvell documentation actually call it 2500base-x when it says it
> doesn't support in-band autoneg?

Yes, it does.

> > But when it was first implemented, for some reason it was thought that
> > 2500base-x is just 1000base-x at 2.5x speed, and 1000base-x does
> > support inband AN. Also it worked during tests for both switches and
> > SOC NICs, so it was enabled.
> > 
> > At the time 2500base-x was not standardized. Now 2500base-x is
> > stanradrized, and the standard says that 2500base-x does not support
> > clause 37 AN. I guess this is because where it is used, it is intended
> > to work with clause 73 AN somehow.  
> 
> When you say 2500base-x is standardized, do you mean there is a document
> somewhere which I could use to read more about this?

IEEE Std 802.3cb-2018: Amendment 1: Physical Layer Specifications and
Management Parameters for 2.5 Gb/s and 5 Gb/s Operation over Backplane.

Annex 127A (informative): Compatibility of 2.5GBASE-X PCS/PMA with
1000BASE-X PCS/PMA running 2.5 times faster

  ...
  This annex discusses the restrictions when operating 2.5GBASE-X
  PCS/PMA with a 1000BASE-X PCS/PMA link partner running 2.5 times
  faster. Compatibility of the PMD is outside the scope of this annex.
  In this annex when 1000BASE-X PCS/PMA is referred to, the 2.5 times
  speed up is implied.
  ...
  The 2.5GBASE-X PCS does not support Clause 37 Auto-Negotiation.
  Hence, the 1000BASE-X PCS is expected to have its Clause 37
  Auto-Negotiation functionality disabled so that the /C/ ordered set
  will not be transmitted. If a 2.5GBASE-X PCS receives /C/ ordered
  set, then undefined behavior may occur.
  ...

Marek

WARNING: multiple messages have this Message-ID (diff)
From: "Marek Behún" <kabel@kernel.org>
To: Vladimir Oltean <olteanv@gmail.com>
Cc: "Russell King (Oracle)" <linux@armlinux.org.uk>,
	Andrew Lunn <andrew@lunn.ch>,
	Heiner Kallweit <hkallweit1@gmail.com>,
	Alexandre Belloni <alexandre.belloni@bootlin.com>,
	Alvin __ipraga <alsi@bang-olufsen.dk>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	Claudiu Manoil <claudiu.manoil@nxp.com>,
	Daniel Scally <djrscally@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	DENG Qingfang <dqfext@gmail.com>,
	Eric Dumazet <edumazet@google.com>,
	Florian Fainelli <f.fainelli@gmail.com>,
	George McCollister <george.mccollister@gmail.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Hauke Mehrtens <hauke@hauke-m.de>,
	Heikki Krogerus <heikki.krogerus@linux.intel.com>,
	Jakub Kicinski <kuba@kernel.org>,
	Kurt Kanzenbach <kurt@linutronix.de>,
	Landen Chao <Landen.Chao@mediatek.com>,
	Linus Walleij <linus.walleij@linaro.org>,
	linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org,
	Matthias Brugger <matthias.bgg@gmail.com>,
	netdev@vger.kernel.org, Paolo Abeni <pabeni@redhat.com>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Sakari Ailus <sakari.ailus@linux.intel.com>,
	Sean Wang <sean.wang@mediatek.com>,
	UNGLinuxDriver@microchip.com,
	Vivien Didelot <vivien.didelot@gmail.com>,
	Woojung Huh <woojung.huh@microchip.com>
Subject: Re: [PATCH net-next 3/6] net: dsa: add support for retrieving the interface mode
Date: Fri, 22 Jul 2022 14:59:36 +0200	[thread overview]
Message-ID: <20220722145936.497ac73f@dellmb> (raw)
In-Reply-To: <20220721182216.z4vdaj4zfb6w3emo@skbuf>

On Thu, 21 Jul 2022 21:22:16 +0300
Vladimir Oltean <olteanv@gmail.com> wrote:

> On Thu, Jul 21, 2022 at 07:21:45PM +0200, Marek Behún wrote:
> > Marvell documentation says that 2500base-x does not implement inband
> > AN.  
> 
> Does Marvell documentation actually call it 2500base-x when it says it
> doesn't support in-band autoneg?

Yes, it does.

> > But when it was first implemented, for some reason it was thought that
> > 2500base-x is just 1000base-x at 2.5x speed, and 1000base-x does
> > support inband AN. Also it worked during tests for both switches and
> > SOC NICs, so it was enabled.
> > 
> > At the time 2500base-x was not standardized. Now 2500base-x is
> > stanradrized, and the standard says that 2500base-x does not support
> > clause 37 AN. I guess this is because where it is used, it is intended
> > to work with clause 73 AN somehow.  
> 
> When you say 2500base-x is standardized, do you mean there is a document
> somewhere which I could use to read more about this?

IEEE Std 802.3cb-2018: Amendment 1: Physical Layer Specifications and
Management Parameters for 2.5 Gb/s and 5 Gb/s Operation over Backplane.

Annex 127A (informative): Compatibility of 2.5GBASE-X PCS/PMA with
1000BASE-X PCS/PMA running 2.5 times faster

  ...
  This annex discusses the restrictions when operating 2.5GBASE-X
  PCS/PMA with a 1000BASE-X PCS/PMA link partner running 2.5 times
  faster. Compatibility of the PMD is outside the scope of this annex.
  In this annex when 1000BASE-X PCS/PMA is referred to, the 2.5 times
  speed up is implied.
  ...
  The 2.5GBASE-X PCS does not support Clause 37 Auto-Negotiation.
  Hence, the 1000BASE-X PCS is expected to have its Clause 37
  Auto-Negotiation functionality disabled so that the /C/ ordered set
  will not be transmitted. If a 2.5GBASE-X PCS receives /C/ ordered
  set, then undefined behavior may occur.
  ...

Marek

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2022-07-22 12:59 UTC|newest]

Thread overview: 168+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-15 16:00 [PATCH net-next 0/6] net: dsa: always use phylink Russell King (Oracle)
2022-07-15 16:00 ` Russell King (Oracle)
2022-07-15 16:01 ` [PATCH net-next 1/6] net: phylink: split out and export interface to caps translation Russell King (Oracle)
2022-07-15 16:01   ` Russell King (Oracle)
2022-07-15 16:01 ` [PATCH net-next 2/6] software node: allow named software node to be created Russell King
2022-07-15 16:01   ` Russell King
2022-07-15 19:57   ` Andy Shevchenko
2022-07-15 19:57     ` Andy Shevchenko
2022-07-15 20:17     ` Vladimir Oltean
2022-07-15 20:17       ` Vladimir Oltean
2022-07-15 20:33       ` Andy Shevchenko
2022-07-15 20:33         ` Andy Shevchenko
2022-07-15 20:48         ` Vladimir Oltean
2022-07-15 20:48           ` Vladimir Oltean
2022-07-18 12:29           ` Andy Shevchenko
2022-07-18 12:29             ` Andy Shevchenko
2022-07-18 13:27             ` Russell King (Oracle)
2022-07-18 13:27               ` Russell King (Oracle)
2022-07-18 18:43               ` Andy Shevchenko
2022-07-18 18:43                 ` Andy Shevchenko
2022-07-18 18:53                 ` Andy Shevchenko
2022-07-18 18:53                   ` Andy Shevchenko
2022-07-18 19:14                   ` Russell King (Oracle)
2022-07-18 19:14                     ` Russell King (Oracle)
2022-07-18 19:24                     ` Andy Shevchenko
2022-07-18 19:24                       ` Andy Shevchenko
2022-07-18 20:39                       ` Marek Behún
2022-07-18 20:39                         ` Marek Behún
2022-07-18 20:48                         ` Andy Shevchenko
2022-07-18 20:48                           ` Andy Shevchenko
2022-07-19  7:18                           ` Marek Behún
2022-07-19  7:18                             ` Marek Behún
2022-07-29 12:08                             ` Andy Shevchenko
2022-07-29 12:08                               ` Andy Shevchenko
2022-07-18 19:11                 ` Russell King (Oracle)
2022-07-18 19:11                   ` Russell King (Oracle)
2022-07-18 20:07                   ` Andy Shevchenko
2022-07-18 20:07                     ` Andy Shevchenko
2022-07-18 20:38                     ` Russell King (Oracle)
2022-07-18 20:38                       ` Russell King (Oracle)
2022-07-19  8:50                       ` Sakari Ailus
2022-07-19  8:50                         ` Sakari Ailus
2022-07-20 22:56                         ` Vladimir Oltean
2022-07-20 22:56                           ` Vladimir Oltean
2022-07-22  6:21                           ` Sakari Ailus
2022-07-22  6:21                             ` Sakari Ailus
2022-07-18 20:42                   ` Andrew Lunn
2022-07-18 20:42                     ` Andrew Lunn
2022-07-15 16:01 ` [PATCH net-next 3/6] net: dsa: add support for retrieving the interface mode Russell King (Oracle)
2022-07-15 16:01   ` Russell King (Oracle)
2022-07-15 17:24   ` Vladimir Oltean
2022-07-15 17:24     ` Vladimir Oltean
2022-07-15 21:31     ` Russell King (Oracle)
2022-07-15 21:31       ` Russell King (Oracle)
2022-07-15 22:23       ` Vladimir Oltean
2022-07-15 22:23         ` Vladimir Oltean
2022-07-15 22:57         ` Russell King (Oracle)
2022-07-15 22:57           ` Russell King (Oracle)
2022-07-16 10:57           ` Vladimir Oltean
2022-07-16 10:57             ` Vladimir Oltean
2022-07-16 11:13             ` Russell King (Oracle)
2022-07-16 11:13               ` Russell King (Oracle)
2022-07-16 12:36               ` Vladimir Oltean
2022-07-16 12:36                 ` Vladimir Oltean
2022-07-18  8:48                 ` Russell King (Oracle)
2022-07-18  8:48                   ` Russell King (Oracle)
2022-07-20 22:44                   ` Vladimir Oltean
2022-07-20 22:44                     ` Vladimir Oltean
2022-07-21 13:46                     ` Vladimir Oltean
2022-07-21 13:46                       ` Vladimir Oltean
2022-07-21 14:46                       ` Andrew Lunn
2022-07-21 14:46                         ` Andrew Lunn
2022-07-21 14:54                       ` Russell King (Oracle)
2022-07-21 14:54                         ` Russell King (Oracle)
2022-07-21 15:15                         ` Vladimir Oltean
2022-07-21 15:15                           ` Vladimir Oltean
2022-07-21 17:21                           ` Marek Behún
2022-07-21 17:21                             ` Marek Behún
2022-07-21 18:15                             ` Russell King (Oracle)
2022-07-21 18:15                               ` Russell King (Oracle)
2022-07-21 18:22                             ` Vladimir Oltean
2022-07-21 18:22                               ` Vladimir Oltean
2022-07-21 21:14                               ` Russell King (Oracle)
2022-07-21 21:14                                 ` Russell King (Oracle)
2022-07-21 21:36                                 ` Vladimir Oltean
2022-07-21 21:36                                   ` Vladimir Oltean
2022-07-22  8:28                                   ` Russell King (Oracle)
2022-07-22  8:28                                     ` Russell King (Oracle)
2022-07-22 10:52                                     ` Vladimir Oltean
2022-07-22 10:52                                       ` Vladimir Oltean
2022-07-22 11:44                                       ` Russell King (Oracle)
2022-07-22 11:44                                         ` Russell King (Oracle)
2022-07-22 12:14                                         ` Russell King (Oracle)
2022-07-22 12:14                                           ` Russell King (Oracle)
2022-07-22 12:46                                         ` Vladimir Oltean
2022-07-22 12:46                                           ` Vladimir Oltean
2022-07-22 13:16                                           ` Russell King (Oracle)
2022-07-22 13:16                                             ` Russell King (Oracle)
2022-07-22 16:56                                             ` Vladimir Oltean
2022-07-22 16:56                                               ` Vladimir Oltean
2022-07-22 21:20                                               ` Russell King (Oracle)
2022-07-22 21:20                                                 ` Russell King (Oracle)
2022-07-22 21:53                                                 ` Andrew Lunn
2022-07-22 21:53                                                   ` Andrew Lunn
2022-07-22 22:35                                                 ` Andrew Lunn
2022-07-22 22:35                                                   ` Andrew Lunn
2022-07-22 22:39                                                 ` Vladimir Oltean
2022-07-22 22:39                                                   ` Vladimir Oltean
2022-07-23  7:12                                                   ` Russell King (Oracle)
2022-07-23  7:12                                                     ` Russell King (Oracle)
2022-07-23 13:44                                                     ` Vladimir Oltean
2022-07-23 13:44                                                       ` Vladimir Oltean
2022-07-25 10:11                                                       ` Russell King (Oracle)
2022-07-25 10:11                                                         ` Russell King (Oracle)
2022-07-23 17:26                                                   ` Marek Behún
2022-07-23 17:26                                                     ` Marek Behún
2022-07-24 17:39                                                     ` Vladimir Oltean
2022-07-24 17:39                                                       ` Vladimir Oltean
2022-07-22 13:20                                         ` Andrew Lunn
2022-07-22 13:20                                           ` Andrew Lunn
2022-07-22 12:59                               ` Marek Behún [this message]
2022-07-22 12:59                                 ` Marek Behún
2022-07-22 13:23                                 ` Russell King (Oracle)
2022-07-22 13:23                                   ` Russell King (Oracle)
2022-07-22 14:19                                   ` Marek Behún
2022-07-22 14:19                                     ` Marek Behún
2022-07-15 16:01 ` [PATCH net-next 4/6] net: dsa: mv88e6xxx: report the default interface mode for the port Russell King (Oracle)
2022-07-15 16:01   ` Russell King (Oracle)
2022-07-15 16:01 ` [PATCH net-next 5/6] net: dsa: use swnode fixed-link if using default params Russell King (Oracle)
2022-07-15 16:01   ` Russell King (Oracle)
2022-07-15 20:11   ` Andy Shevchenko
2022-07-15 20:11     ` Andy Shevchenko
2022-07-15 21:36     ` Russell King (Oracle)
2022-07-15 21:36       ` Russell King (Oracle)
2022-07-18 18:59       ` Andy Shevchenko
2022-07-18 18:59         ` Andy Shevchenko
2022-07-18 19:13         ` Russell King (Oracle)
2022-07-18 19:13           ` Russell King (Oracle)
2022-07-18 20:08           ` Andy Shevchenko
2022-07-18 20:08             ` Andy Shevchenko
2022-07-15 16:01 ` [PATCH net-next 6/6] net: dsa: mv88e6xxx: remove handling for DSA and CPU ports Russell King (Oracle)
2022-07-15 16:01   ` Russell King (Oracle)
2022-07-15 17:17 ` [PATCH net-next 0/6] net: dsa: always use phylink Vladimir Oltean
2022-07-15 17:17   ` Vladimir Oltean
2022-07-15 20:59   ` Russell King (Oracle)
2022-07-15 20:59     ` Russell King (Oracle)
2022-07-15 23:03     ` Jakub Kicinski
2022-07-15 23:03       ` Jakub Kicinski
2022-07-16 11:15       ` Vladimir Oltean
2022-07-16 11:15         ` Vladimir Oltean
2022-07-16 11:43         ` Russell King (Oracle)
2022-07-16 11:43           ` Russell King (Oracle)
2022-07-16 13:13           ` Vladimir Oltean
2022-07-16 13:13             ` Vladimir Oltean
2022-07-18  8:53             ` Russell King (Oracle)
2022-07-18  8:53               ` Russell King (Oracle)
2022-07-18 12:45               ` Vladimir Oltean
2022-07-18 12:45                 ` Vladimir Oltean
2022-07-18 13:02                 ` Russell King (Oracle)
2022-07-18 13:02                   ` Russell King (Oracle)
2022-07-18 14:25                   ` Vladimir Oltean
2022-07-18 14:25                     ` Vladimir Oltean
2022-07-16 23:44         ` Jakub Kicinski
2022-07-16 23:44           ` Jakub Kicinski
2022-07-27  9:00 ` Marek Behún
2022-07-27  9:00   ` Marek Behún
2022-07-27 13:38   ` Vladimir Oltean
2022-07-27 13:38     ` Vladimir Oltean

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220722145936.497ac73f@dellmb \
    --to=kabel@kernel.org \
    --cc=Landen.Chao@mediatek.com \
    --cc=UNGLinuxDriver@microchip.com \
    --cc=alexandre.belloni@bootlin.com \
    --cc=alsi@bang-olufsen.dk \
    --cc=andrew@lunn.ch \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=claudiu.manoil@nxp.com \
    --cc=davem@davemloft.net \
    --cc=djrscally@gmail.com \
    --cc=dqfext@gmail.com \
    --cc=edumazet@google.com \
    --cc=f.fainelli@gmail.com \
    --cc=george.mccollister@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=hauke@hauke-m.de \
    --cc=heikki.krogerus@linux.intel.com \
    --cc=hkallweit1@gmail.com \
    --cc=kuba@kernel.org \
    --cc=kurt@linutronix.de \
    --cc=linus.walleij@linaro.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux@armlinux.org.uk \
    --cc=matthias.bgg@gmail.com \
    --cc=netdev@vger.kernel.org \
    --cc=olteanv@gmail.com \
    --cc=pabeni@redhat.com \
    --cc=rafael@kernel.org \
    --cc=sakari.ailus@linux.intel.com \
    --cc=sean.wang@mediatek.com \
    --cc=vivien.didelot@gmail.com \
    --cc=woojung.huh@microchip.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.