Re: [PATCH v2 05/14] net: ethernet: ti: cpsw: don't check slave num in runtime

2016-08-09 Thread Ivan Khoronzhuk
On 09.08.16 14:34, Grygorii Strashko wrote: On 08/06/2016 01:48 PM, Ivan Khoronzhuk wrote: No need to check const slave num in runtime for every packet, and ndev for slaves w/o ndev is anyway NULL. So remove redundant check. Signed-off-by: Ivan Khoronzhuk --- drivers/net/ethernet/ti/cpsw.c

Re: [PATCH v2 05/14] net: ethernet: ti: cpsw: don't check slave num in runtime

2016-08-09 Thread Grygorii Strashko
On 08/06/2016 01:48 PM, Ivan Khoronzhuk wrote: > No need to check const slave num in runtime for every packet, > and ndev for slaves w/o ndev is anyway NULL. So remove redundant > check. > > Signed-off-by: Ivan Khoronzhuk > --- > drivers/net/ethernet/ti/cpsw.c | 3 +-- > 1 file changed, 1 insert

Re: [PATCH v2 05/14] net: ethernet: ti: cpsw: don't check slave num in runtime

2016-08-09 Thread Mugunthan V N
On Saturday 06 August 2016 04:18 PM, Ivan Khoronzhuk wrote: > No need to check const slave num in runtime for every packet, > and ndev for slaves w/o ndev is anyway NULL. So remove redundant > check. > > Signed-off-by: Ivan Khoronzhuk Reviewed-by: Mugunthan V N Regards Mugunthan V N

[PATCH v2 05/14] net: ethernet: ti: cpsw: don't check slave num in runtime

2016-08-06 Thread Ivan Khoronzhuk
No need to check const slave num in runtime for every packet, and ndev for slaves w/o ndev is anyway NULL. So remove redundant check. Signed-off-by: Ivan Khoronzhuk --- drivers/net/ethernet/ti/cpsw.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/ethernet/ti/cp