Re: [PATCH] net: hns: dereference ppe_cb->ppe_common_cb if it is non-null

2016-08-25 Thread Yisen Zhuang
this > by moving the initialisation of dsaf_dev once we know > ppe_cb->ppe_common_cb is OK to dereference. > > Signed-off-by: Colin Ian King Acked-by: Yisen Zhuang

Re: [PATCH 1/1 linux-next] net: hns: fix typo in g_gmac_stats_string[]

2016-08-10 Thread Yisen Zhuang
Hi Fabian, This patch is fine to me, many thanks. Yisen 在 2016/8/10 23:48, Fabian Frederick 写道: > s/gamc/gmac/ > > Signed-off-by: Fabian Frederick > --- > drivers/net/ethernet/hisilicon/hns/hns_dsaf_gmac.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/e

Re: [PATCH 06/15] ethernet: hisilicon: hns: hns_dsaf_mac: add missing of_node_put after calling of_parse_phandle

2016-07-27 Thread Yisen Zhuang
在 2016/7/27 10:20, Peter Chen 写道: > of_node_put needs to be called when the device node which is got > from of_parse_phandle has finished using. > > Signed-off-by: Peter Chen > --- > drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c | 9 ++--- > 1 file changed, 6 insertions(+), 3 deletions

[PATCH v3 net-next 4/9] net: hns: delete redundant parenthese

2016-07-01 Thread Yisen Zhuang
From: Daode Huang According to the previous review comments from Andy, this patch deletes the redundant parens in the patch. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH v3 net-next 1/9] MAINTAINERS: add maintainers for hns driver

2016-07-01 Thread Yisen Zhuang
From: Daode Huang This patch adds maintainers for hisilicon network subsystem driver Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- change log: v3: match all files in and below drivers/net/ethernet/hisilicon/ v2: the same as v1 v1: first submit Link: https://lkml.org/lkml

[PATCH v3 net-next 6/9] net: hns: normalize two different loop

2016-07-01 Thread Yisen Zhuang
From: Daode Huang There are two approaches to assign data, one does 2 loops, another does 1 loop. This patch normalize the different methods to 1 loop. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 18 +- 1

[PATCH v3 net-next 0/9] net: hns: fix the typo of hns

2016-07-01 Thread Yisen Zhuang
This series includes typo fixes which review by Andy, adding the hns maintainer to MAINTAINERS, as below: > from Daode: adds the maintainer for hns driver; > from Daode: fix the typo of hns reviewed by Andy Shevchenko; > from Kejian: one remove redundant function and two fix to get configura

[PATCH v3 net-next 7/9] net: hns: remove redundant hns_mac_dev_to_enet_if()

2016-07-01 Thread Yisen Zhuang
redundant. Reported-by: Jinchuan Tian Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c b/drivers/net/ethernet/hisilicon

[PATCH v3 net-next 2/9] net: hns: fix code style about hns driver

2016-07-01 Thread Yisen Zhuang
From: Daode Huang This patch fixes code sytle of hns driver to make it simple. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_misc.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/drivers/net

[PATCH v3 net-next 5/9] net: hns: add a space before "*/"

2016-07-01 Thread Yisen Zhuang
From: Daode Huang In comment line, some time miss a space before */, so this patch adds a space before */. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_enet.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH v3 net-next 9/9] net: hns: get reset registers from DT

2016-07-01 Thread Yisen Zhuang
From: Kejian Yan Since the registers of subctrl may be different, it is better to mv the registers from hns mdio driver routine to device tree node. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v2: fix the wrong indentations v1: first submit link: https

[PATCH v3 net-next 3/9] net: hns: change code style from a = a + x to a += x

2016-07-01 Thread Yisen Zhuang
From: Daode Huang This patch fixes the code style in hns driver. Change it from "buff = buff + xxx" to "buff += xxx". The reveiw comments is from andy. Reviewed-by: Andriy Shevchenko Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net/et

[PATCH v3 net-next 8/9] net: hns: add media-type property for hns

2016-07-01 Thread Yisen Zhuang
From: Kejian Yan It is PORT_TP type if the service port is GE mode. It is wrong to judge the port type by using if it is service port. Adding the media type to know port type. Reported-by: Jinchuan Tian Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon

[PATCH v2 net-next 9/9] net: hns: get reset registers from DT

2016-06-30 Thread Yisen Zhuang
From: Kejian Yan Since the registers of subctrl may be different, it is better to mv the registers from hns mdio driver routine to device tree node. Signed-off-by: Kejian Yan --- change log: v2: fix the wrong indentations v1: first submit link: https://lkml.org/lkml/2016/6/27/182 --- driv

