Re: [PATCH v15 0/4] Adding the Sparx5 Serdes driver
On 18-02-21, 17:14, Steen Hegelund wrote: > Adding the Sparx5 Serdes driver > > This series of patches provides the serdes driver for the Microchip Sparx5 > ethernet switch. > > The serdes driver supports the 10G and 25G serdes instances available in the > Sparx5. > > The Sparx5 serdes support several interface modes with several speeds and also > allows the client to change the mode and the speed according to changing in > the > environment such as changing cables from DAC to fiber. Applied patch 1 thru 3... thanks -- ~Vinod
Re: [PATCH v15 0/4] Adding the Sparx5 Serdes driver
Hi Vinod, On Tue, 2021-03-16 at 10:23 +0530, Vinod Koul wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > Hello Steen, > > On 15-03-21, 16:04, Steen Hegelund wrote: > > Hi Kishon, Vinod, Andrew, Jacub, and David, > > > > I just wanted to know if you think that the Generic PHY subsystem might > > not be the right place for this Ethernet SerDes PHY driver after all. > > > > Originally I chose this subsystem for historic reasons: The > > Microchip/Microsemi Ocelot SerDes driver was added here when it was > > upstreamed. > > On the other hand the Ocelot Serdes can do both PCIe and Ethernet, so > > it might fit the signature of a generic PHY better. > > > > At the moment the acceptance of the Sparx5 Serdes driver is blocking us > > from adding the Sparx5 SwitchDev driver (to net), so it is really > > important for us to resolve which subsystem the Serdes driver belongs > > to. > > > > I am very much looking forward to your response. > > Generic PHY IMO is the right place for this series, I shall review it > shortly and do the needful. I have asked Kishon to check the new phy API > and ack it... > > Thanks > -- > ~Vinod Thank you very much for the confirmation. BR Steen
Re: [PATCH v15 0/4] Adding the Sparx5 Serdes driver
Hi Jacub, On Mon, 2021-03-15 at 10:26 -0700, Jakub Kicinski wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > On Mon, 15 Mar 2021 16:04:24 +0100 Steen Hegelund wrote: > > Hi Kishon, Vinod, Andrew, Jacub, and David, > > > > I just wanted to know if you think that the Generic PHY subsystem might > > not be the right place for this Ethernet SerDes PHY driver after all. > > > > Originally I chose this subsystem for historic reasons: The > > Microchip/Microsemi Ocelot SerDes driver was added here when it was > > upstreamed. > > On the other hand the Ocelot Serdes can do both PCIe and Ethernet, so > > it might fit the signature of a generic PHY better. > > Are you saying this PHY is Ethernet only? Yes this particular PHY is Ethernet only (but the Sparx5 also has a separate PCI PHY). > > > At the moment the acceptance of the Sparx5 Serdes driver is blocking us > > from adding the Sparx5 SwitchDev driver (to net), so it is really > > important for us to resolve which subsystem the Serdes driver belongs > > to. > > > > I am very much looking forward to your response. > > FWIW even if this is merged via gen phy subsystem we can pull it into > net-next as well to unblock your other work in this dev cycle. You just > need to send the patches as a pull request, based on merge-base between > the gen phy tree and net-next. -- BR Steen -=-=-=-=-=-=-=-=-=-=-=-=-=-= steen.hegel...@microchip.com
Re: [PATCH v15 0/4] Adding the Sparx5 Serdes driver
Hello Steen, On 15-03-21, 16:04, Steen Hegelund wrote: > Hi Kishon, Vinod, Andrew, Jacub, and David, > > I just wanted to know if you think that the Generic PHY subsystem might > not be the right place for this Ethernet SerDes PHY driver after all. > > Originally I chose this subsystem for historic reasons: The > Microchip/Microsemi Ocelot SerDes driver was added here when it was > upstreamed. > On the other hand the Ocelot Serdes can do both PCIe and Ethernet, so > it might fit the signature of a generic PHY better. > > At the moment the acceptance of the Sparx5 Serdes driver is blocking us > from adding the Sparx5 SwitchDev driver (to net), so it is really > important for us to resolve which subsystem the Serdes driver belongs > to. > > I am very much looking forward to your response. Generic PHY IMO is the right place for this series, I shall review it shortly and do the needful. I have asked Kishon to check the new phy API and ack it... Thanks -- ~Vinod
Re: [PATCH v15 0/4] Adding the Sparx5 Serdes driver
On Mon, 15 Mar 2021 16:04:24 +0100 Steen Hegelund wrote: > Hi Kishon, Vinod, Andrew, Jacub, and David, > > I just wanted to know if you think that the Generic PHY subsystem might > not be the right place for this Ethernet SerDes PHY driver after all. > > Originally I chose this subsystem for historic reasons: The > Microchip/Microsemi Ocelot SerDes driver was added here when it was > upstreamed. > On the other hand the Ocelot Serdes can do both PCIe and Ethernet, so > it might fit the signature of a generic PHY better. Are you saying this PHY is Ethernet only? > At the moment the acceptance of the Sparx5 Serdes driver is blocking us > from adding the Sparx5 SwitchDev driver (to net), so it is really > important for us to resolve which subsystem the Serdes driver belongs > to. > > I am very much looking forward to your response. FWIW even if this is merged via gen phy subsystem we can pull it into net-next as well to unblock your other work in this dev cycle. You just need to send the patches as a pull request, based on merge-base between the gen phy tree and net-next.
Re: [PATCH v15 0/4] Adding the Sparx5 Serdes driver
Hi Kishon, Vinod, Andrew, Jacub, and David, I just wanted to know if you think that the Generic PHY subsystem might not be the right place for this Ethernet SerDes PHY driver after all. Originally I chose this subsystem for historic reasons: The Microchip/Microsemi Ocelot SerDes driver was added here when it was upstreamed. On the other hand the Ocelot Serdes can do both PCIe and Ethernet, so it might fit the signature of a generic PHY better. At the moment the acceptance of the Sparx5 Serdes driver is blocking us from adding the Sparx5 SwitchDev driver (to net), so it is really important for us to resolve which subsystem the Serdes driver belongs to. I am very much looking forward to your response. BR Steen On Thu, 2021-02-18 at 17:14 +0100, Steen Hegelund wrote: > Adding the Sparx5 Serdes driver > > This series of patches provides the serdes driver for the Microchip > Sparx5 > ethernet switch. > > The serdes driver supports the 10G and 25G serdes instances available > in the > Sparx5. > > The Sparx5 serdes support several interface modes with several speeds > and also > allows the client to change the mode and the speed according to > changing in the > environment such as changing cables from DAC to fiber. > > The serdes driver is to be used by the Sparx5 switchdev driver that > will follow in subsequent series. > > Sparx5 Architecture: > > > Below is a diagram of the Ethernet transport part of the Sparx5 chip. > > The diagram shows the switch core that sends/receives traffic via the > Frame Bus > and passes to the Port Modules. > The Port Modules are able to talk to a SerDes via a Port Muxing > configuration. > The SerDes instances (33 in all) then passes the traffic on its lanes > to the > attached cuPHY or SFP module. > > +-+ > | | > | Switch Core | > | | > +++ > | > ---+--+--+---+--+-+ Frame > Bus > | | | | > +--+-+ +--+-+ +--+-+ +--+-+ > |1G/2.G Port | |5G Port | |10G Port | |25GG Port | > |Modules | |Modules | |Modules | |Modules | > |MAC, PCS | |MAC, PCS | |MAC, PCS | |MAC, PCS | > +--+-+ +--+-+ +--+-+ +--+-+ > | | | | > ---+-++---+--+--+---+-+ Port > Muxing > | | | > +-++ +-++ +--+---+ > |SerDes 5G | |SerDes 10G| |SerDes 25G| SerDes > Driver > |Lane (13) | |Lane (12) | |Lane (8) | Controls > these > +-++ +-++ +-++ > | | | > to cuPHY to cuPHY to cuPHY > or SFP or SFP or SFP > > The 33 SerDes instances are handled internally by 2 SerDes macros > types: > > - A 10G SerDes macro that supports the following rates and modes: > - 100 Mbps: > - 100BASE-FX > - 1.25 Gbps: > - SGMII > - 1000BASE-X > - 1000BASE-KX > - 3.125 Gbps: > - 2.5GBASE-X > - 2.5GBASE-KX > - 5 Gbps: > - QSGMII > - USGMII > - 5.15625 Gbps: > - 5GBASE-KR > - 5G-USXGMII > - 10 Gbps: > - 10G-USGMII > - 10.3125 Gbps: > - 10GBASE-R > - 10GBASE-KR > - USXGMII > > - A 25G SerDes macro that supports the following rates and modes: > - 1.25 Gbps: > - SGMII > - 1000BASE-X > - 1000BASE-KX > - 3.125 Gbps: > - 2.5GBASE-X > - 2.5GBASE-KX > - 5 Gbps: > - QSGMII > - USGMII > - 5.15625 Gbps: > - 5GBASE-KR > - 5G-USXGMII > - 10 Gbps: > - 10G-USGMII > - 10.3125 Gbps: > - 10GBASE-R > - 10GBASE-KR > - USXGMII > - 10.3125 Gbps: > - 10GBASE-R > - 10GBASE-KR > - USXGMII > - 25.78125 Gbps: > - 25GBASE-KR > - 25GBASE-CR > - 25GBASE-SR > - 25GBASE-LR > - 25GBASE-ER > > The SerDes driver handles these SerDes instances and configures them > based on > the selected mode, speed and media type. > > In the current version of the SerDes driver only a subset of the above > modes > are supported: the modes that can be tested on our current evaluation > boards > (PCB134 and PCB35). > > The first 13 10G SerDes macros are limited to 6G, and this gives the > SerDes > instance architecture shown on the diagram above. > > The Port Muxing allows a Port Module to use a specific SerDes instance, > but not > all
Re: [PATCH v15 0/4] Adding the Sparx5 Serdes driver
From: Steen Hegelund Date: Thu, 18 Feb 2021 17:14:47 +0100 > Adding the Sparx5 Serdes driver > The net-next tree is closed so this will have to wait until the next merge window. Thanks.
[PATCH v15 0/4] Adding the Sparx5 Serdes driver
Adding the Sparx5 Serdes driver This series of patches provides the serdes driver for the Microchip Sparx5 ethernet switch. The serdes driver supports the 10G and 25G serdes instances available in the Sparx5. The Sparx5 serdes support several interface modes with several speeds and also allows the client to change the mode and the speed according to changing in the environment such as changing cables from DAC to fiber. The serdes driver is to be used by the Sparx5 switchdev driver that will follow in subsequent series. Sparx5 Architecture: Below is a diagram of the Ethernet transport part of the Sparx5 chip. The diagram shows the switch core that sends/receives traffic via the Frame Bus and passes to the Port Modules. The Port Modules are able to talk to a SerDes via a Port Muxing configuration. The SerDes instances (33 in all) then passes the traffic on its lanes to the attached cuPHY or SFP module. +-+ | | | Switch Core | | | +++ | ---+--+--+---+--+-+ Frame Bus | | | | +--+-+ +--+-+ +--+-+ +--+-+ |1G/2.G Port | |5G Port | |10G Port| |25GG Port | |Modules | |Modules | |Modules | |Modules | |MAC, PCS| |MAC, PCS| |MAC, PCS| |MAC, PCS| +--+-+ +--+-+ +--+-+ +--+-+ | | | | ---+-++---+--+--+---+-+ Port Muxing || | +-++ +-++ +--+---+ |SerDes 5G | |SerDes 10G| |SerDes 25G|SerDes Driver |Lane (13) | |Lane (12) | |Lane (8) |Controls these +-++ +-++ +-++ ||| to cuPHY to cuPHY to cuPHY or SFP or SFP or SFP The 33 SerDes instances are handled internally by 2 SerDes macros types: - A 10G SerDes macro that supports the following rates and modes: - 100 Mbps: - 100BASE-FX - 1.25 Gbps: - SGMII - 1000BASE-X - 1000BASE-KX - 3.125 Gbps: - 2.5GBASE-X - 2.5GBASE-KX - 5 Gbps: - QSGMII - USGMII - 5.15625 Gbps: - 5GBASE-KR - 5G-USXGMII - 10 Gbps: - 10G-USGMII - 10.3125 Gbps: - 10GBASE-R - 10GBASE-KR - USXGMII - A 25G SerDes macro that supports the following rates and modes: - 1.25 Gbps: - SGMII - 1000BASE-X - 1000BASE-KX - 3.125 Gbps: - 2.5GBASE-X - 2.5GBASE-KX - 5 Gbps: - QSGMII - USGMII - 5.15625 Gbps: - 5GBASE-KR - 5G-USXGMII - 10 Gbps: - 10G-USGMII - 10.3125 Gbps: - 10GBASE-R - 10GBASE-KR - USXGMII - 10.3125 Gbps: - 10GBASE-R - 10GBASE-KR - USXGMII - 25.78125 Gbps: - 25GBASE-KR - 25GBASE-CR - 25GBASE-SR - 25GBASE-LR - 25GBASE-ER The SerDes driver handles these SerDes instances and configures them based on the selected mode, speed and media type. In the current version of the SerDes driver only a subset of the above modes are supported: the modes that can be tested on our current evaluation boards (PCB134 and PCB35). The first 13 10G SerDes macros are limited to 6G, and this gives the SerDes instance architecture shown on the diagram above. The Port Muxing allows a Port Module to use a specific SerDes instance, but not all combinations are allowed. This is functionality as well as the configuration of the Port Modules is handled by the SwitchDev Driver. The Sparx5 Chip Register Model can be browsed at this location: https://github.com/microchip-ung/sparx-5_reginfo and the datasheet is available here: https://ww1.microchip.com/downloads/en/DeviceDoc/SparX-5_Family_L2L3_Enterprise_10G_Ethernet_Switches_Datasheet_3822B.pdf History: v14 -> v15: Changed the default interface function return value to -ENODEV. Moved the CMU initialization, so that it is always done before the serdes instance configuration. Added a reference to the Sparx5 datasheet. v13 -> v14: Changed the 25g apply, 10g apply and the CMU configuration functions to use a table based register update structure. The table entries still need serdes indices/instances so the table must be generated per serdes. v12 -> v13: Interface changes: - Added set_media and set_speed interfaces on the generic phy - Removed the ethernet SerDes
