mbox series

[net-next,0/2] net: dsa: bcm_sf2: Utilize PHYLINK for all ports

Message ID 20220815175009.2681932-1-f.fainelli@gmail.com (mailing list archive)
Headers show
Series net: dsa: bcm_sf2: Utilize PHYLINK for all ports | expand

Message

Florian Fainelli Aug. 15, 2022, 5:50 p.m. UTC 
Hi all,

This patch series has the bcm_sf2 driver utilize PHYLINK to configure
the CPU port link parameters to unify the configuration and pave the way
for DSA to utilize PHYLINK for all ports in the future.

Tested on BCM7445 and BCM7278

Florian Fainelli (2):
  net: dsa: bcm_sf2: Introduce helper for port override offset
  net: dsa: bcm_sf2: Have PHYLINK configure CPU/IMP port(s)

 drivers/net/dsa/bcm_sf2.c | 130 ++++++++++++++++++--------------------
 1 file changed, 63 insertions(+), 67 deletions(-)

Comments

Jakub Kicinski Aug. 17, 2022, 3:54 p.m. UTC | #1 
On Mon, 15 Aug 2022 10:50:07 -0700 Florian Fainelli wrote:
> Hi all,
> 
> This patch series has the bcm_sf2 driver utilize PHYLINK to configure
> the CPU port link parameters to unify the configuration and pave the way
> for DSA to utilize PHYLINK for all ports in the future.
> 
> Tested on BCM7445 and BCM7278

Last call for reviews..
Vladimir Oltean Aug. 17, 2022, 5:27 p.m. UTC | #2 
On Wed, Aug 17, 2022 at 08:54:14AM -0700, Jakub Kicinski wrote:
> On Mon, 15 Aug 2022 10:50:07 -0700 Florian Fainelli wrote:
> > Hi all,
> > 
> > This patch series has the bcm_sf2 driver utilize PHYLINK to configure
> > the CPU port link parameters to unify the configuration and pave the way
> > for DSA to utilize PHYLINK for all ports in the future.
> > 
> > Tested on BCM7445 and BCM7278
> 
> Last call for reviews..

My review is: let's see what breaks...
Jakub Kicinski Aug. 17, 2022, 5:56 p.m. UTC | #3 
On Wed, 17 Aug 2022 20:27:04 +0300 Vladimir Oltean wrote:
> On Wed, Aug 17, 2022 at 08:54:14AM -0700, Jakub Kicinski wrote:
> > On Mon, 15 Aug 2022 10:50:07 -0700 Florian Fainelli wrote:  
> > > Hi all,
> > > 
> > > This patch series has the bcm_sf2 driver utilize PHYLINK to configure
> > > the CPU port link parameters to unify the configuration and pave the way
> > > for DSA to utilize PHYLINK for all ports in the future.
> > > 
> > > Tested on BCM7445 and BCM7278  
> > 
> > Last call for reviews..  
> 
> My review is: let's see what breaks...

*nod*, thanks!
patchwork-bot+netdevbpf@kernel.org Aug. 17, 2022, 6:40 p.m. UTC | #4 
Hello:

This series was applied to netdev/net-next.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Mon, 15 Aug 2022 10:50:07 -0700 you wrote:
> Hi all,
> 
> This patch series has the bcm_sf2 driver utilize PHYLINK to configure
> the CPU port link parameters to unify the configuration and pave the way
> for DSA to utilize PHYLINK for all ports in the future.
> 
> Tested on BCM7445 and BCM7278
> 
> [...]

Here is the summary with links:
  - [net-next,1/2] net: dsa: bcm_sf2: Introduce helper for port override offset
    https://git.kernel.org/netdev/net-next/c/1ed26ce4850a
  - [net-next,2/2] net: dsa: bcm_sf2: Have PHYLINK configure CPU/IMP port(s)
    https://git.kernel.org/netdev/net-next/c/4d2f6dde4daa

You are awesome, thank you!