[PATCH v2 net-next 8/9] net: hns: add media-type property for hns

2016-06-30 Thread Yisen Zhuang
From: Kejian Yan It is PORT_TP type if the service port is GE mode. It is wrong to judge the port type by using if it is service port. Adding the media type to know port type. Reported-by: Jinchuan Tian Signed-off-by: Kejian Yan --- drivers/net/ethernet/hisilicon/hns/hnae.h | 9 +

[PATCH v2 net-next 2/9] net: hns: fix code style about hns driver

2016-06-30 Thread Yisen Zhuang
From: Daode Huang This patch fixes code sytle of hns driver to make it simple. Signed-off-by: Daode Huang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_misc.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_misc

[PATCH v2 net-next 5/9] net: hns: add a space before "*/"

2016-06-30 Thread Yisen Zhuang
From: Daode Huang In comment line, some time miss a space before */, so this patch adds a space before */. Signed-off-by: Daode Huang --- drivers/net/ethernet/hisilicon/hns/hns_enet.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns

[PATCH v2 net-next 3/9] net: hns: change code style from a = a + x to a += x

2016-06-30 Thread Yisen Zhuang
From: Daode Huang This patch fixes the code style in hns driver. Change it from "buff = buff + xxx" to "buff += xxx". The reveiw comments is from andy. Reviewed-by: Andriy Shevchenko Signed-off-by: Daode Huang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 32 +++---

[PATCH v2 net-next 1/9] MAINTAINERS: add maintainers for hns driver

2016-06-30 Thread Yisen Zhuang
+5421,15 @@ F: include/uapi/linux/if_hippi.h F: net/802/hippi.c F: drivers/net/hippi/ +HISILICON NETWORK SUBSYSTEM DRIVER +M: Yisen Zhuang +M: Salil Mehta +L: netdev@vger.kernel.org +W: http://www.hisilicon.com +S: Maintained +F: drivers/net/ethernet

[PATCH v2 net-next 0/9] net: hns: fix the typo of hns

2016-06-30 Thread Yisen Zhuang
This series includes typo fixes which review by Andy, adding the hns maintainer to MAINTAINERS, as below: > from Daode: adds the maintainer for hns driver; > from Daode: fix the typo of hns reviewed by Andy Shevchenko; > from Kejian: one remove redundant function and two fix to get configura

[PATCH v2 net-next 6/9] net: hns: normalize two different loop

2016-06-30 Thread Yisen Zhuang
From: Daode Huang There are two approaches to assign data, one does 2 loops, another does 1 loop. This patch normalize the different methods to 1 loop. Signed-off-by: Daode Huang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 18 +- 1 file changed, 9 insertions(+), 9

[PATCH v2 net-next 7/9] net: hns: remove redundant hns_mac_dev_to_enet_if()

2016-06-30 Thread Yisen Zhuang
From: Kejian Yan The sequence of hns_mac_dev_to_enet_if() is the same as hns_get_enet_interface(), and hns_get_enet_interface() is called by initialization to get the mac mode. And the mode is not changed anywhere. Thus add hns_mac_dev_to_enet_if() function to get the mac mode is obviously redund

[PATCH v2 net-next 4/9] net: hns: delete redundant parenthese

2016-06-30 Thread Yisen Zhuang
From: Daode Huang According to the previous review comments from Andy, this patch deletes the redundant parens in the patch. Signed-off-by: Daode Huang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/eth

[PATCH net-next 7/9] net: hns: remove redundant hns_mac_dev_to_enet_if()

2016-06-27 Thread Yisen Zhuang
From: Kejian Yan The sequence of hns_mac_dev_to_enet_if() is the same as hns_get_enet_interface(), and hns_get_enet_interface() is called by initialization to get the mac mode. And the mode is not changed anywhere. Thus add hns_mac_dev_to_enet_if() function to get the mac mode is obviously redund

[PATCH net-next 1/9] MAINTAINERS: add maintainers for hns driver

2016-06-27 Thread Yisen Zhuang
+5421,15 @@ F: include/uapi/linux/if_hippi.h F: net/802/hippi.c F: drivers/net/hippi/ +HISILICON NETWORK SUBSYSTEM DRIVER +M: Yisen Zhuang +M: Salil Mehta +L: netdev@vger.kernel.org +W: http://www.hisilicon.com +S: Maintained +F: drivers/net/ethernet

[PATCH net-next 3/9] net: hns: change code style from a = a + x to a += x

