2018-07-27 16:55:51

by Ivan Khoronzhuk

Subject: [PATCH net-next] net: ethernet: ti: cpsw: add missed RX_CTAG feature for second slave

Seems it was missed while adding for first net dev in dual-emac mode.

Signed-off-by: Ivan Khoronzhuk <[email protected]>
---
Based on net-next/master

drivers/net/ethernet/ti/cpsw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c
index 171abcfb6184..1b54c26c2bec 100644
--- a/drivers/net/ethernet/ti/cpsw.c
+++ b/drivers/net/ethernet/ti/cpsw.c
@@ -3287,7 +3287,7 @@ static int cpsw_probe_dual_emac(struct cpsw_priv *priv)

priv_sl2->emac_port = 1;
cpsw->slaves[1].ndev = ndev;
- ndev->features |= NETIF_F_HW_VLAN_CTAG_FILTER;
+ ndev->features |= NETIF_F_HW_VLAN_CTAG_FILTER | NETIF_F_HW_VLAN_CTAG_RX;

ndev->netdev_ops = &cpsw_netdev_ops;
ndev->ethtool_ops = &cpsw_ethtool_ops;
--
2.17.1

2018-07-27 17:39:23

by Grygorii Strashko

Subject: Re: [PATCH net-next] net: ethernet: ti: cpsw: add missed RX_CTAG feature for second slave

On 07/27/2018 11:54 AM, Ivan Khoronzhuk wrote:
> Seems it was missed while adding for first net dev in dual-emac mode.
>
> Signed-off-by: Ivan Khoronzhuk <[email protected]>
> ---
> Based on net-next/master

Reviewed-by: Grygorii Strashko <[email protected]>


--
regards,
-grygorii

2018-07-29 19:54:01

by David Miller

Subject: Re: [PATCH net-next] net: ethernet: ti: cpsw: add missed RX_CTAG feature for second slave

From: Ivan Khoronzhuk <[email protected]>
Date: Fri, 27 Jul 2018 19:54:39 +0300

> Seems it was missed while adding for first net dev in dual-emac mode.
>
> Signed-off-by: Ivan Khoronzhuk <[email protected]>

Applied to net-next.