mbox series

[net-next,0/3] dpaa2-eth: small updates

Message ID 20200925144421.7811-1-ioana.ciornei@nxp.com
Headers show
Series dpaa2-eth: small updates | expand

Message

Ioana Ciornei Sept. 25, 2020, 2:44 p.m. UTC
This patch set is just a collection of small updates to the dpaa2-eth
driver.

First, we only need to check the availability of the DTS child node, not
both child and parent node. Then remove a call to
dpaa2_eth_link_state_update() which is now just a leftover and it's not
useful in how are things working now in the PHY integration. Lastly,
modify how the driver is behaving when the the flow steering table is
used between all the traffic classes.

Ioana Ciornei (2):
  dpaa2-mac: do not check for both child and parent DTS nodes
  dpaa2-eth: no need to check link state right after ndo_open

Ionut-robert Aron (1):
  dpaa2-eth: install a single steering rule when SHARED_FS is enabled

 .../net/ethernet/freescale/dpaa2/dpaa2-eth.c  | 25 ++++++++++---------
 .../ethernet/freescale/dpaa2/dpaa2-ethtool.c  |  2 +-
 .../net/ethernet/freescale/dpaa2/dpaa2-mac.c  |  3 +--
 drivers/net/ethernet/freescale/dpaa2/dpni.h   |  4 +++
 4 files changed, 19 insertions(+), 15 deletions(-)

Comments

Andrew Lunn Sept. 25, 2020, 4:31 p.m. UTC | #1
On Fri, Sep 25, 2020 at 05:44:19PM +0300, Ioana Ciornei wrote:
> There is no need to check if both the MDIO controller node and its

> child node, the PCS device, are available since there is no chance that

> the child node would be enabled when the parent it's not.

> 

> Suggested-by: Andrew Lunn <andrew@lunn.ch>

> Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com>


Reviewed-by: Andrew Lunn <andrew@lunn.ch>


    Andrew
David Miller Sept. 26, 2020, 12:14 a.m. UTC | #2
From: Ioana Ciornei <ioana.ciornei@nxp.com>
Date: Fri, 25 Sep 2020 17:44:18 +0300

> This patch set is just a collection of small updates to the dpaa2-eth
> driver.
> 
> First, we only need to check the availability of the DTS child node, not
> both child and parent node. Then remove a call to
> dpaa2_eth_link_state_update() which is now just a leftover and it's not
> useful in how are things working now in the PHY integration. Lastly,
> modify how the driver is behaving when the the flow steering table is
> used between all the traffic classes.

Series applied.