2016-06-27 Thread Yisen Zhuang
From: Daode Huang This patch fixes the code style in hns driver. Change it from "buff = buff + xxx" to "buff += xxx". The reveiw comments is from andy. Reviewed-by: Andriy Shevchenko Signed-off-by: Daode Huang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 32 +++---

[PATCH net-next 4/9] net: hns: delete redundant parens

2016-06-27 Thread Yisen Zhuang
From: Daode Huang According to the previous review comments from Andy, this patch deletes the redundant parens in the patch. Signed-off-by: Daode Huang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/eth

[PATCH net-next 9/9] net: hns: get reset registers from DT

2016-06-27 Thread Yisen Zhuang
From: Kejian Yan Since the registers of subctrl may be different, it is better to mv the registers from hns mdio driver routine to device tree node. Signed-off-by: Kejian Yan --- drivers/net/ethernet/hisilicon/hns_mdio.c | 80 +-- 1 file changed, 66 insertions(+), 1

[PATCH net-next 2/9] net: hns: fix code style about hns driver

2016-06-27 Thread Yisen Zhuang
From: Daode Huang This patch fixes code sytle of hns driver to make it simple. Signed-off-by: Daode Huang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_misc.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_misc

[PATCH net-next 5/9] net: hns: add a space before "*/"

2016-06-27 Thread Yisen Zhuang
From: Daode Huang In comment line, some time miss a space before */, so this patch adds a space before */. Signed-off-by: Daode Huang --- drivers/net/ethernet/hisilicon/hns/hns_enet.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns

[PATCH net-next 6/9] net: hns: normalize two different loop

2016-06-27 Thread Yisen Zhuang
From: Daode Huang There are two approaches to assign data, one does 2 loops, another does 1 loop. This patch normalize the different methods to 1 loop. Signed-off-by: Daode Huang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 18 +- 1 file changed, 9 insertions(+), 9

[PATCH net-next 8/9] net: hns: add media-type property for hns

2016-06-27 Thread Yisen Zhuang
From: Kejian Yan It is PORT_TP type if the service port is GE mode. It is wrong to judge the port type by using if it is service port. Adding the media type to know port type. Reported-by: Jinchuan Tian Signed-off-by: Kejian Yan --- drivers/net/ethernet/hisilicon/hns/hnae.h | 9 +

[PATCH net-next 0/9] net: hns: fix the typo of hns

2016-06-27 Thread Yisen Zhuang
This series includes typo fixes which review by Andy, adding the hns maintainer to MAINTAINERS, > adds the maintainer for hns driver from Daode; > from Daode: fix the typo of hns reviewed by Andy Shevchenko; > from Kejian: one remove redundant function and two fix to get configuration from D

Re: [PATCH net-next 10/19] net: hns: bugfix about pfc pause frame statistics

2016-06-22 Thread Yisen Zhuang
在 2016/6/22 17:41, Andy Shevchenko 写道: > On Wed, 2016-06-22 at 09:43 +0800, Yisen Zhuang wrote: >> >> 在 2016/6/21 18:32, Andy Shevchenko 写道: >>> On Tue, 2016-06-21 at 11:56 +0800, Yisen Zhuang wrote: >>>> From: Daode Huang >>>> >>>> Fo

Re: [PATCH net-next 16/19] net: hns: fix bug that alloc skb fail lead to port unavailable

2016-06-21 Thread Yisen Zhuang
在 2016/6/21 21:25, Sergei Shtylyov 写道: > Hello. > > On 6/21/2016 6:56 AM, Yisen Zhuang wrote: > >> From: Jun He >> >> When hns_nic_poll_rx_skb alloc skb fail, it will break receive cycle and >> read new fbd_num to start new receive cycle. It reco

Re: [PATCH net-next 10/19] net: hns: bugfix about pfc pause frame statistics

2016-06-21 Thread Yisen Zhuang
在 2016/6/21 18:32, Andy Shevchenko 写道: > On Tue, 2016-06-21 at 11:56 +0800, Yisen Zhuang wrote: >> From: Daode Huang >> >> For SoC hip06, PFC pause handled in dsaf, while hip05 in XGMAC, >> so change the statistics of pfc pause in dsaf and remove the old &g

Re: [PATCH net-next 01/19] net: hns: bug fix of ge reset sequence

2016-06-21 Thread Yisen Zhuang
在 2016/6/21 18:35, Andy Shevchenko 写道: > On Tue, 2016-06-21 at 11:56 +0800, Yisen Zhuang wrote: >> From: Qianqian Xie >> >> The bit fileds of PPE reset register are different between HNS v1 and >> HNS v2, but the current procedure just only match HNS v1. H

[PATCH net-next 06/19] net: hns: fix the error info when dma_set_mask_and_coherent fail

2016-06-20 Thread Yisen Zhuang
From: Qianqian Xie The error info should be printed as "set mask to 64bit fail!" instead of "set mask to 32bit fail!" in dma_set_mask_and_coherent(). Signed-off-by: Qianqian Xie Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_enet.c | 2 +- 1 file

[PATCH net-next 16/19] net: hns: fix bug that alloc skb fail lead to port unavailable

2016-06-20 Thread Yisen Zhuang
we will goto out when alloc skb fail to fix this bug. Signed-off-by: Jun He Signed-off-by: Ding Tianhong Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_enet.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns

[PATCH net-next 05/19] net: hns: Remove unnecessary device resource free

2016-06-20 Thread Yisen Zhuang
From: Qianqian Xie The driver uses devm_ioremap_resource, it will unmap the map automatically, remove the unnecessary the resource free. Signed-off-by: Qianqian Xie Reported-by: Kefeng Wang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 39

[PATCH net-next 04/19] net: hns: typo fix of annotation info for hns_nic_reset_subtask()

2016-06-20 Thread Yisen Zhuang
From: Qianqian Xie The annotation info for hns_nic_reset_subtask() should be "for resetting subtask" instead of "for resetting suntask". Signed-off-by: Qianqian Xie Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_enet.c | 2 +- 1 file change

[PATCH net-next 08/19] net: hns: fix ethtool loopback fail bug

2016-06-20 Thread Yisen Zhuang
eported-by: hejun Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_ethtool.c | 35 ++-- 1 file changed, 14 insertions(+), 21 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns_ethtool.c b/drivers/net/ethernet/his

[PATCH net-next 02/19] net: hns: fix hns dsaf v1 dont support tx_pause close

2016-06-20 Thread Yisen Zhuang
From: Qianqian Xie For service port, hns dsaf v1 support to close tx_pause. However, the port will be invalid when it run command ethtool to close tx_pause. This patch will fix it. Signed-off-by: Qianqian Xie Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c

[PATCH net-next 19/19] net: hns: bug fix about TSO on|off when there is traffic

2016-06-20 Thread Yisen Zhuang
Huang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_enet.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns_enet.c b/drivers/net/ethernet/hisilicon/hns/hns_enet.c index c0ce37b..d5297ec 100644 --- a/drivers

[PATCH net-next 03/19] net: hns: add skb_reset_mac_header() after skb being alloc

2016-06-20 Thread Yisen Zhuang
From: Kejian Yan HNS receives a packet without doing anything, but it should call skb_reset_mac_header() to initialize the header before using eth_hdr(). Fixes: 0d6b425a3773c3445b0f51b2f333821beaacb619 Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon

[PATCH net-next 17/19] net: hns: fix sbm default parameters config error

2016-06-20 Thread Yisen Zhuang
From: Daode Huang The default sbm config parameter leaves little buffer when there is heavy traffic, which will cause packets drop. This patch changes them to make enough buffers for handling packets. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon

[PATCH net-next 15/19] net: hns: bug fix about led control logic when link down

2016-06-20 Thread Yisen Zhuang
driver itself. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_misc.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_misc.c b/drivers/net/ethernet/hisilicon/hns

[PATCH net-next 12/19] net: hns: fix bug of getting the wrong tcam data

2016-06-20 Thread Yisen Zhuang
From: Daode Huang The current driver stores the high bit value of tcam data register to the tcam data low element, stores the low bit value of tcam data register to tcam data high element, this patch fixes this bug. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net

[PATCH net-next 14/19] net: hns: delete redundancy ring enable operations

2016-06-20 Thread Yisen Zhuang
From: Daode Huang When network interface is enabled, the ring enable operation is conducted twice. This patch deletes the redundancy code of ring enable, and integrates hnae_ae_ops.toggle_queue_status other functions to hns_ae_start. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang

[PATCH net-next 00/19] net: hns: fix some bugs in hns driver

2016-06-20 Thread Yisen Zhuang
This series includes some bugs fixed. All these patches needs to be applied after the patchset about ACPI support, so this series is floated to net-next list. The patches are: > from Daode, fixes about pfc pause frame, getting coaslesce, led control logic, TSO on|off and tcam table configuratio

[PATCH net-next 10/19] net: hns: bugfix about pfc pause frame statistics

2016-06-20 Thread Yisen Zhuang
From: Daode Huang For SoC hip06, PFC pause handled in dsaf, while hip05 in XGMAC, so change the statistics of pfc pause in dsaf and remove the old pfc pause frame statistics. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_ae_adapt.c | 6

[PATCH net-next 13/19] net: hns: add get_coalesce_range api for hns

2016-06-20 Thread Yisen Zhuang
From: Daode Huang This patch adds get_coalesce_range api for hns, it shows range of coalesce usecs and frames that can be set on this interface. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hnae.h | 5 + drivers/net/ethernet

[PATCH net-next 09/19] net: hns: fix the wrong speed for bond

2016-06-20 Thread Yisen Zhuang
bond's wrong speed. Thus only one state machine should be used and if phy_state_machine is used, it does not need to do hns_nic_update_link_status(). Signed-off-by: Qianqian Xie Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_enet.c| 43 +--- dr

[PATCH net-next 18/19] net: hns: change the default coalesce usecs

2016-06-20 Thread Yisen Zhuang
From: Daode Huang The default coalesce timeout is 3us, which is will cause CPU usage is too high. This patch change it to 50us in order to reduce CPU usage and the value makes sure network latency also meets requirement. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net

[PATCH net-next 01/19] net: hns: bug fix of ge reset sequence

2016-06-20 Thread Yisen Zhuang
From: Qianqian Xie The bit fileds of PPE reset register are different between HNS v1 and HNS v2, but the current procedure just only match HNS v1. Here is a patch to fix it. Signed-off-by: Kejian Yan Signed-off-by: Qianqian Xie Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon

[PATCH net-next 11/19] net: hns: add spin lock for tcam table operation

2016-06-20 Thread Yisen Zhuang
From: Daode Huang This patch adds spin lock for tcam table operation, there maybe a race condition happens when more than one thread try to change the tcam talbe entries. Signed-off-by: Daode Huang Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 25

[PATCH net-next 07/19] net: hns: select Hilink before serdes loopback for HNS V2

2016-06-20 Thread Yisen Zhuang
From: Kejian Yan As Hilink3 and Hilink4 use the same xge training and xge u adaptor for HNSv2, it needs to select which Hilink to be set before relative serdes being configed. The hilink_access_sel is the register to do that. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers

Re: [patch net-next] net: hns: add skb_reset_mac_header() after skb being alloc

2016-06-20 Thread Yisen Zhuang
在 2016/6/15 18:30, Yisen Zhuang 写道: > Hi David, > > Thanks for your suggestions. > > Please see my comments below. > > Thanks, > > Yisen > > 在 2016/6/15 13:41, David Miller 写道: >> From: Yisen Zhuang >> Date: Mon, 13 Jun 2016 20:41:22 +0800 &g

Re: [patch net-next] net: hns: add skb_reset_mac_header() after skb being alloc

2016-06-15 Thread Yisen Zhuang
Hi David, Thanks for your suggestions. Please see my comments below. Thanks, Yisen 在 2016/6/15 13:41, David Miller 写道: > From: Yisen Zhuang > Date: Mon, 13 Jun 2016 20:41:22 +0800 > >> From: Kejian Yan >> >> HNS receives a packet without doing a

Re: [PATCH] net: hns: update the dependency

2016-06-15 Thread Yisen Zhuang
Hi David, I'm really sorry for this. Because i didn't receive the first two emails, i resented it a few times. I will pay more attention next time. Thanks, Yisen 在 2016/6/15 14:24, David Miller 写道: > From: Yisen Zhuang > Date: Wed, 15 Jun 2016 14:03:33 +0800 > >>

Re: [PATCH] net: hns: update the dependency

2016-06-14 Thread Yisen Zhuang
Hi David, You mean that i send this patch 3 times? I am sorry for this. I don't know why you can receive 3 times. I can only receive an email for this patch. Thanks, Yisen 在 2016/6/15 13:26, David Miller 写道: > From: Yisen Zhuang > Date: Mon, 13 Jun 2016 19:56:27 +0800 > &

[patch net-next] net: hns: add skb_reset_mac_header() after skb being alloc

2016-06-13 Thread Yisen Zhuang
From: Kejian Yan HNS receives a packet without doing anything, but it should call skb_reset_mac_header() to initialize the header before using eth_hdr(). Fixes: 0d6b425a3773c3445b0f51b2f333821beaacb619 Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon

[PATCH] net: hns: update the dependency

2016-06-13 Thread Yisen Zhuang
From: Kejian Yan After the patchset about adding support of ACPI (commit id is 6343488) being applied, HNS does not depend on OF. It depends on OF or ACPI, so the Kconfig file needs to be updated. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon

[PATCH] net: hns: update the dependency

2016-06-13 Thread Yisen Zhuang
From: Kejian Yan After the patchset about adding support of ACPI (commit id is 6343488) being applied, HNS does not depend on OF. It depends on OF or ACPI, so the Kconfig file needs to be updated. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon

[PATCH] net: hns: update the dependency

2016-06-13 Thread Yisen Zhuang
From: Kejian Yan After the patchset about adding support of ACPI (commit id is 6343488) being applied, HNS does not depend on OF. It depends on OF or ACPI, so the Kconfig file needs to be updated. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon

[PATCH v4 net-next 06/13] net: hns: use platform_get_irq instead of irq_of_parse_and_map

2016-06-02 Thread Yisen Zhuang
From: Kejian Yan As irq_of_parse_and_map is only used by DT case, it is excepted to use a uniform interface. So it is used platform_get_irq() instead. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.c | 5 ++--- 1 file changed, 2

[PATCH v4 net-next 09/13] net: hns: add dsaf misc operation method

2016-06-02 Thread Yisen Zhuang
From: Kejian Yan The misc operation for different hw platform may be different, if using current implementation, it will add a new branch on each function for every new hw platform, so we add a method for this operation. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net

[PATCH v4 net-next 08/13] net: hns: add uniform interface for phy connection

2016-06-02 Thread Yisen Zhuang
to handle that sequence by both DT and ACPI. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v2: 1. remove the redundant functions, and 2. adds fwnode match method beside DT and ACPI. v1: first submit link: https://lkml.org/lkml/2016/5/13/100 --- drivers/net

[PATCH v4 net-next 00/13] net: hns: add support of ACPI

2016-06-02 Thread Yisen Zhuang
From: Kejian Yan This series adds HNS support of acpi. The routine will call some ACPI helper functions, like acpi_dev_found() and acpi_evaluate_dsm(), which are not included in other cases. In order to make system compile successfully in other cases except ACPI, it needs to add relative stub fun

[PATCH v4 net-next 01/13] ACPI: bus: add stub acpi_dev_found() to linux/acpi.h

2016-06-02 Thread Yisen Zhuang
successfully in non-ACPI cases. Cc: Rafael J. Wysocki Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- include/linux/acpi.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/linux/acpi.h b/include/linux/acpi.h index 288fac5..3025d19 100644 --- a/include/linux/acpi.h +++ b/include

[PATCH v4 net-next 12/13] net: hns: implement the miscellaneous operation by asl

2016-06-02 Thread Yisen Zhuang
From: Kejian Yan The miscellaneous operation is implemented in BIOS, the kernel can call _DSM method help to call the implementation in ACPI case. Here is a patch to do that. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v2: use a serial function to implement the

[PATCH v4 net-next 03/13] net: hisilicon: cleanup to prepare for other cases

2016-06-02 Thread Yisen Zhuang
From: Kejian Yan Hns-mdio only supports DT case now. do some cleanup to prepare for introducing other cases later, no functional change. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v4: mii-id gets from dev_name instead of address v3: first submit Link: https

[PATCH v4 net-next 07/13] net: hns: enet specify a reference to dsaf by fwnode_handle

2016-06-02 Thread Yisen Zhuang
From: Kejian Yan As device_node is only used by DT case, it is expected to find uniform ways. So fwnode_handle is the suitable method. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v2: remove the redundant line v1: first submit link: https://lkml.org/lkml/2016/5/13

[PATCH v4 net-next 04/13] net: hisilicon: add support of acpi for hns-mdio

2016-06-02 Thread Yisen Zhuang
DSDT in ACPI case. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v2: 1. use dev_of_node instead of IS_ENABLED macro 2. Add ACPI bits v1: first submit Link: https://lkml.org/lkml/2016/5/13/93 --- drivers/net/ethernet/hisilicon/hns_mdio.c | 106

[PATCH v4 net-next 13/13] net: hns: net: hns: enet adds support of acpi

2016-06-02 Thread Yisen Zhuang
From: Kejian Yan Enet needs to get configration parameter by acpi. This patch adds support of ACPI for enet. The configuration parameter will be configed in BIOS. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v2: 1. use acpi_dev_found() instead of

[PATCH v4 net-next 11/13] net: hns: register phy device in each mac initial sequence

2016-06-02 Thread Yisen Zhuang
ff-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v2: fix the build error by kbuild test robot v1: first submit link: https://lkml.org/lkml/2016/5/13/97 --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c | 133 -- 1 file changed, 126 insertions(+), 7 dele

[PATCH v4 net-next 10/13] net: hns: dsaf adds support of acpi

2016-06-02 Thread Yisen Zhuang
From: Kejian Yan Dsaf needs to get configuration parameter by ACPI, so this patch add support of ACPI. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v2: 1. use dev_of_node() instead of IS_ENABLED() to check if it is in DT case, 2. split a new patch to implement

[PATCH v4 net-next 02/13] ACPI: bus: add stub acpi_evaluate_dsm() to linux/acpi.h

2016-06-02 Thread Yisen Zhuang
-ACPI cases. Cc: Rafael J. Wysocki Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- include/linux/acpi.h | 8 1 file changed, 8 insertions(+) diff --git a/include/linux/acpi.h b/include/linux/acpi.h index 3025d19..4d4bb49 100644 --- a/include/linux/acpi.h +++ b/include/linux

[PATCH v4 net-next 05/13] net: hns: use device_* APIs instead of of_* APIs

2016-06-02 Thread Yisen Zhuang
From: Kejian Yan OF series functions can be used only for DT case. Use unified device property function instead to support both DT and ACPI. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 9 + drivers/net/ethernet

Re: [PATCH v3 net-next 03/13] net: hisilicon: cleanup to prepare for other cases

2016-06-02 Thread Yisen Zhuang
Hi David, Thanks for your comment, i will use dev name of this mdio bus instead of address. Thanks, Yisen 在 2016/6/2 7:07, David Miller 写道: > From: Yisen Zhuang > Date: Mon, 30 May 2016 20:34:14 +0800 > >> -static void hns_mdio_bus_name(char *name, struct device_node *np)

[PATCH v3 net-next 10/13] net: hns: dsaf adds support of acpi

2016-05-30 Thread Yisen Zhuang
From: Kejian Yan Dsaf needs to get configuration parameter by ACPI, so this patch add support of ACPI. Signed-off-by: Kejian Yan Reviewed-by: Andy Shevchenko Signed-off-by: Yisen Zhuang --- change log: v3: add Reviewed-by: Andy Shevchenko v2: 1. use dev_of_node() instead of IS_ENABLED

[PATCH v3 net-next 00/13] net: hns: add support of ACPI

2016-05-30 Thread Yisen Zhuang
From: Kejian Yan This series adds HNS support of acpi. The routine will call some ACPI helper functions, like acpi_dev_found() and acpi_evaluate_dsm(), which are not included in other cases. In order to make system compile successfully in other cases except ACPI, it needs to add relative stub fun

[PATCH v3 net-next 01/13] ACPI: bus: add stub acpi_dev_found() to linux/acpi.h

2016-05-30 Thread Yisen Zhuang
successfully in non-ACPI cases. Cc: Rafael J. Wysocki Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- include/linux/acpi.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/linux/acpi.h b/include/linux/acpi.h index 288fac5..3025d19 100644 --- a/include/linux/acpi.h +++ b/include

[PATCH v3 net-next 08/13] net: hns: add uniform interface for phy connection

2016-05-30 Thread Yisen Zhuang
to handle that sequence by both DT and ACPI. Signed-off-by: Kejian Yan Reviewed-by: Andy Shevchenko Signed-off-by: Yisen Zhuang --- change log: v3: add Reviewed-by: Andy Shevchenko v2: 1. remove the redundant functions, and 2. adds fwnode match method beside DT and ACPI. link: https

[PATCH v3 net-next 02/13] ACPI: bus: add stub acpi_evaluate_dsm() to linux/acpi.h

2016-05-30 Thread Yisen Zhuang
-ACPI cases. Cc: Rafael J. Wysocki Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- include/linux/acpi.h | 8 1 file changed, 8 insertions(+) diff --git a/include/linux/acpi.h b/include/linux/acpi.h index 3025d19..4d4bb49 100644 --- a/include/linux/acpi.h +++ b/include/linux

[PATCH v3 net-next 06/13] net: hns: use platform_get_irq instead of irq_of_parse_and_map

2016-05-30 Thread Yisen Zhuang
From: Kejian Yan As irq_of_parse_and_map is only used by DT case, it is excepted to use a uniform interface. So it is used platform_get_irq() instead. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.c | 5 ++--- 1 file changed, 2

[PATCH v3 net-next 04/13] net: hisilicon: add support of acpi for hns-mdio

2016-05-30 Thread Yisen Zhuang
DSDT in ACPI case. Signed-off-by: Kejian Yan Reviewed-by: Andy Shevchenko Signed-off-by: Yisen Zhuang --- change log: v3: add Reviewed-by: Andy Shevchenko v2: 1. use dev_of_node instead of IS_ENABLED macro 2. Add ACPI bits Link: https://lkml.org/lkml/2016/5/29/185 v1

[PATCH v3 net-next 11/13] net: hns: register phy device in each mac initial sequence

2016-05-30 Thread Yisen Zhuang
ff-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v2: fix the build error by kbuild test robot v1: first submit link: https://lkml.org/lkml/2016/5/13/97 --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c | 133 -- 1 file changed, 126 insertions(+), 7 dele

[PATCH v3 net-next 05/13] net: hns: use device_* APIs instead of of_* APIs

2016-05-30 Thread Yisen Zhuang
From: Kejian Yan OF series functions can be used only for DT case. Use unified device property function instead to support both DT and ACPI. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 9 + drivers/net/ethernet

[PATCH v3 net-next 13/13] net: hns: net: hns: enet adds support of acpi

2016-05-30 Thread Yisen Zhuang
From: Kejian Yan Enet needs to get configration parameter by acpi. This patch adds support of ACPI for enet. The configuration parameter will be configed in BIOS. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- change log: v2: 1. use acpi_dev_found() instead of

[PATCH v3 net-next 07/13] net: hns: enet specify a reference to dsaf by fwnode_handle

2016-05-30 Thread Yisen Zhuang
From: Kejian Yan As device_node is only used by DT case, it is expected to find uniform ways. So fwnode_handle is the suitable method. Signed-off-by: Kejian Yan Reviewed-by: Andy Shevchenko Signed-off-by: Yisen Zhuang --- change log: v3: add Reviewed-by: Andy Shevchenko v2: remove the

[PATCH v3 net-next 12/13] net: hns: implement the miscellaneous operation by asl

2016-05-30 Thread Yisen Zhuang
From: Kejian Yan The miscellaneous operation is implemented in BIOS, the kernel can call _DSM method help to call the implementation in ACPI case. Here is a patch to do that. Signed-off-by: Kejian Yan Reviewed-by: Andy Shevchenko Signed-off-by: Yisen Zhuang --- change log: v3: fix potential

[PATCH v3 net-next 09/13] net: hns: add dsaf misc operation method

2016-05-30 Thread Yisen Zhuang
From: Kejian Yan The misc operation for different hw platform may be different, if using current implementation, it will add a new branch on each function for every new hw platform, so we add a method for this operation. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net

[PATCH v3 net-next 03/13] net: hisilicon: cleanup to prepare for other cases

2016-05-30 Thread Yisen Zhuang
From: Kejian Yan Hns-mdio only supports DT case now. do some cleanup to prepare for introducing other cases later, no functional change. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns_mdio.c | 46 +++ 1 file changed

Re: [PATCH 1/1] net: hns: avoid null pointer dereference

2016-05-17 Thread Yisen Zhuang
Hi Heinrich, This patch is fine to me. Thanks, Yisen 在 2016/5/18 4:01, Heinrich Schuchardt 写道: > In the statement > assert(priv || priv->ae_handle); > the right side of || is only evaluated if priv is null. > > Signed-off-by: Heinrich Schuchardt > --- > drivers/net/ethernet/hisilicon/hns/h

[patch net-next 07/11] net: hns: dsaf adds support of acpi

2016-05-13 Thread Yisen Zhuang
From: Kejian Yan Dsaf needs to get configuration parameter by ACPI, so this patch add support of ACPI. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c | 80 +++ drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 86

[patch net-next 03/11] net: hns: use platform_get_irq instead of irq_of_parse_and_map

2016-05-13 Thread Yisen Zhuang
From: Kejian Yan As irq_of_parse_and_map is only used by OF case, it is excepted to use a uniform interface. So it is used platform_get_irq() instead. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.c | 5 ++--- 1 file changed, 2

[patch net-next 05/11] net: hns: add uniform interface for phy connection

2016-05-13 Thread Yisen Zhuang
to handle that sequence by both OF and ACPI. Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hnae.h | 2 +- drivers/net/ethernet/hisilicon/hns/hns_ae_adapt.c | 2 +- drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c | 34

[patch net-next 09/11] ACPI: bus: add stub acpi_evaluate_dsm() to linux/acpi.h

2016-05-13 Thread Yisen Zhuang
-ACPI cases. Cc: Rafael J. Wysocki Signed-off-by: Kejian Yan Signed-off-by: Yisen Zhuang --- include/linux/acpi.h | 8 1 file changed, 8 insertions(+) diff --git a/include/linux/acpi.h b/include/linux/acpi.h index e578bce..5c5566e 100644 --- a/include/linux/acpi.h +++ b/include/linux

[patch net-next 08/11] net: hns: register phy device in each mac initial sequence

2016-05-13 Thread Yisen Zhuang
ff-by: Kejian Yan Signed-off-by: Yisen Zhuang --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c | 119 +- 1 file changed, 115 insertions(+), 4 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c b/drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c

  1   2   >